Skip to content

Commit

Permalink
Testing: Ruff, Ignore UP037
Browse files Browse the repository at this point in the history
Ruff 0.4.5 changes the logic used to detect UP037,
mainly via this PR: astral-sh/ruff#11485.

I am seeing some possible false positive due to this, e.g.:

> Error: lib/rucio/core/authentication.py:571:27: UP037 Remove quotes from type annotation

where it asks to remove quotes from  a type
that's only imported in a type-checking block.

This is a very new Ruff version so it's best to check back at a later
point to see if this is a common issue for other repositories too.
  • Loading branch information
rdimaio authored and labkode committed Jun 26, 2024
1 parent d48aa77 commit 082a2f7
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions pyproject.toml
Original file line number Diff line number Diff line change
Expand Up @@ -56,6 +56,7 @@ ignore = [
"UP030", # Use implicit references for positional format fields
"UP031", # Use format specifiers instead of percent format
"UP032", # Use f-string instead of `format` call
"UP037", # Remove quotes from type annotation. Seeing possible false positives caused by https://github.com/astral-sh/ruff/pull/11485
"S101", # Pending https://github.com/rucio/rucio/issues/6680
"S105", # Pending https://github.com/rucio/rucio/issues/6696
"S108", # Pending https://github.com/rucio/rucio/issues/6655
Expand Down

0 comments on commit 082a2f7

Please sign in to comment.