Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Tweak relative-imports message #9365

Merged
merged 1 commit into from
Jan 2, 2024

Conversation

tjkuson
Copy link
Contributor

@tjkuson tjkuson commented Jan 2, 2024

Summary

Changes message from "Relative imports are banned" to "Prefer absolute imports over relative imports from parent modules".

Closes #9363.

Test Plan

cargo test

Copy link
Member

@charliermarsh charliermarsh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great, thank you!

@charliermarsh charliermarsh added the documentation Improvements or additions to documentation label Jan 2, 2024
@charliermarsh charliermarsh enabled auto-merge (squash) January 2, 2024 20:05
@tjkuson
Copy link
Contributor Author

tjkuson commented Jan 2, 2024

I'm not entirely sure why this PR breaks the fuzzer...

@charliermarsh charliermarsh merged commit 38f4d9e into astral-sh:main Jan 2, 2024
15 of 16 checks passed
tjkuson added a commit to tjkuson/ruff that referenced this pull request Jan 2, 2024
@tjkuson tjkuson deleted the improve-message branch January 2, 2024 20:13
Copy link
Contributor

github-actions bot commented Jan 2, 2024

ruff-ecosystem results

Linter (stable)

ℹ️ ecosystem check detected linter changes. (+1653 -1653 violations, +0 -0 fixes in 2 projects; 39 projects unchanged)

apache/airflow (+42 -42 violations, +0 -0 fixes)

ruff check --no-cache --exit-zero --no-preview --select ALL

+ tests/providers/amazon/aws/hooks/test_eks.py:57:1: TID252 Prefer absolute imports over relative imports from parent modules
+ tests/providers/amazon/aws/hooks/test_eks.py:57:1: TID252 Prefer absolute imports over relative imports from parent modules
+ tests/providers/amazon/aws/hooks/test_eks.py:57:1: TID252 Prefer absolute imports over relative imports from parent modules
+ tests/providers/amazon/aws/hooks/test_eks.py:57:1: TID252 Prefer absolute imports over relative imports from parent modules
+ tests/providers/amazon/aws/hooks/test_eks.py:57:1: TID252 Prefer absolute imports over relative imports from parent modules
+ tests/providers/amazon/aws/hooks/test_eks.py:57:1: TID252 Prefer absolute imports over relative imports from parent modules
+ tests/providers/amazon/aws/hooks/test_eks.py:57:1: TID252 Prefer absolute imports over relative imports from parent modules
+ tests/providers/amazon/aws/hooks/test_eks.py:57:1: TID252 Prefer absolute imports over relative imports from parent modules
+ tests/providers/amazon/aws/hooks/test_eks.py:57:1: TID252 Prefer absolute imports over relative imports from parent modules
+ tests/providers/amazon/aws/hooks/test_eks.py:57:1: TID252 Prefer absolute imports over relative imports from parent modules
... 74 additional changes omitted for project

bokeh/bokeh (+1611 -1611 violations, +0 -0 fixes)

ruff check --no-cache --exit-zero --no-preview --select ALL

+ src/bokeh/application/application.py:42:1: TID252 Prefer absolute imports over relative imports from parent modules
- src/bokeh/application/application.py:42:1: TID252 Relative imports from parent modules are banned
+ src/bokeh/application/application.py:43:1: TID252 Prefer absolute imports over relative imports from parent modules
- src/bokeh/application/application.py:43:1: TID252 Relative imports from parent modules are banned
+ src/bokeh/application/application.py:44:1: TID252 Prefer absolute imports over relative imports from parent modules
- src/bokeh/application/application.py:44:1: TID252 Relative imports from parent modules are banned
+ src/bokeh/application/application.py:50:5: TID252 Prefer absolute imports over relative imports from parent modules
- src/bokeh/application/application.py:50:5: TID252 Relative imports from parent modules are banned
+ src/bokeh/application/handlers/code.py:45:1: TID252 Prefer absolute imports over relative imports from parent modules
- src/bokeh/application/handlers/code.py:45:1: TID252 Relative imports from parent modules are banned
+ src/bokeh/application/handlers/code.py:46:1: TID252 Prefer absolute imports over relative imports from parent modules
- src/bokeh/application/handlers/code.py:46:1: TID252 Relative imports from parent modules are banned
+ src/bokeh/application/handlers/code.py:47:1: TID252 Prefer absolute imports over relative imports from parent modules
+ src/bokeh/application/handlers/code.py:47:1: TID252 Prefer absolute imports over relative imports from parent modules
- src/bokeh/application/handlers/code.py:47:1: TID252 Relative imports from parent modules are banned
- src/bokeh/application/handlers/code.py:47:1: TID252 Relative imports from parent modules are banned
+ src/bokeh/application/handlers/code_runner.py:33:1: TID252 Prefer absolute imports over relative imports from parent modules
- src/bokeh/application/handlers/code_runner.py:33:1: TID252 Relative imports from parent modules are banned
+ src/bokeh/application/handlers/code_runner.py:34:1: TID252 Prefer absolute imports over relative imports from parent modules
- src/bokeh/application/handlers/code_runner.py:34:1: TID252 Relative imports from parent modules are banned
+ src/bokeh/application/handlers/directory.py:65:1: TID252 Prefer absolute imports over relative imports from parent modules
- src/bokeh/application/handlers/directory.py:65:1: TID252 Relative imports from parent modules are banned
+ src/bokeh/application/handlers/directory.py:66:1: TID252 Prefer absolute imports over relative imports from parent modules
- src/bokeh/application/handlers/directory.py:66:1: TID252 Relative imports from parent modules are banned
+ src/bokeh/application/handlers/directory.py:67:1: TID252 Prefer absolute imports over relative imports from parent modules
+ src/bokeh/application/handlers/directory.py:67:1: TID252 Prefer absolute imports over relative imports from parent modules
- src/bokeh/application/handlers/directory.py:67:1: TID252 Relative imports from parent modules are banned
- src/bokeh/application/handlers/directory.py:67:1: TID252 Relative imports from parent modules are banned
+ src/bokeh/application/handlers/directory.py:78:5: TID252 Prefer absolute imports over relative imports from parent modules
- src/bokeh/application/handlers/directory.py:78:5: TID252 Relative imports from parent modules are banned
+ src/bokeh/application/handlers/document_lifecycle.py:25:1: TID252 Prefer absolute imports over relative imports from parent modules
- src/bokeh/application/handlers/document_lifecycle.py:25:1: TID252 Relative imports from parent modules are banned
+ src/bokeh/application/handlers/function.py:44:1: TID252 Prefer absolute imports over relative imports from parent modules
- src/bokeh/application/handlers/function.py:44:1: TID252 Relative imports from parent modules are banned
+ src/bokeh/application/handlers/function.py:45:1: TID252 Prefer absolute imports over relative imports from parent modules
- src/bokeh/application/handlers/function.py:45:1: TID252 Relative imports from parent modules are banned
+ src/bokeh/application/handlers/handler.py:52:1: TID252 Prefer absolute imports over relative imports from parent modules
- src/bokeh/application/handlers/handler.py:52:1: TID252 Relative imports from parent modules are banned
+ src/bokeh/application/handlers/handler.py:53:1: TID252 Prefer absolute imports over relative imports from parent modules
+ src/bokeh/application/handlers/handler.py:53:1: TID252 Prefer absolute imports over relative imports from parent modules
- src/bokeh/application/handlers/handler.py:53:1: TID252 Relative imports from parent modules are banned
- src/bokeh/application/handlers/handler.py:53:1: TID252 Relative imports from parent modules are banned
+ src/bokeh/application/handlers/lifecycle.py:28:1: TID252 Prefer absolute imports over relative imports from parent modules
- src/bokeh/application/handlers/lifecycle.py:28:1: TID252 Relative imports from parent modules are banned
+ src/bokeh/application/handlers/lifecycle.py:29:1: TID252 Prefer absolute imports over relative imports from parent modules
+ src/bokeh/application/handlers/lifecycle.py:29:1: TID252 Prefer absolute imports over relative imports from parent modules
- src/bokeh/application/handlers/lifecycle.py:29:1: TID252 Relative imports from parent modules are banned
- src/bokeh/application/handlers/lifecycle.py:29:1: TID252 Relative imports from parent modules are banned
... 3174 additional changes omitted for project

Changes by rule (1 rules affected)

code total + violation - violation + fix - fix
TID252 3306 1653 1653 0 0

Linter (preview)

ℹ️ ecosystem check detected linter changes. (+1572 -1572 violations, +0 -0 fixes in 2 projects; 39 projects unchanged)

apache/airflow (+42 -42 violations, +0 -0 fixes)

ruff check --no-cache --exit-zero --preview --select ALL

+ tests/providers/amazon/aws/hooks/test_eks.py:57:1: TID252 Prefer absolute imports over relative imports from parent modules
+ tests/providers/amazon/aws/hooks/test_eks.py:57:1: TID252 Prefer absolute imports over relative imports from parent modules
+ tests/providers/amazon/aws/hooks/test_eks.py:57:1: TID252 Prefer absolute imports over relative imports from parent modules
+ tests/providers/amazon/aws/hooks/test_eks.py:57:1: TID252 Prefer absolute imports over relative imports from parent modules
+ tests/providers/amazon/aws/hooks/test_eks.py:57:1: TID252 Prefer absolute imports over relative imports from parent modules
+ tests/providers/amazon/aws/hooks/test_eks.py:57:1: TID252 Prefer absolute imports over relative imports from parent modules
+ tests/providers/amazon/aws/hooks/test_eks.py:57:1: TID252 Prefer absolute imports over relative imports from parent modules
+ tests/providers/amazon/aws/hooks/test_eks.py:57:1: TID252 Prefer absolute imports over relative imports from parent modules
+ tests/providers/amazon/aws/hooks/test_eks.py:57:1: TID252 Prefer absolute imports over relative imports from parent modules
+ tests/providers/amazon/aws/hooks/test_eks.py:57:1: TID252 Prefer absolute imports over relative imports from parent modules
... 74 additional changes omitted for project

bokeh/bokeh (+1530 -1530 violations, +0 -0 fixes)

ruff check --no-cache --exit-zero --preview --select ALL

