Skip to content

[OSB] Redundancy in pre-commit hooks #42

@tsmbland

Description

@tsmbland

In theory, all the functionality of the following pre-commit hooks can be achieved by ruff:

  • black
  • pyupgrade
  • isort
  • interrogate
  • nbQA
  • bandit

We would advise removing these hooks and just using ruff instead. You should look through the ruff documentation and make sure you're using the appropriate configuration to get the desired behaviour.

You should configure ruff in the pyproject.toml, where you can also remove your configurations for the other tools.

Have a look at this template for inspiration. Note that the modern approach is to separately call ruff-check and ruff-format

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type
    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions