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.

Sign up for GitHub

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

Jump to bottom

Migrate formatter and linter to ruff #72

Merged
merged 3 commits into from
Nov 20, 2024
Merged

Migrate formatter and linter to ruff #72

merged 3 commits into from
Nov 20, 2024

Commits on Nov 20, 2024

  1. Migrate to same linting and formatting as in aiida-core 

    We migrate to ruff for the linter and formatter with the settings of
aiida-core as well as the pre-commit hooks used for formatting. This
makes the repository more compatible with aiida-core and also allows
solves more linter problems automatically which should speed up
development
    @agoscinski
    agoscinski committed Nov 20, 2024
    Configuration menu
    Copy the full SHA
    e6ca8d2 View commit details
    Browse the repository at this point in the history

  2. Remove following pep8 name convention 

    In several places we do not follow the pep8 naming convention but rather
GraphQL one. We therefore add the N80* linter ignores to disable the
linter for these cases.
    @agoscinski
    agoscinski committed Nov 20, 2024
    Configuration menu
    Copy the full SHA
    bf53b22 View commit details
    Browse the repository at this point in the history

  3. Fix errors produced by formatter and linter migration 

    This commit is mainly the result of running `pre-commit run --all-files`
with the previous changes in the formatter and linter. There is some
additional manual changes to fix linebreaks.
    @agoscinski
    agoscinski committed Nov 20, 2024
    Configuration menu
    Copy the full SHA
    11d8aad View commit details
    Browse the repository at this point in the history