Skip to content

Resolve BXXX series ignored Ruff rules #2381

@kattni

Description

@kattni

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 BXXX series:

  • B007
  • B018
  • B019
  • B024
  • B027
  • B904

Work through the issues as follows:

  1. Comment out each error number in the ignore list found here.
  2. Run pre-commit run -a to run the checks.
  3. Work through each of the errors in the result until they are all resolved.
  4. pre-commit run -a should pass without the errors in the ignore list.
  5. Remove the BXXX series of rules from the ignore list.
  6. If this is the last of the errors in the ignore list, remove the ignore list completely, including the comment above.
  7. 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.

Metadata

Metadata

Assignees

No one assigned

    Labels

    enhancementNew features, or improvements to existing features.good first issueIs this your first time contributing? This could be a good place to start!

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions