fix(cli): Respect force-exclude for binary files

Fixes #1126
This commit is contained in:
Ed Page 2024-10-23 05:13:27 -05:00
parent b33ced2143
commit 1fb118d52f
3 changed files with 15 additions and 13 deletions

View file

@ -298,13 +298,20 @@ fn run_checks(args: &args::Args) -> proc_exit::ExitResult {
};
if single_threaded {
typos_cli::file::walk_path(walk.build(), selected_checks, &engine, reporter)
typos_cli::file::walk_path(
walk.build(),
selected_checks,
&engine,
reporter,
args.force_exclude,
)
} else {
typos_cli::file::walk_path_parallel(
walk.build_parallel(),
selected_checks,
&engine,
reporter,
args.force_exclude,
)
}
.map_err(|e| {

View file

@ -669,9 +669,10 @@ pub fn walk_path(
checks: &dyn FileChecker,
engine: &crate::policy::ConfigEngine<'_>,
reporter: &dyn report::Report,
force_exclude: bool,
) -> Result<(), ignore::Error> {
for entry in walk {
walk_entry(entry, checks, engine, reporter)?;
walk_entry(entry, checks, engine, reporter, force_exclude)?;
}
Ok(())
}
@ -681,11 +682,12 @@ pub fn walk_path_parallel(
checks: &dyn FileChecker,
engine: &crate::policy::ConfigEngine<'_>,
reporter: &dyn report::Report,
force_exclude: bool,
) -> Result<(), ignore::Error> {
let error: std::sync::Mutex<Result<(), ignore::Error>> = std::sync::Mutex::new(Ok(()));
walk.run(|| {
Box::new(|entry: Result<ignore::DirEntry, ignore::Error>| {
match walk_entry(entry, checks, engine, reporter) {
match walk_entry(entry, checks, engine, reporter, force_exclude) {
Ok(()) => ignore::WalkState::Continue,
Err(err) => {
*error.lock().unwrap() = Err(err);
@ -703,6 +705,7 @@ fn walk_entry(
checks: &dyn FileChecker,
engine: &crate::policy::ConfigEngine<'_>,
reporter: &dyn report::Report,
force_exclude: bool,
) -> Result<(), ignore::Error> {
let entry = match entry {
Ok(entry) => entry,
@ -722,7 +725,7 @@ fn walk_entry(
return Ok(());
}
if entry.file_type().map(|t| t.is_file()).unwrap_or(true) {
let explicit = entry.depth() == 0;
let explicit = entry.depth() == 0 && !force_exclude;
let (path, lookup_path) = if entry.is_stdin() {
let path = std::path::Path::new("-");
let cwd = std::env::current_dir().map_err(|err| {

View file

@ -1,13 +1,5 @@
bin.name = "typos"
args = "file.ignore ignore/file parent/ignore/file binary --force-exclude"
stdin = ""
stdout = """
error: `hello` should be `goodbye`
--> binary:3:1
|
3 | hello
| ^^^^^
|
"""
stdout = ""
stderr = ""
status.code = 2