Skip to content

Document link between import-outside-top-level (PLC0415) and lint.flake8-tidy-imports.banned-module-level-imports#18733

Merged
ntBre merged 3 commits into
astral-sh:mainfrom
Avasam:document-link-between-import-outside-top-level-PLC0415-and-lint_flake8-tidy-imports_banned-module-level-imports
Jul 3, 2025
Merged

Document link between import-outside-top-level (PLC0415) and lint.flake8-tidy-imports.banned-module-level-imports#18733
ntBre merged 3 commits into
astral-sh:mainfrom
Avasam:document-link-between-import-outside-top-level-PLC0415-and-lint_flake8-tidy-imports_banned-module-level-imports

Conversation

@Avasam

@Avasam Avasam commented Jun 17, 2025

Copy link
Copy Markdown
Contributor

@Avasam Avasam force-pushed the document-link-between-import-outside-top-level-PLC0415-and-lint_flake8-tidy-imports_banned-module-level-imports branch from 34f0033 to d194c49 Compare June 17, 2025 19:17
Comment thread crates/ruff_linter/src/rules/pylint/rules/import_outside_top_level.rs Outdated
Comment thread crates/ruff_workspace/src/options.rs Outdated
@Avasam Avasam changed the title Document link between import-outside-top-level-PLC0415 and lint_flake8-tidy-imports_banned-module-level-imports Document link between import-outside-top-level (PLC0415) and lint.flake8-tidy-imports.banned-module-level-imports Jun 17, 2025
@github-actions

github-actions Bot commented Jun 17, 2025

Copy link
Copy Markdown
Contributor

ruff-ecosystem results

Linter (stable)

✅ ecosystem check detected no linter changes.

Linter (preview)

✅ ecosystem check detected no linter changes.

@ntBre ntBre self-requested a review June 17, 2025 22:44
@ntBre ntBre added the documentation Improvements or additions to documentation label Jun 17, 2025

@ntBre ntBre left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thank you! This looks great. We might just need to clarify that TID253 has to be enabled. Good idea to check that!

Comment thread crates/ruff_linter/src/rules/pylint/rules/import_outside_top_level.rs Outdated
Comment thread crates/ruff_workspace/src/options.rs Outdated
@Avasam

Avasam commented Jun 21, 2025

Copy link
Copy Markdown
Contributor Author

Thanks for the review! I'm away for the weekend, I'll complete this next week. (It's not urgent anyway)

@ntBre ntBre left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks great, thank you!

@ntBre ntBre enabled auto-merge (squash) July 3, 2025 14:09
@ntBre ntBre merged commit 4f36f06 into astral-sh:main Jul 3, 2025
34 checks passed
@Avasam Avasam deleted the document-link-between-import-outside-top-level-PLC0415-and-lint_flake8-tidy-imports_banned-module-level-imports branch July 3, 2025 14:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants