user0/rust
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

tidy: run typos check in src root, not current dir

Browse source
This commit is contained in:
binarycat 2025-08-09 11:32:24 -05:00
parent 1cb4fd7dd1
commit 6a51eef11b
1 changed files with 17 additions and 4 deletions
Download patch file Download diff file Expand all files Collapse all files

21
src/tools/tidy/src/extra_checks/mod.rs
View file

@ -295,7 +295,7 @@ fn check_impl(
} else {
eprintln!("spellcheck files");
}
spellcheck_runner(&outdir, &cargo, &args)?;
spellcheck_runner(root_path, &outdir, &cargo, &args)?;
}
if js_lint || js_typecheck {
@ -579,12 +579,25 @@ fn shellcheck_runner(args: &[&OsStr]) -> Result<(), Error> {
}
/// Ensure that spellchecker is installed then run it at the given path
fn spellcheck_runner(outdir: &Path, cargo: &Path, args: &[&str]) -> Result<(), Error> {
fn spellcheck_runner(
src_root: &Path,
outdir: &Path,
cargo: &Path,
args: &[&str],
) -> Result<(), Error> {
let bin_path =
crate::ensure_version_or_cargo_install(outdir, cargo, "typos-cli", "typos", "1.34.0")?;
let status = Command::new(bin_path).args(args).status()?;
if status.success() { Ok(()) } else { Err(Error::FailedCheck("typos")) }
match Command::new(bin_path).current_dir(src_root).args(args).status() {
Ok(status) => {
if status.success() {
Ok(())
} else {
Err(Error::FailedCheck("typos"))
}
}
Err(err) => Err(Error::Generic(format!("failed to run typos tool: {err:?}"))),
}
}
/// Check git for tracked files matching an extension