Skip to content

Remove rayon dependency and replace with std::thread-based parallelization#128

Merged
sile merged 4 commits intomasterfrom
remove-rayon
Feb 13, 2026
Merged

Remove rayon dependency and replace with std::thread-based parallelization#128
sile merged 4 commits intomasterfrom
remove-rayon

Conversation

@sile
Copy link
Copy Markdown
Owner

@sile sile commented Feb 13, 2026

This PR removes the rayon crate dependency and replaces its parallel iteration functionality with a custom implementation using Rust's standard library std::thread.

Key changes:

  • Removed rayon and either from Cargo.toml and Cargo.lock
  • Replaced rayon::iter::IntoParallelIterator with a custom filter_target_files function that uses std::thread::scope for manual thread pool management
  • Added helper functions contains_stdin_path and should_run_in_parallel to handle parallel execution logic, including a warning when stdin is used with parallel mode
  • Simplified parallel filtering in both format_files and check_files functions to use the new unified filter_target_files implementation
  • Uses AtomicUsize and Mutex for thread-safe coordination without external dependencies

@sile sile merged commit 0994098 into master Feb 13, 2026
8 checks passed
@sile sile deleted the remove-rayon branch February 13, 2026 11:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant