Cyclic0007/ruff
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Remove IO based lints from linter benchmark (#7240)

Browse Source
This commit is contained in:
Micha Reiser
2023-09-08 13:39:50 +02:00
committed by GitHub
parent 0a07a2ca62
commit 9cb5ce750e
1 changed files with 9 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

11
crates/ruff_benchmark/benches/linter.rs
View File

@@ -1,7 +1,8 @@
use ruff::linter::lint_only;
use ruff::settings::rule_table::RuleTable;
use ruff::settings::{flags, Settings};
use ruff::source_kind::SourceKind;
use ruff::RuleSelector;
use ruff::{registry::Rule, RuleSelector};
use ruff_benchmark::criterion::{
criterion_group, criterion_main, BenchmarkGroup, BenchmarkId, Criterion, Throughput,
};
@@ -78,8 +79,14 @@ fn benchmark_default_rules(criterion: &mut Criterion) {
}
fn benchmark_all_rules(criterion: &mut Criterion) {
let mut rules: RuleTable = RuleSelector::All.into_iter().collect();
// Disable IO based rules because it is a source of flakiness
rules.disable(Rule::ShebangMissingExecutableFile);
rules.disable(Rule::ShebangNotExecutable);
let settings = Settings {
rules: RuleSelector::All.into_iter().collect(),
rules,
..Settings::default()
};