| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,21 @@ | ||
| extend = "../.ruff.toml" # Inherit the project-wide settings | ||
|
|
||
| # Unlike Tools/, stdlib can use newer syntax than PYTHON_FOR_REGEN | ||
| target-version = "py315" | ||
|
|
||
| [lint] | ||
| select = [ | ||
| "F401", # Unused import | ||
| ] | ||
|
|
||
| [lint.per-file-ignores] | ||
| "ctypes/__init__.py" = ["F401"] # Re-exports from _ctypes | ||
| "ensurepip/__init__.py" = ["F401"] # `import zlib` availability check | ||
|
Comment thread
Copy link
Copy Markdown
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Qualityin a case like this where there's only one error, I'd usually lean towards using a noqa comment in the code rather than per-file-ignores. But, again, perhaps that's best done as a followup..
Sorry, something went wrong.
All reactions
Copy link
Copy Markdown
Member
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low QualityAnd for this.
Sorry, something went wrong.
All reactions
|
||
| "idlelib/idle_test/htest.py" = ["F401"] # Import for Windows DPI side effect | ||
| "idlelib/idle_test/test_iomenu.py" = ["F401"] # Imports checked for existence | ||
| "importlib/_abc.py" = ["F401"] # Bootstrap-sensitive _bootstrap import | ||
| "importlib/machinery.py" = ["F401"] # NamespacePath re-export | ||
| "importlib/metadata/__init__.py" = ["F401"] # Synced from importlib_metadata | ||
| "profiling/sampling/sample.py" = ["F401"] # Re-exports PROFILING_MODE_* constants | ||
| "ssl.py" = ["F401"] # Re-exports from _ssl | ||
| "warnings.py" = ["F401"] # Re-exports from _py_warnings | ||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,14 @@ | ||
| extend = "../.ruff.toml" # Inherit the project-wide settings | ||
|
|
||
| [per-file-target-version] | ||
| "jit/*.py" = "py312" # Reused-quote f-strings (PEP 701) | ||
|
|
||
| [lint] | ||
| select = [ | ||
| "F401", # Unused import | ||
| ] | ||
|
Comment thread
Copy link
Copy Markdown
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low QualityIs it needed to repeat it? It's already in the parent (project) configuration, no?
Sorry, something went wrong.
All reactions
Copy link
Copy Markdown
Member
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low QualityThe parent config doesn't define a select, so defaults to ["E4", "E7", "E9", "F"]. We don't want to run the whole F category, or those other ones, so this select overwrites that one. (There is an extend-select that is in addition to select, but we do want a smaller set here.)
Sorry, something went wrong.
All reactions
Copy link
Copy Markdown
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low QualityOh sorry, I mixed Lib/.ruff.toml and .ruff.toml. You're right.
Sorry, something went wrong.
hugovk reacted with thumbs up emoji
All reactions
|
||
|
|
||
| [lint.per-file-ignores] | ||
| "c-analyzer/c_parser/__init__.py" = ["F401"] # Re-exports from submodules | ||
| "c-analyzer/c_parser/preprocessor/__init__.py" = ["F401"] # Re-exports from submodules | ||
| "c-analyzer/c_parser/preprocessor/common.py" = ["F401"] # Re-exported by preprocessor/__init__.py | ||
|
Comment thread
Comment on lines
+11
to
+14
Copy link
Copy Markdown
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low QualityAdding __all__ and listing the re-exports in __all__ would probably be a more idiomatic way of fixing these errors (but perhaps that's something for a followup PR, since there's a chance it could be controversial :-)
Sorry, something went wrong.
All reactions
Copy link
Copy Markdown
Member
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low QualityWill open a followup for this.
Sorry, something went wrong.
All reactions
|
||
| Back | FazBrowse Home | New Git URL |
Uh oh!
There was an error while loading. Please reload this page.