Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Test to show all linter exceptions #46

Open
MaggieFero opened this issue Mar 5, 2024 · 0 comments
Open

Test to show all linter exceptions #46

MaggieFero opened this issue Mar 5, 2024 · 0 comments

Comments

@MaggieFero
Copy link
Collaborator

Is your feature request related to a problem? Please describe.
Right now, you have to deliberately look for linter exceptions by searching for each individual linter's exception rules. This isn't exactly a problem, but is a little annoying.

Describe the solution you'd like
We could use an action for a test that always passes but lists out all the current exceptions to linter rules across all files to increase the visibility of linter exceptions for ease of finding and removal.

Describe alternatives you've considered
We could also do a big cleanup push and then fail on new exceptions, but that feels draconian and might require us to delay merges from upstream in some cases. We could also just keep searching for each linter when we have time to do some cleanup, which is fine but annoying and subject to human error.

Additional context
Improving the visibility of linter exceptions would allow us to more easily locate those places and find ways to accomplish our goals without making as many exceptions to the linter rules. This could help us have cleaner and more consistent code.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant