mirrors/ruff
1
0
Fork 0
mirror of https://github.com/astral-sh/ruff.git synced 2025-09-30 13:51:16 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 3

[ty] Add missing ticks and use consistent quoting

Browse source 
This irked me while I was reading the code, so I just tried to fix what
I could see.
This commit is contained in:
Andrew Gallant 2025-07-28 10:55:41 -04:00 • committed by Andrew Gallant
parent edb2f8e997
commit 13624ce17f
1 changed files with 12 additions and 12 deletions
Download patch file Download diff file Expand all files Collapse all files

24
crates/ty_python_semantic/src/module_resolver/resolver.rs
View file

@ -194,19 +194,19 @@ impl SearchPaths {
for path in extra_paths { for path in extra_paths {
let path = canonicalize(path, system); let path = canonicalize(path, system);
tracing::debug!("Adding extra search-path '{path}'"); tracing::debug!("Adding extra search-path `{path}`");
static_paths.push(SearchPath::extra(system, path)?); static_paths.push(SearchPath::extra(system, path)?);
} }
for src_root in src_roots { for src_root in src_roots {
tracing::debug!("Adding first-party search path '{src_root}'"); tracing::debug!("Adding first-party search path `{src_root}`");
static_paths.push(SearchPath::first_party(system, src_root.to_path_buf())?); static_paths.push(SearchPath::first_party(system, src_root.to_path_buf())?);
} }
let (typeshed_versions, stdlib_path) = if let Some(typeshed) = typeshed { let (typeshed_versions, stdlib_path) = if let Some(typeshed) = typeshed {
let typeshed = canonicalize(typeshed, system); let typeshed = canonicalize(typeshed, system);
tracing::debug!("Adding custom-stdlib search path '{typeshed}'"); tracing::debug!("Adding custom-stdlib search path `{typeshed}`");
let versions_path = typeshed.join("stdlib/VERSIONS"); let versions_path = typeshed.join("stdlib/VERSIONS");
@ -235,7 +235,7 @@ impl SearchPaths {
let mut site_packages: Vec<_> = Vec::with_capacity(site_packages_paths.len()); let mut site_packages: Vec<_> = Vec::with_capacity(site_packages_paths.len());
for path in site_packages_paths { for path in site_packages_paths {
tracing::debug!("Adding site-packages search path '{path}'"); tracing::debug!("Adding site-packages search path `{path}`");
site_packages.push(SearchPath::site_packages(system, path.clone())?); site_packages.push(SearchPath::site_packages(system, path.clone())?);
} }
@ -527,7 +527,7 @@ impl<'db> Iterator for PthFileIterator<'db> {
let contents = match system.read_to_string(&path) { let contents = match system.read_to_string(&path) {
Ok(contents) => contents, Ok(contents) => contents,
Err(error) => { Err(error) => {
tracing::warn!("Failed to read .pth file '{path}': {error}"); tracing::warn!("Failed to read .pth file `{path}`: {error}");
continue; continue;
} }
}; };
@ -588,7 +588,7 @@ fn resolve_name(db: &dyn Db, name: &ModuleName, mode: ModuleResolveMode) -> Opti
Ok((package_kind, ResolvedName::FileModule(module))) => { Ok((package_kind, ResolvedName::FileModule(module))) => {
if package_kind.is_root() && module.kind.is_module() { if package_kind.is_root() && module.kind.is_module() {
tracing::trace!( tracing::trace!(
"Search path '{search_path} contains a module \ "Search path `{search_path}` contains a module \
named `{stub_name}` but a standalone module isn't a valid stub." named `{stub_name}` but a standalone module isn't a valid stub."
); );
} else { } else {
@ -600,12 +600,12 @@ fn resolve_name(db: &dyn Db, name: &ModuleName, mode: ModuleResolveMode) -> Opti
} }
Err(PackageKind::Root) => { Err(PackageKind::Root) => {
tracing::trace!( tracing::trace!(
"Search path '{search_path}' contains no stub package named `{stub_name}`." "Search path `{search_path}` contains no stub package named `{stub_name}`."
); );
} }
Err(PackageKind::Regular) => { Err(PackageKind::Regular) => {
tracing::trace!( tracing::trace!(
"Stub-package in `{search_path} doesn't contain module: `{name}`" "Stub-package in `{search_path}` doesn't contain module: `{name}`"
); );
// stub exists, but the module doesn't. // stub exists, but the module doesn't.
// TODO: Support partial packages. // TODO: Support partial packages.
@ -613,7 +613,7 @@ fn resolve_name(db: &dyn Db, name: &ModuleName, mode: ModuleResolveMode) -> Opti
} }
Err(PackageKind::Namespace) => { Err(PackageKind::Namespace) => {
tracing::trace!( tracing::trace!(
"Stub-package in `{search_path} doesn't contain module: \ "Stub-package in `{search_path}` doesn't contain module: \
`{name}` but it is a namespace package, keep going." `{name}` but it is a namespace package, keep going."
); );
// stub exists, but the module doesn't. But this is a namespace package, // stub exists, but the module doesn't. But this is a namespace package,
@ -633,18 +633,18 @@ fn resolve_name(db: &dyn Db, name: &ModuleName, mode: ModuleResolveMode) -> Opti
Err(kind) => match kind { Err(kind) => match kind {
PackageKind::Root => { PackageKind::Root => {
tracing::trace!( tracing::trace!(
"Search path '{search_path}' contains no package named `{name}`." "Search path `{search_path}` contains no package named `{name}`."
); );
} }
PackageKind::Regular => { PackageKind::Regular => {
// For regular packages, don't search the next search path. All files of that // For regular packages, don't search the next search path. All files of that
// package must be in the same location // package must be in the same location
tracing::trace!("Package in `{search_path} doesn't contain module: `{name}`"); tracing::trace!("Package in `{search_path}` doesn't contain module: `{name}`");
return None; return None;
} }
PackageKind::Namespace => { PackageKind::Namespace => {
tracing::trace!( tracing::trace!(
"Package in `{search_path} doesn't contain module: \ "Package in `{search_path}` doesn't contain module: \
`{name}` but it is a namespace package, keep going." `{name}` but it is a namespace package, keep going."
); );
} }