Skip to content

[pep8-naming] Fix formatting of __all__ (N816) #20301

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.

Sign up for GitHub

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

Merged
ntBre merged 2 commits into from
Sep 8, 2025
Merged

[pep8-naming] Fix formatting of __all__ (N816) #20301

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
7a44f61
Fix formatting of __all__ in N816
onerandomusername Sep 8, 2025
153d77a
Update mixed_case_variable_in_global_scope.rs
ntBre Sep 8, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions crates/ruff_linter/src/rules/pep8_naming/rules/mixed_case_variable_in_global_scope.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -18,8 +18,8 @@ use crate::rules::pep8_naming::helpers;
/// > (Let’s hope that these variables are meant for use inside one module
/// > only.) The conventions are about the same as those for functions.
/// >
/// > Modules that are designed for use via from M import * should use the
/// > __all__ mechanism to prevent exporting globals, or use the older
/// > Modules that are designed for use via `from M import *` should use the
/// > `__all__` mechanism to prevent exporting globals, or use the older
/// > convention of prefixing such globals with an underscore (which you might
/// > want to do to indicate these globals are “module non-public”).
/// >
Expand Down