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

Switch from `if let` to `match`

This commit is contained in:
Zanie 2024-01-30 13:57:11 -06:00
parent 4ba3ee334b
commit fedca811ed
1 changed files with 33 additions and 29 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

62
crates/ruff_workspace/src/configuration.rs
View File

@ -917,41 +917,45 @@ impl LintConfiguration {
// Deprecated rules
if kind.is_enable() {
if let RuleSelector::Linter(linter) = selector {
if linter.rules().all(|rule| rule.is_deprecated()) {
deprecated_selectors.insert(selector.clone());
match selector {
// Deprecated linter
RuleSelector::Linter(linter) => {
if linter.rules().all(|rule| rule.is_deprecated()) {
deprecated_selectors.insert(selector.clone());
}
}
}
if let RuleSelector::Rule {
prefix,
redirected_from: None,
}
| RuleSelector::Prefix {
prefix,
redirected_from: _,
} = selector
{
if prefix.rules().all(|rule| rule.is_deprecated()) {
deprecated_selectors.insert(selector.clone());
// Deprecated rule without redirect or prefix with only deprecated rules
RuleSelector::Rule {
prefix,
redirected_from: None,
}
}
if let RuleSelector::Rule {
prefix: redirected_to,
redirected_from: Some(redirected_from),
} = selector
{
if let Ok(redirected_selector) =
RuleSelector::from_str_no_redirect(redirected_from)
{
if let RuleSelector::Rule { ref prefix, .. }
| RuleSelector::Prefix { ref prefix, .. } = redirected_selector
| RuleSelector::Prefix {
prefix,
redirected_from: _,
} => {
if prefix.rules().all(|rule| rule.is_deprecated()) {
deprecated_selectors.insert(selector.clone());
}
}
// Deprecated rule with redirect
RuleSelector::Rule {
prefix: redirected_to,
redirected_from: Some(redirected_from),
} => {
if let Ok(redirected_selector) =
RuleSelector::from_str_no_redirect(redirected_from)
{
if prefix.rules().all(|rule| rule.is_deprecated()) {
deprecated_redirected_selectors
.insert((redirected_selector, redirected_to));
if let RuleSelector::Rule { ref prefix, .. }
| RuleSelector::Prefix { ref prefix, .. } = redirected_selector
{
if prefix.rules().all(|rule| rule.is_deprecated()) {
deprecated_redirected_selectors
.insert((redirected_selector, redirected_to));
}
}
}
}
_ => (),
}
}