-
-
Notifications
You must be signed in to change notification settings - Fork 484
Open
Labels
enhancementNew features, or improvements to existing features.New features, or improvements to existing features.good first issueIs this your first time contributing? This could be a good place to start!Is this your first time contributing? This could be a good place to start!
Description
What is the problem or limitation you are having?
When Ruff was enabled, for review purposes, the identified errors were globally ignored, with the intention of resolving them in subsequent PRs.
Describe the solution you'd like
This repo currently ignores the following rules from the EXXX series:
E501
Work through the issues as follows:
- Comment out the error number in the
ignorelist found here. - Run
pre-commit run -ato run the checks. - Work through each of the errors in the result until they are all resolved.
pre-commit run -ashould pass without the errors in theignorelist.- Remove the
E501rule from theignorelist. - If this is the last of the errors in the
ignorelist, remove theignorelist completely, including the comment above. - Submit the PR.
Describe alternatives you've considered
None.
Additional context
The SIMXXX checks are explicitly disabled in the configuration. Do not remove the commented out SIM check, and avoid resolving the SIMXXX errors.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
enhancementNew features, or improvements to existing features.New features, or improvements to existing features.good first issueIs this your first time contributing? This could be a good place to start!Is this your first time contributing? This could be a good place to start!