move rule to pylint

crates/ruff_linter/src/checkers/ast/analyze/statement.rs

@@ -369,7 +369,7 @@ pub(crate) fn statement(stmt: &Stmt, checker: &mut Checker) {
             if checker.enabled(Rule::UnreachableCode) {
                 checker
                     .diagnostics
-                    .extend(ruff::rules::unreachable::in_function(name, body));
+                    .extend(pylint::rules::in_function(name, body));
             }
             if checker.enabled(Rule::ReimplementedOperator) {
                 refurb::rules::reimplemented_operator(checker, &function_def.into());

crates/ruff_linter/src/codes.rs

@@ -268,6 +268,7 @@ pub fn code_to_rule(linter: Linter, code: &str) -> Option<(RuleGroup, Rule)> {
         (Pylint, "R6104") => (RuleGroup::Preview, rules::pylint::rules::NonAugmentedAssignment),
         (Pylint, "R6201") => (RuleGroup::Preview, rules::pylint::rules::LiteralMembership),
         (Pylint, "R6301") => (RuleGroup::Preview, rules::pylint::rules::NoSelfUse),
+        (Pylint, "W0101") => (RuleGroup::Preview, rules::pylint::rules::UnreachableCode),
         (Pylint, "W0108") => (RuleGroup::Preview, rules::pylint::rules::UnnecessaryLambda),
         (Pylint, "W0177") => (RuleGroup::Preview, rules::pylint::rules::NanComparison),
         (Pylint, "W0120") => (RuleGroup::Stable, rules::pylint::rules::UselessElseOnLoop),
@@ -960,7 +961,6 @@ pub fn code_to_rule(linter: Linter, code: &str) -> Option<(RuleGroup, Rule)> {
         (Ruff, "011") => (RuleGroup::Removed, rules::ruff::rules::RuffStaticKeyDictComprehension),
         (Ruff, "012") => (RuleGroup::Stable, rules::ruff::rules::MutableClassDefault),
         (Ruff, "013") => (RuleGroup::Stable, rules::ruff::rules::ImplicitOptional),
-        (Ruff, "014") => (RuleGroup::Preview, rules::ruff::rules::UnreachableCode),
         (Ruff, "015") => (RuleGroup::Stable, rules::ruff::rules::UnnecessaryIterableAllocationForFirstElement),
         (Ruff, "016") => (RuleGroup::Stable, rules::ruff::rules::InvalidIndexType),
         (Ruff, "017") => (RuleGroup::Stable, rules::ruff::rules::QuadraticListSummation),

crates/ruff_linter/src/rules/pylint/mod.rs

@@ -161,6 +161,7 @@ mod tests {
     #[test_case(Rule::UselessImportAlias, Path::new("import_aliasing.py"))]
     #[test_case(Rule::UselessReturn, Path::new("useless_return.py"))]
     #[test_case(Rule::UselessWithLock, Path::new("useless_with_lock.py"))]
+    #[test_case(Rule::UnreachableCode, Path::new("unreachable.py"))]
     #[test_case(
         Rule::YieldFromInAsyncFunction,
         Path::new("yield_from_in_async_function.py")

crates/ruff_linter/src/rules/pylint/rules/mod.rs

@@ -95,6 +95,7 @@ pub(crate) use unnecessary_direct_lambda_call::*;
 pub(crate) use unnecessary_dunder_call::*;
 pub(crate) use unnecessary_lambda::*;
 pub(crate) use unnecessary_list_index_lookup::*;
+pub(crate) use unreachable::*;
 pub(crate) use unspecified_encoding::*;
 pub(crate) use useless_else_on_loop::*;
 pub(crate) use useless_exception_statement::*;
@@ -201,6 +202,7 @@ mod unnecessary_direct_lambda_call;
 mod unnecessary_dunder_call;
 mod unnecessary_lambda;
 mod unnecessary_list_index_lookup;
+mod unreachable;
 mod unspecified_encoding;
 mod useless_else_on_loop;
 mod useless_exception_statement;

crates/ruff_linter/src/rules/ruff/rules/snapshots/ruff_linter__rules__ruff__rules__unreachable__tests__assert.py.md.snap → crates/ruff_linter/src/rules/pylint/rules/snapshots/ruff_linter__rules__pylint__rules__unreachable__tests__assert.py.md.snap

@@ -1,5 +1,5 @@
 ---
-source: crates/ruff_linter/src/rules/ruff/rules/unreachable.rs
+source: crates/ruff_linter/src/rules/pylint/rules/unreachable.rs
 description: "This is a Mermaid graph. You can use https://mermaid.live to visualize it as a diagram."
 ---
 ## Function 0

crates/ruff_linter/src/rules/ruff/rules/snapshots/ruff_linter__rules__ruff__rules__unreachable__tests__async-for.py.md.snap → crates/ruff_linter/src/rules/pylint/rules/snapshots/ruff_linter__rules__pylint__rules__unreachable__tests__async-for.py.md.snap

@@ -1,5 +1,5 @@
 ---
-source: crates/ruff_linter/src/rules/ruff/rules/unreachable.rs
+source: crates/ruff_linter/src/rules/pylint/rules/unreachable.rs
 description: "This is a Mermaid graph. You can use https://mermaid.live to visualize it as a diagram."
 ---
 ## Function 0

crates/ruff_linter/src/rules/ruff/rules/snapshots/ruff_linter__rules__ruff__rules__unreachable__tests__for.py.md.snap → crates/ruff_linter/src/rules/pylint/rules/snapshots/ruff_linter__rules__pylint__rules__unreachable__tests__for.py.md.snap

@@ -1,5 +1,5 @@
 ---
-source: crates/ruff_linter/src/rules/ruff/rules/unreachable.rs
+source: crates/ruff_linter/src/rules/pylint/rules/unreachable.rs
 description: "This is a Mermaid graph. You can use https://mermaid.live to visualize it as a diagram."
 ---
 ## Function 0

crates/ruff_linter/src/rules/ruff/rules/snapshots/ruff_linter__rules__ruff__rules__unreachable__tests__if.py.md.snap → crates/ruff_linter/src/rules/pylint/rules/snapshots/ruff_linter__rules__pylint__rules__unreachable__tests__if.py.md.snap

@@ -1,5 +1,5 @@
 ---
-source: crates/ruff_linter/src/rules/ruff/rules/unreachable.rs
+source: crates/ruff_linter/src/rules/pylint/rules/unreachable.rs
 description: "This is a Mermaid graph. You can use https://mermaid.live to visualize it as a diagram."
 ---
 ## Function 0

crates/ruff_linter/src/rules/ruff/rules/snapshots/ruff_linter__rules__ruff__rules__unreachable__tests__match.py.md.snap → crates/ruff_linter/src/rules/pylint/rules/snapshots/ruff_linter__rules__pylint__rules__unreachable__tests__match.py.md.snap

@@ -1,5 +1,5 @@
 ---
-source: crates/ruff_linter/src/rules/ruff/rules/unreachable.rs
+source: crates/ruff_linter/src/rules/pylint/rules/unreachable.rs
 description: "This is a Mermaid graph. You can use https://mermaid.live to visualize it as a diagram."
 ---
 ## Function 0

crates/ruff_linter/src/rules/ruff/rules/snapshots/ruff_linter__rules__ruff__rules__unreachable__tests__raise.py.md.snap → crates/ruff_linter/src/rules/pylint/rules/snapshots/ruff_linter__rules__pylint__rules__unreachable__tests__raise.py.md.snap

@@ -1,5 +1,5 @@
 ---
-source: crates/ruff_linter/src/rules/ruff/rules/unreachable.rs
+source: crates/ruff_linter/src/rules/pylint/rules/unreachable.rs
 description: "This is a Mermaid graph. You can use https://mermaid.live to visualize it as a diagram."
 ---
 ## Function 0

crates/ruff_linter/src/rules/ruff/rules/snapshots/ruff_linter__rules__ruff__rules__unreachable__tests__simple.py.md.snap → crates/ruff_linter/src/rules/pylint/rules/snapshots/ruff_linter__rules__pylint__rules__unreachable__tests__simple.py.md.snap

@@ -1,5 +1,5 @@
 ---
-source: crates/ruff_linter/src/rules/ruff/rules/unreachable.rs
+source: crates/ruff_linter/src/rules/pylint/rules/unreachable.rs
 description: "This is a Mermaid graph. You can use https://mermaid.live to visualize it as a diagram."
 ---
 ## Function 0

crates/ruff_linter/src/rules/ruff/rules/snapshots/ruff_linter__rules__ruff__rules__unreachable__tests__try.py.md.snap → crates/ruff_linter/src/rules/pylint/rules/snapshots/ruff_linter__rules__pylint__rules__unreachable__tests__try.py.md.snap

@@ -1,5 +1,5 @@
 ---
-source: crates/ruff_linter/src/rules/ruff/rules/unreachable.rs
+source: crates/ruff_linter/src/rules/pylint/rules/unreachable.rs
 description: "This is a Mermaid graph. You can use https://mermaid.live to visualize it as a diagram."
 ---
 ## Function 0

crates/ruff_linter/src/rules/ruff/rules/snapshots/ruff_linter__rules__ruff__rules__unreachable__tests__while.py.md.snap → crates/ruff_linter/src/rules/pylint/rules/snapshots/ruff_linter__rules__pylint__rules__unreachable__tests__while.py.md.snap

@@ -1,5 +1,5 @@
 ---
-source: crates/ruff_linter/src/rules/ruff/rules/unreachable.rs
+source: crates/ruff_linter/src/rules/pylint/rules/unreachable.rs
 description: "This is a Mermaid graph. You can use https://mermaid.live to visualize it as a diagram."
 ---
 ## Function 0

crates/ruff_linter/src/rules/ruff/rules/unreachable.rs → crates/ruff_linter/src/rules/pylint/rules/unreachable.rs

@@ -1053,7 +1053,7 @@ mod tests {
     use std::fmt::Write;
     use test_case::test_case;
 
-    use crate::rules::ruff::rules::unreachable::{BasicBlocks, BlockIndex, Condition, NextBlock};
+    use crate::rules::pylint::rules::unreachable::{BasicBlocks, BlockIndex, Condition, NextBlock};
 
     #[test_case("simple.py")]
     #[test_case("if.py")]