Python
/
ruff
mirror of https://github.com/astral-sh/ruff
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Use feature instead of test cfg

This commit is contained in:
Zanie 2023-09-13 15:33:16 -05:00
parent daa9b4db72
commit eccda3d3ed
5 changed files with 9 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
crates/ruff/Cargo.toml
View File

@ -89,3 +89,5 @@ default = []
schemars = ["dep:schemars"] schemars = ["dep:schemars"]
# Enables the UnreachableCode rule # Enables the UnreachableCode rule
unreachable-code = [] unreachable-code = []
# Enables rules for internal integration tests
test-rules = []

6
crates/ruff/src/codes.rs
View File

@ -865,11 +865,11 @@ pub fn code_to_rule(linter: Linter, code: &str) -> Option<(RuleGroup, Rule)> {
(Ruff, "017") => (RuleGroup::Nursery, rules::ruff::rules::QuadraticListSummation), (Ruff, "017") => (RuleGroup::Nursery, rules::ruff::rules::QuadraticListSummation),
(Ruff, "100") => (RuleGroup::Unspecified, rules::ruff::rules::UnusedNOQA), (Ruff, "100") => (RuleGroup::Unspecified, rules::ruff::rules::UnusedNOQA),
(Ruff, "200") => (RuleGroup::Unspecified, rules::ruff::rules::InvalidPyprojectToml), (Ruff, "200") => (RuleGroup::Unspecified, rules::ruff::rules::InvalidPyprojectToml),
#[cfg(test)] #[cfg(feature = "test-rules")]
(Ruff, "900") => (RuleGroup::Unspecified, rules::ruff::rules::StableTestRule), (Ruff, "900") => (RuleGroup::Unspecified, rules::ruff::rules::StableTestRule),
#[cfg(test)] #[cfg(feature = "test-rules")]
(Ruff, "901") => (RuleGroup::Nursery, rules::ruff::rules::NurseryTestRule), (Ruff, "901") => (RuleGroup::Nursery, rules::ruff::rules::NurseryTestRule),
#[cfg(test)] #[cfg(feature = "test-rules")]
(Ruff, "902") => (RuleGroup::Preview, rules::ruff::rules::PreviewTestRule), (Ruff, "902") => (RuleGroup::Preview, rules::ruff::rules::PreviewTestRule),
// flake8-django // flake8-django

2
crates/ruff/src/linter.rs
View File

@ -214,7 +214,7 @@ pub fn check_path(
} }
// Raise violations for internal test rules // Raise violations for internal test rules
#[cfg(test)] #[cfg(feature = "test-rules")]
{ {
if settings.rules.enabled(Rule::StableTestRule) { if settings.rules.enabled(Rule::StableTestRule) {
diagnostics.push(Diagnostic::new( diagnostics.push(Diagnostic::new(

4
crates/ruff/src/rules/ruff/rules/mod.rs
View File

@ -10,7 +10,7 @@ pub(crate) use mutable_class_default::*;
pub(crate) use mutable_dataclass_default::*; pub(crate) use mutable_dataclass_default::*;
pub(crate) use pairwise_over_zipped::*; pub(crate) use pairwise_over_zipped::*;
pub(crate) use static_key_dict_comprehension::*; pub(crate) use static_key_dict_comprehension::*;
#[cfg(test)] #[cfg(feature = "test-rules")]
pub(crate) use test_rules::*; pub(crate) use test_rules::*;
pub(crate) use unnecessary_iterable_allocation_for_first_element::*; pub(crate) use unnecessary_iterable_allocation_for_first_element::*;
#[cfg(feature = "unreachable-code")] #[cfg(feature = "unreachable-code")]
@ -31,7 +31,7 @@ mod mutable_class_default;
mod mutable_dataclass_default; mod mutable_dataclass_default;
mod pairwise_over_zipped; mod pairwise_over_zipped;
mod static_key_dict_comprehension; mod static_key_dict_comprehension;
#[cfg(test)] #[cfg(feature = "test-rules")]
mod test_rules; mod test_rules;
mod unnecessary_iterable_allocation_for_first_element; mod unnecessary_iterable_allocation_for_first_element;
#[cfg(feature = "unreachable-code")] #[cfg(feature = "unreachable-code")]

1
crates/ruff_cli/Cargo.toml
View File

@ -69,6 +69,7 @@ walkdir = { version = "2.3.2" }
wild = { version = "2" } wild = { version = "2" }
[dev-dependencies] [dev-dependencies]
ruff = { path = "../ruff", features = ["clap", "test-rules"] }
assert_cmd = { version = "2.0.8" } assert_cmd = { version = "2.0.8" }
# Avoid writing colored snapshots when running tests from the terminal # Avoid writing colored snapshots when running tests from the terminal
colored = { workspace = true, features = ["no-color"]} colored = { workspace = true, features = ["no-color"]}