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.

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

STY: Apply assorted ruff/refurb rules #867

Merged
merged 3 commits into from
Apr 17, 2024

Conversation

DimitriPapadopoulos
Copy link
Contributor

No description provided.

FURB118 Use `operator.or_` instead of defining a lambda
FURB118 Use `operator.add` instead of defining a lambda
FURB148 `enumerate` index is unused, use `for x in y` instead
FURB168 Prefer `is` operator over `isinstance` to check if an object is `None`
@effigies effigies changed the title STY: Apply assorted ruff/efurb rules STY: Apply assorted ruff/refurb rules Apr 17, 2024
@effigies effigies merged commit 782a115 into nipreps:master Apr 17, 2024
17 of 22 checks passed
@DimitriPapadopoulos DimitriPapadopoulos deleted the refurb branch April 17, 2024 20:01
effigies pushed a commit that referenced this pull request Jun 10, 2024
* STY: Apply ruff/refurb rule (FURB118)

FURB118 Use `operator.or_` instead of defining a lambda
FURB118 Use `operator.add` instead of defining a lambda

* STY: Apply ruff/refurb rule (FURB148)

FURB148 `enumerate` index is unused, use `for x in y` instead

* STY: Apply ruff/refurb rule (FURB168)

FURB168 Prefer `is` operator over `isinstance` to check if an object is `None`
effigies added a commit that referenced this pull request Jun 10, 2024
1.10.2 (June 10, 2024)

Bug-fix release in the 1.10.x series

* ENH: Modify FSSource to output T2 (#868)
* STY: Apply ruff/flake8-implicit-str-concat rule ISC001 (#870)
* STY: Remove extraneous quotes (#859)
* STY: Apply assorted ruff/refurb rules (#867)
* MNT: update vendored docs files (#834)
* MNT: Fix deprectation error in CI (#866)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants