language-servers/ruff
1
0
Fork 0
mirror of https://github.com/astral-sh/ruff.git synced 2025-10-22 16:22:52 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 4

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

Browse source 
<!--
Thank you for contributing to Ruff/ty! To help us out with reviewing,
please consider the following:

- Does this pull request include a summary of the change? (See below.)
- Does this pull request include a descriptive title? (Please prefix
with `[ty]` for ty pull
  requests.)
- Does this pull request include references to any relevant issues?
-->

## Summary

Noticed this was not escaped when writing a project that parses the
result of `ruff rule --outputformat json`. This is visible here:
<https://docs.astral.sh/ruff/rules/mixed-case-variable-in-global-scope/#why-is-this-bad>

## Test Plan

documentation only

---------

Co-authored-by: Brent Westbrook <36778786+ntBre@users.noreply.github.com>
This commit is contained in:
arielle 2025-09-08 10:40:38 -04:00 • committed by GitHub
parent 916968d0ff
commit ab86ae1760
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
1 changed files with 2 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

4
crates/ruff_linter/src/rules/pep8_naming/rules/mixed_case_variable_in_global_scope.rs
View file

@ -18,8 +18,8 @@ use crate::rules::pep8_naming::helpers;
/// > (Let’s hope that these variables are meant for use inside one module /// > (Let’s hope that these variables are meant for use inside one module
/// > only.) The conventions are about the same as those for functions. /// > only.) The conventions are about the same as those for functions.
/// > /// >
/// > Modules that are designed for use via from M import * should use the /// > Modules that are designed for use via `from M import *` should use the
/// > __all__ mechanism to prevent exporting globals, or use the older /// > `__all__` mechanism to prevent exporting globals, or use the older
/// > convention of prefixing such globals with an underscore (which you might /// > convention of prefixing such globals with an underscore (which you might
/// > want to do to indicate these globals are “module non-public”). /// > want to do to indicate these globals are “module non-public”).
/// > /// >