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

Add ruff #907

Merged
merged 37 commits into from
Jan 16, 2025
Merged

Add ruff #907

merged 37 commits into from
Jan 16, 2025

Conversation

mirecheck
Copy link
Collaborator

No description provided.

@mirecheck mirecheck requested a review from tomjelinek as a code owner January 2, 2025 16:13
@mirecheck
add ruff python linter and formatter
d0fbfec
@mirecheck
ruff: re-format with ruff isort
e2e52b1
@mirecheck
ruff: re-format with ruff
2a7a99b 
deviations from black:
* https://docs.astral.sh/ruff/formatter/black/#trailing-commas-are-inserted-when-expanding-a-function-definition-with-a-single-argument
@mirecheck
ruff: re-format with ruff
ed5979d 
deviations from black:
  * https://docs.astral.sh/ruff/formatter/black/#own-line-comments-on-expressions-dont-cause-the-expression-to-expand
@mirecheck
ruff: re-format with ruff
78d53b0 
deviations from black:
* https://docs.astral.sh/ruff/formatter/black/#call-chain-calls-break-differently-in-some-cases
@mirecheck
ruff: re-format with ruff
0e8b5bf 
deviations from black:
* astral-sh/ruff#11820
@mirecheck
ruff: re-format with ruff
d3cedc3 
deviations from black:
* astral-sh/ruff#14143
@mirecheck
ruff: re-format with ruff
972f4ff 
deviations from black:
* astral-sh/ruff#8598
@mirecheck
ruff: re-format with ruff
14e0df2 
deviations from black:
* neither known deviations documentation nor ghissue
@mirecheck
ruff: fixes after format
a9c49ad 
* https://docs.astral.sh/ruff/formatter/black/#trailing-end-of-line-comments
@mirecheck
ruff: enable ruff default linters (E4, E7, E9, and F`)
4e05102
@mirecheck
ruff: enable ruff linter SIM
c4583a3 
* https://docs.astral.sh/ruff/rules/#flake8-simplify-sim
@mirecheck
gitlab-ci: add ruff, remove black and isort
2aa52cf
@mirecheck
ruff: disable rule F401 unused-import
9d57bad 
* https://docs.astral.sh/ruff/rules/unused-import/
@mirecheck
ruff: disable rule SIM102 collapsible-if
46a7e3a 
* https://docs.astral.sh/ruff/rules/collapsible-if/
@mirecheck
ruff: enable ruff linter I (isort)
6b997d5 
* https://docs.astral.sh/ruff/rules/#isort-i
@mirecheck
ruff: enable ruff linter ASYNC (flake8-async)
9ca5b2d 
* https://docs.astral.sh/ruff/rules/#flake8-async-async
@mirecheck
ruff: enable ruff linter B (flake8-bugbear)
049c67f 
* https://docs.astral.sh/ruff/rules/#flake8-bugbear-b
@mirecheck
ruff: enable ruff linter A (flake8-builtins)
cfe8cd0 
* https://docs.astral.sh/ruff/rules/#flake8-builtins-a
@mirecheck
ruff: enable ruff linter LOG (flake8-logging)
d7de580 
* https://docs.astral.sh/ruff/rules/#flake8-logging-log
@mirecheck
ruff: enable ruff linter G (flake8-logging-format)
2b842cd 
* https://docs.astral.sh/ruff/rules/#flake8-logging-format-g
@mirecheck
ruff: enable ruff linter C4 (flake8-comprehensions)
e6b6b0e 
* https://docs.astral.sh/ruff/rules/#flake8-comprehensions-c4
* disable:
  C408 https://docs.astral.sh/ruff/rules/unnecessary-collection-call/
@mirecheck
ruff: enable ruff linter PIE (flake8-pie)
72dbded 
* https://docs.astral.sh/ruff/rules/#flake8-pie-pie
@mirecheck
ruff: enable ruff linter RET (flake8-return)
59ae922 
* https://docs.astral.sh/ruff/rules/#flake8-return-ret
@mirecheck
ruff: enable ruff linter SLF (flake8-self)
42b31a1 
* https://docs.astral.sh/ruff/rules/#flake8-self-slf
@mirecheck
ruff: enable ruff linter SLOT (flake8-slots)
753e53d 
* https://docs.astral.sh/ruff/rules/#flake8-self-slf
* https://docs.python.org/3/reference/datamodel.html#slots
* https://wiki.python.org/moin/UsingSlots
@mirecheck
ruff: enable ruff linter TC (flake8-type-checking)
65d6de5 
* https://docs.astral.sh/ruff/rules/#flake8-type-checking-tc
mirecheck and others added 10 commits January 15, 2025 16:34
@mirecheck
ruff: enable ruff linter PERF (perflint)
d80e206 
* https://docs.astral.sh/ruff/rules/#perflint-perf
@mirecheck
ruff: enable ruff linter ARG (flake8-unused-arguments)
45742b6 
* https://docs.astral.sh/ruff/rules/#flake8-unused-arguments-arg
* disable:
  * ARG005 https://docs.astral.sh/ruff/rules/unused-lambda-argument/
@mirecheck
ruff: enable previously disabled PL (pylint) rules
88be2e1 
* PLR0911 https://docs.astral.sh/ruff/rules/too-many-return-statements/
* PLR0912 https://docs.astral.sh/ruff/rules/too-many-branches/
* PLR0913 https://docs.astral.sh/ruff/rules/too-many-arguments/
* PLR0915 https://docs.astral.sh/ruff/rules/too-many-statements/
* PLR2004 https://docs.astral.sh/ruff/rules/magic-value-comparison/
* PLR5501 https://docs.astral.sh/ruff/rules/collapsible-else-if/
* PLW0602 https://docs.astral.sh/ruff/rules/global-variable-not-assigned/
* PLW0603 https://docs.astral.sh/ruff/rules/global-statement/
* PLW1509 https://docs.astral.sh/ruff/rules/subprocess-popen-preexec-fn/
* PLW2901 https://docs.astral.sh/ruff/rules/redefined-loop-name/
@mirecheck
remove black code formatter
e88135e
@mirecheck
remove isort
6cada97
@mirecheck
remove pylint
2d3b0be
@mirecheck
update ruff
90958d7
@tomjelinek @mirecheck
remove report processor side effects from conditions
9484768
@mirecheck
fix .gitlab-ci.yml
af36463 
* add make command removed by mistake
@mirecheck
ruff: update to 0.9.1
1912e29 
* https://astral.sh/blog/ruff-v0.9.0
* https://github.com/astral-sh/ruff/releases/tag/0.9.0
* https://github.com/astral-sh/ruff/releases/tag/0.9.1
* fix A005 https://docs.astral.sh/ruff/rules/stdlib-module-shadowing/
* fix A006 https://docs.astral.sh/ruff/rules/builtin-lambda-argument-shadowing/
* https://docs.astral.sh/ruff/formatter/#f-string-formatting
* https://docs.astral.sh/ruff/formatter/#format-suppression
@tomjelinek tomjelinek merged commit aa1f640 into main Jan 16, 2025
1 check passed
@tomjelinek tomjelinek deleted the add-ruff branch January 16, 2025 08:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tomjelinek tomjelinek tomjelinek approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@mirecheck @tomjelinek