Skip to content

Conversation

@angela-lee1
Copy link
Member

Fixes #1968

What changes did you make and why did you make them?

"To streamline linting remediation across the VRMS codebase, we need a script that outputs a list of files with linting errors. This will help prioritize fixes and track progress, and align work across team members."

  • Create a reusable script, scripts/list-lint-files.js.
  • Add the command "lint:files": "node scripts/list-lint-files.js"in the package.json
  • It outputs a list of affected files in JSON format with a relative file path.
  • It outputs the total number of affected files in a TXT file.

@jng34 jng34 self-requested a review October 10, 2025 20:36
Copy link
Member

@jng34 jng34 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@angela-lee1 The script works as expected; lint report and files count were generated.

@jng34 jng34 added the ready for 2nd review When a PR is ready for another reviewer label Oct 21, 2025
@bconti123 bconti123 self-requested a review October 28, 2025 02:39
@bconti123
Copy link
Member

bconti123 commented Nov 1, 2025

@angela-lee1 The script looks good! Could you add lint-files.count.txt and lint-report.json to .gitignore? We don't usually push generated reports to the repo. What do you think?

@angela-lee1
Copy link
Member Author

Thank you, @bconti123! I will take a look at it.

@bconti123
Copy link
Member

@angela-lee1

Add the following entries to .gitignore:

# Lint reports (generated files)
lint-files.count.txt
lint-report.json

After that, I’ll approve the PR and it should be good to go.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready for 2nd review When a PR is ready for another reviewer

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Create script to output list of files containing linting errors

4 participants