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

Simplify linters using ruff #218

Merged

sbidoul merged 2 commits into OCA:master from acsone:simplify-linters

Oct 28, 2023

Member

sbidoul commented Oct 28, 2023 •

edited

Loading

Simplify the linters of this project (not the templatate... yet).

sbidoul added 2 commits

October 28, 2023 14:25


          Simplify linters using ruff

299cb36


          Update pre-commit workflow

4ab2e12

sbidoul force-pushed the simplify-linters branch from a51463e to 4ab2e12 Compare

October 28, 2023 12:25

Member Author

sbidoul commented Oct 28, 2023

I'm going ahead and merge this

there is no risk as this only lints the tests of this project, basically
the fact that the flake8 and isort config are symlinks to the template configs is problematic as this triggers surprising side effects so since this PR removes them, it simplifies maintenance of this repo

sbidoul merged commit 659c987 into OCA:master

7 checks passed

sbidoul deleted the simplify-linters branch

October 28, 2023 16:35

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet