Python
/
ruff
mirror of https://github.com/astral-sh/ruff
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

[ty] Change frequency of invalid `__all__` debug message 

This was being emitted for every symbol we checked, which
is clearly too frequent. This switches to emitting it once
per module.
This commit is contained in:
Andrew Gallant 2025-12-10 14:25:45 -05:00
parent 0181568fb5
commit c7086b3180
No known key found for this signature in database
GPG Key ID: 5518C8B38E0693E0
1 changed files with 10 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

18
crates/ty_ide/src/symbols.rs
View File

@ -470,6 +470,11 @@ impl<'db> SymbolVisitor<'db> {
}
fn into_flat_symbols(mut self) -> FlatSymbols {
// If `__all__` was found but wasn't recognized,
// then we emit a diagnostic message indicating as such.
if self.all_invalid {
tracing::debug!("Invalid `__all__` in `{}`", self.file.path(self.db));
}
// We want to filter out some of the symbols we collected.
// Specifically, to respect conventions around library
// interface.
@ -810,14 +815,11 @@ impl<'db> SymbolVisitor<'db> {
// if a name should be part of the exported API of a module
// or not. When there is `__all__`, we currently follow it
// strictly.
if self.all_origin.is_some() {
// If `__all__` is somehow invalid, ignore it and fall
// through as-if `__all__` didn't exist.
if self.all_invalid {
tracing::debug!("Invalid `__all__` in `{}`", self.file.path(self.db));
} else {
return self.all_names.contains(&*symbol.name);
}
//
// If `__all__` is somehow invalid, ignore it and fall
// through as-if `__all__` didn't exist.
if self.all_origin.is_some() && !self.all_invalid {
return self.all_names.contains(&*symbol.name);
}
// "Imported symbols are considered private by default. A fixed