+ src/bokeh/application/handlers/code.py:45:1: TID252 Prefer absolute imports over relative imports from parent modules
- src/bokeh/application/handlers/code.py:45:1: TID252 Relative imports from parent modules are banned
+ src/bokeh/application/handlers/code.py:46:1: TID252 Prefer absolute imports over relative imports from parent modules
- src/bokeh/application/handlers/code.py:46:1: TID252 Relative imports from parent modules are banned
+ src/bokeh/application/handlers/code.py:47:1: TID252 Prefer absolute imports over relative imports from parent modules
+ src/bokeh/application/handlers/code.py:47:1: TID252 Prefer absolute imports over relative imports from parent modules
- src/bokeh/application/handlers/code.py:47:1: TID252 Relative imports from parent modules are banned
- src/bokeh/application/handlers/code.py:47:1: TID252 Relative imports from parent modules are banned
+ src/bokeh/application/handlers/code_runner.py:33:1: TID252 Prefer absolute imports over relative imports from parent modules
- src/bokeh/application/handlers/code_runner.py:33:1: TID252 Relative imports from parent modules are banned
+ src/bokeh/application/handlers/code_runner.py:34:1: TID252 Prefer absolute imports over relative imports from parent modules
- src/bokeh/application/handlers/code_runner.py:34:1: TID252 Relative imports from parent modules are banned
+ src/bokeh/application/handlers/directory.py:65:1: TID252 Prefer absolute imports over relative imports from parent modules
- src/bokeh/application/handlers/directory.py:65:1: TID252 Relative imports from parent modules are banned
+ src/bokeh/application/handlers/directory.py:66:1: TID252 Prefer absolute imports over relative imports from parent modules
- src/bokeh/application/handlers/directory.py:66:1: TID252 Relative imports from parent modules are banned
+ src/bokeh/application/handlers/directory.py:67:1: TID252 Prefer absolute imports over relative imports from parent modules
+ src/bokeh/application/handlers/directory.py:67:1: TID252 Prefer absolute imports over relative imports from parent modules
- src/bokeh/application/handlers/directory.py:67:1: TID252 Relative imports from parent modules are banned
- src/bokeh/application/handlers/directory.py:67:1: TID252 Relative imports from parent modules are banned
+ src/bokeh/application/handlers/directory.py:78:5: TID252 Prefer absolute imports over relative imports from parent modules
- src/bokeh/application/handlers/directory.py:78:5: TID252 Relative imports from parent modules are banned
+ src/bokeh/application/handlers/document_lifecycle.py:25:1: TID252 Prefer absolute imports over relative imports from parent modules
- src/bokeh/application/handlers/document_lifecycle.py:25:1: TID252 Relative imports from parent modules are banned
+ src/bokeh/application/handlers/function.py:44:1: TID252 Prefer absolute imports over relative imports from parent modules
- src/bokeh/application/handlers/function.py:44:1: TID252 Relative imports from parent modules are banned
+ src/bokeh/application/handlers/function.py:45:1: TID252 Prefer absolute imports over relative imports from parent modules
- src/bokeh/application/handlers/function.py:45:1: TID252 Relative imports from parent modules are banned
+ src/bokeh/application/handlers/handler.py:52:1: TID252 Prefer absolute imports over relative imports from parent modules
- src/bokeh/application/handlers/handler.py:52:1: TID252 Relative imports from parent modules are banned
+ src/bokeh/application/handlers/handler.py:53:1: TID252 Prefer absolute imports over relative imports from parent modules
+ src/bokeh/application/handlers/handler.py:53:1: TID252 Prefer absolute imports over relative imports from parent modules
- src/bokeh/application/handlers/handler.py:53:1: TID252 Relative imports from parent modules are banned
- src/bokeh/application/handlers/handler.py:53:1: TID252 Relative imports from parent modules are banned
+ src/bokeh/application/handlers/lifecycle.py:28:1: TID252 Prefer absolute imports over relative imports from parent modules
- src/bokeh/application/handlers/lifecycle.py:28:1: TID252 Relative imports from parent modules are banned
+ src/bokeh/application/handlers/lifecycle.py:29:1: TID252 Prefer absolute imports over relative imports from parent modules
+ src/bokeh/application/handlers/lifecycle.py:29:1: TID252 Prefer absolute imports over relative imports from parent modules
- src/bokeh/application/handlers/lifecycle.py:29:1: TID252 Relative imports from parent modules are banned
- src/bokeh/application/handlers/lifecycle.py:29:1: TID252 Relative imports from parent modules are banned
+ src/bokeh/application/handlers/notebook.py:36:1: TID252 Prefer absolute imports over relative imports from parent modules
- src/bokeh/application/handlers/notebook.py:36:1: TID252 Relative imports from parent modules are banned
+ src/bokeh/application/handlers/notebook.py:37:1: TID252 Prefer absolute imports over relative imports from parent modules
- src/bokeh/application/handlers/notebook.py:37:1: TID252 Relative imports from parent modules are banned
+ src/bokeh/application/handlers/script.py:52:1: TID252 Prefer absolute imports over relative imports from parent modules
- src/bokeh/application/handlers/script.py:52:1: TID252 Relative imports from parent modules are banned
+ src/bokeh/application/handlers/server_lifecycle.py:29:1: TID252 Prefer absolute imports over relative imports from parent modules
- src/bokeh/application/handlers/server_lifecycle.py:29:1: TID252 Relative imports from parent modules are banned
... 3012 additional changes omitted for project

Changes by rule (1 rules affected)

code total + violation - violation + fix - fix
TID252 3144 1572 1572 0 0

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Improve relative-imports (TID252) message
2 participants