Rename checks.rs to registry.rs (#1566)

.github/workflows/ci.yaml

@@ -38,9 +38,9 @@ jobs:
       - run: cargo build --all --release
       - run: ./target/release/ruff_dev generate-all
       - run: git diff --quiet README.md || echo "::error file=README.md::This file is outdated. Run 'cargo +nightly dev generate-all'."
-      - run: git diff --quiet src/checks_gen.rs || echo "::error file=src/checks_gen.rs::This file is outdated. Run 'cargo +nightly dev generate-all'."
+      - run: git diff --quiet src/registry_gen.rs || echo "::error file=src/registry_gen.rs::This file is outdated. Run 'cargo +nightly dev generate-all'."
       - run: git diff --quiet ruff.schema.json || echo "::error file=ruff.schema.json::This file is outdated. Run 'cargo +nightly dev generate-all'."
-      - run: git diff --exit-code -- README.md src/checks_gen.rs ruff.schema.json
+      - run: git diff --exit-code -- README.md src/registry_gen.rs ruff.schema.json
 
   cargo-fmt:
     name: "cargo fmt"

flake8_to_ruff/src/converter.rs

@@ -1,13 +1,13 @@
 use std::collections::{BTreeSet, HashMap};
 
 use anyhow::Result;
-use ruff::checks_gen::CheckCodePrefix;
 use ruff::flake8_pytest_style::types::{
     ParametrizeNameType, ParametrizeValuesRowType, ParametrizeValuesType,
 };
 use ruff::flake8_quotes::settings::Quote;
 use ruff::flake8_tidy_imports::settings::Strictness;
 use ruff::pydocstyle::settings::Convention;
+use ruff::registry_gen::CheckCodePrefix;
 use ruff::settings::options::Options;
 use ruff::settings::pyproject::Pyproject;
 use ruff::{
@@ -345,8 +345,8 @@ mod tests {
     use std::collections::HashMap;
 
     use anyhow::Result;
-    use ruff::checks_gen::CheckCodePrefix;
     use ruff::pydocstyle::settings::Convention;
+    use ruff::registry_gen::CheckCodePrefix;
     use ruff::settings::options::Options;
     use ruff::settings::pyproject::Pyproject;
     use ruff::{flake8_quotes, pydocstyle};

flake8_to_ruff/src/parser.rs

@@ -3,8 +3,8 @@ use std::str::FromStr;
 use anyhow::{bail, Result};
 use once_cell::sync::Lazy;
 use regex::Regex;
-use ruff::checks::PREFIX_REDIRECTS;
+use ruff::registry::PREFIX_REDIRECTS;
-use ruff::checks_gen::CheckCodePrefix;
+use ruff::registry_gen::CheckCodePrefix;
 use ruff::settings::types::PatternPrefixPair;
 use rustc_hash::FxHashMap;
 
@@ -201,7 +201,7 @@ pub fn collect_per_file_ignores(
 #[cfg(test)]
 mod tests {
     use anyhow::Result;
-    use ruff::checks_gen::CheckCodePrefix;
+    use ruff::registry_gen::CheckCodePrefix;
     use ruff::settings::types::PatternPrefixPair;
 
     use crate::parser::{parse_files_to_codes_mapping, parse_prefix_codes, parse_strings};

flake8_to_ruff/src/plugin.rs

@@ -3,7 +3,7 @@ use std::fmt;
 use std::str::FromStr;
 
 use anyhow::anyhow;
-use ruff::checks_gen::CheckCodePrefix;
+use ruff::registry_gen::CheckCodePrefix;
 
 #[derive(Clone, Ord, PartialOrd, Eq, PartialEq)]
 pub enum Plugin {

registry_gen.rs

@@ -5,8 +5,8 @@ use schemars::JsonSchema;
 use serde::{Deserialize, Serialize};
 use strum_macros::{AsRefStr, EnumString};
 
-use crate::checks::CheckCode;
 use crate::one_time_warning;
+use crate::registry::CheckCode;
 
 #[derive(
     EnumString,

ruff_dev/src/generate_check_code_prefix.rs

@@ -10,7 +10,7 @@ use anyhow::{ensure, Result};
 use clap::Parser;
 use codegen::{Scope, Type, Variant};
 use itertools::Itertools;
-use ruff::checks::{CheckCode, PREFIX_REDIRECTS};
+use ruff::registry::{CheckCode, PREFIX_REDIRECTS};
 use strum::IntoEnumIterator;
 
 const ALL: &str = "ALL";
@@ -19,7 +19,7 @@ const ALL: &str = "ALL";
 #[command(author, version, about, long_about = None)]
 pub struct Cli {
     /// Write the generated source code to stdout (rather than to
-    /// `src/checks_gen.rs`).
+    /// `src/registry_gen.rs`).
     #[arg(long)]
     pub(crate) dry_run: bool,
 }
@@ -172,7 +172,7 @@ pub fn main(cli: &Cli) -> Result<()> {
     output.push_str("use strum_macros::{AsRefStr, EnumString};");
     output.push('\n');
     output.push('\n');
-    output.push_str("use crate::checks::CheckCode;");
+    output.push_str("use crate::registry::CheckCode;");
     output.push('\n');
     output.push_str("use crate::one_time_warning;");
     output.push('\n');
@@ -204,14 +204,14 @@ pub fn main(cli: &Cli) -> Result<()> {
     let Output { status, stdout, .. } = rustfmt.wait_with_output()?;
     ensure!(status.success(), "rustfmt failed with {status}");
 
-    // Write the output to `src/checks_gen.rs` (or stdout).
+    // Write the output to `src/registry_gen.rs` (or stdout).
     if cli.dry_run {
         println!("{}", String::from_utf8(stdout)?);
     } else {
         let file = PathBuf::from(env!("CARGO_MANIFEST_DIR"))
             .parent()
             .expect("Failed to find root directory")
-            .join("src/checks_gen.rs");
+            .join("registry_gen.rs");
         if fs::read(&file).map_or(true, |old| old != stdout) {
             fs::write(&file, stdout)?;
         }

ruff_dev/src/generate_rules_table.rs

@@ -3,7 +3,7 @@
 use anyhow::Result;
 use clap::Args;
 use itertools::Itertools;
-use ruff::checks::{CheckCategory, CheckCode};
+use ruff::registry::{CheckCategory, CheckCode};
 use strum::IntoEnumIterator;
 
 use crate::utils::replace_readme_section;

src/autofix/fixer.rs

@@ -7,7 +7,7 @@ use rustpython_parser::ast::Location;
 
 use crate::ast::types::Range;
 use crate::autofix::Fix;
-use crate::checks::Check;
+use crate::registry::Check;
 use crate::source_code_locator::SourceCodeLocator;
 
 #[derive(Debug, Copy, Clone, Hash)]

src/checkers/ast.rs

@@ -23,13 +23,13 @@ use crate::ast::types::{
 };
 use crate::ast::visitor::{walk_excepthandler, Visitor};
 use crate::ast::{branch_detection, cast, helpers, operations, visitor};
-use crate::checks::{Check, CheckCode, CheckKind, DeferralKeyword};
 use crate::docstrings::definition::{Definition, DefinitionKind, Docstring, Documentable};
 use crate::noqa::Directive;
 use crate::python::builtins::{BUILTINS, MAGIC_GLOBALS};
 use crate::python::future::ALL_FEATURE_NAMES;
 use crate::python::typing;
 use crate::python::typing::SubscriptKind;
+use crate::registry::{Check, CheckCode, CheckKind, DeferralKeyword};
 use crate::settings::types::PythonVersion;
 use crate::settings::{flags, Settings};
 use crate::source_code_locator::SourceCodeLocator;

src/checkers/imports.rs

@@ -5,10 +5,10 @@ use std::path::Path;
 use rustpython_parser::ast::Suite;
 
 use crate::ast::visitor::Visitor;
-use crate::checks::Check;
 use crate::directives::IsortDirectives;
 use crate::isort;
 use crate::isort::track::ImportTracker;
+use crate::registry::Check;
 use crate::settings::{flags, Settings};
 use crate::source_code_locator::SourceCodeLocator;
 use crate::source_code_style::SourceCodeStyleDetector;

src/checkers/lines.rs

@@ -1,9 +1,9 @@
 //! Lint rules based on checking raw physical lines.
 
-use crate::checks::{Check, CheckCode};
 use crate::pycodestyle::checks::{line_too_long, no_newline_at_end_of_file};
 use crate::pygrep_hooks::plugins::{blanket_noqa, blanket_type_ignore};
 use crate::pyupgrade::checks::unnecessary_coding_comment;
+use crate::registry::{Check, CheckCode};
 use crate::settings::{flags, Settings};
 
 pub fn check_lines(
@@ -80,7 +80,7 @@ pub fn check_lines(
 mod tests {
 
     use super::check_lines;
-    use crate::checks::CheckCode;
+    use crate::registry::CheckCode;
     use crate::settings::{flags, Settings};
 
     #[test]

src/checkers/noqa.rs

@@ -7,9 +7,9 @@ use rustpython_parser::ast::Location;
 
 use crate::ast::types::Range;
 use crate::autofix::Fix;
-use crate::checks::{Check, CheckCode, CheckKind, UnusedCodes, CODE_REDIRECTS};
 use crate::noqa;
 use crate::noqa::{is_file_exempt, Directive};
+use crate::registry::{Check, CheckCode, CheckKind, UnusedCodes, CODE_REDIRECTS};
 use crate::settings::{flags, Settings};
 
 pub fn check_noqa(

src/checkers/tokens.rs

@@ -2,8 +2,8 @@
 
 use rustpython_parser::lexer::{LexResult, Tok};
 
-use crate::checks::{Check, CheckCode};
 use crate::lex::docstring_detection::StateMachine;
+use crate::registry::{Check, CheckCode};
 use crate::ruff::checks::Context;
 use crate::settings::flags;
 use crate::source_code_locator::SourceCodeLocator;

src/cli.rs

@@ -4,10 +4,10 @@ use clap::{command, Parser};
 use regex::Regex;
 use rustc_hash::FxHashMap;
 
-use crate::checks::CheckCode;
-use crate::checks_gen::CheckCodePrefix;
 use crate::fs;
 use crate::logging::LogLevel;
+use crate::registry::CheckCode;
+use crate::registry_gen::CheckCodePrefix;
 use crate::settings::types::{
     FilePattern, PatternPrefixPair, PerFileIgnore, PythonVersion, SerializationFormat,
 };

src/commands.rs

@@ -17,12 +17,12 @@ use walkdir::WalkDir;
 
 use crate::autofix::fixer;
 use crate::cache::DEFAULT_CACHE_DIR_NAME;
-use crate::checks::{CheckCode, CheckKind};
 use crate::cli::Overrides;
 use crate::iterators::par_iter;
 use crate::linter::{add_noqa_to_path, lint_path, lint_stdin, Diagnostics};
 use crate::logging::LogLevel;
 use crate::message::Message;
+use crate::registry::{CheckCode, CheckKind};
 use crate::resolver::{FileDiscovery, PyprojectDiscovery};
 use crate::settings::flags;
 use crate::settings::types::SerializationFormat;

src/directives.rs

@@ -6,7 +6,7 @@ use rustpython_ast::Location;
 use rustpython_parser::lexer::{LexResult, Tok};
 
 use crate::ast::types::Range;
-use crate::checks::LintSource;
+use crate::registry::LintSource;
 use crate::{Settings, SourceCodeLocator};
 
 bitflags! {

src/eradicate/checks.rs

@@ -2,8 +2,8 @@ use rustpython_ast::Location;
 
 use crate::ast::types::Range;
 use crate::autofix::Fix;
-use crate::checks::{CheckCode, CheckKind};
 use crate::eradicate::detection::comment_contains_code;
+use crate::registry::{CheckCode, CheckKind};
 use crate::settings::flags;
 use crate::{Check, Settings, SourceCodeLocator};
 

src/eradicate/mod.rs

@@ -9,8 +9,8 @@ mod tests {
     use anyhow::Result;
     use test_case::test_case;
 
-    use crate::checks::CheckCode;
     use crate::linter::test_path;
+    use crate::registry::CheckCode;
     use crate::settings;
 
     #[test_case(CheckCode::ERA001, Path::new("ERA001.py"); "ERA001")]

src/flake8_2020/mod.rs

@@ -8,8 +8,8 @@ mod tests {
     use anyhow::Result;
     use test_case::test_case;
 
-    use crate::checks::CheckCode;
     use crate::linter::test_path;
+    use crate::registry::CheckCode;
     use crate::settings;
 
     #[test_case(CheckCode::YTT101, Path::new("YTT101.py"); "YTT101")]

src/flake8_2020/plugins.rs

@@ -4,7 +4,7 @@ use rustpython_ast::{Cmpop, Constant, Expr, ExprKind, Located};
 use crate::ast::helpers::match_module_member;
 use crate::ast::types::Range;
 use crate::checkers::ast::Checker;
-use crate::checks::{Check, CheckCode, CheckKind};
+use crate::registry::{Check, CheckCode, CheckKind};
 
 fn is_sys(checker: &Checker, expr: &Expr, target: &str) -> bool {
     match_module_member(

src/flake8_annotations/mod.rs

@@ -9,8 +9,8 @@ mod tests {
 
     use anyhow::Result;
 
-    use crate::checks::CheckCode;
     use crate::linter::test_path;
+    use crate::registry::CheckCode;
     use crate::{flake8_annotations, Settings};
 
     #[test]

src/flake8_annotations/plugins.rs

@@ -5,10 +5,10 @@ use crate::ast::types::Range;
 use crate::ast::visitor::Visitor;
 use crate::ast::{cast, helpers, visitor};
 use crate::checkers::ast::Checker;
-use crate::checks::{CheckCode, CheckKind};
 use crate::docstrings::definition::{Definition, DefinitionKind};
 use crate::flake8_annotations::fixes;
 use crate::flake8_annotations::helpers::match_function_def;
+use crate::registry::{CheckCode, CheckKind};
 use crate::visibility::Visibility;
 use crate::{visibility, Check};
 

src/flake8_bandit/mod.rs

@@ -9,8 +9,8 @@ mod tests {
     use anyhow::Result;
     use test_case::test_case;
 
-    use crate::checks::CheckCode;
     use crate::linter::test_path;
+    use crate::registry::CheckCode;
     use crate::settings;
 
     #[test_case(CheckCode::S101, Path::new("S101.py"); "S101")]

src/flake8_bandit/plugins/assert_used.rs

@@ -1,7 +1,7 @@
 use rustpython_ast::{Located, StmtKind};
 
 use crate::ast::types::Range;
-use crate::checks::{Check, CheckKind};
+use crate::registry::{Check, CheckKind};
 
 /// S101
 pub fn assert_used(stmt: &Located<StmtKind>) -> Check {

src/flake8_bandit/plugins/exec_used.rs

@@ -1,7 +1,7 @@
 use rustpython_ast::{Expr, ExprKind};
 
 use crate::ast::types::Range;
-use crate::checks::{Check, CheckKind};
+use crate::registry::{Check, CheckKind};
 
 /// S102
 pub fn exec_used(expr: &Expr, func: &Expr) -> Option<Check> {

src/flake8_bandit/plugins/hardcoded_bind_all_interfaces.rs

@@ -1,5 +1,5 @@
 use crate::ast::types::Range;
-use crate::checks::{Check, CheckKind};
+use crate::registry::{Check, CheckKind};
 
 /// S104
 pub fn hardcoded_bind_all_interfaces(value: &str, range: &Range) -> Option<Check> {

src/flake8_bandit/plugins/hardcoded_password_default.rs

@@ -1,8 +1,8 @@
 use rustpython_ast::{ArgData, Arguments, Expr, Located};
 
 use crate::ast::types::Range;
-use crate::checks::{Check, CheckKind};
 use crate::flake8_bandit::helpers::{matches_password_name, string_literal};
+use crate::registry::{Check, CheckKind};
 
 fn check_password_kwarg(arg: &Located<ArgData>, default: &Expr) -> Option<Check> {
     let string = string_literal(default)?;

src/flake8_bandit/plugins/hardcoded_password_func_arg.rs

@@ -1,8 +1,8 @@
 use rustpython_ast::Keyword;
 
 use crate::ast::types::Range;
-use crate::checks::{Check, CheckKind};
 use crate::flake8_bandit::helpers::{matches_password_name, string_literal};
+use crate::registry::{Check, CheckKind};
 
 /// S106
 pub fn hardcoded_password_func_arg(keywords: &[Keyword]) -> Vec<Check> {

src/flake8_bandit/plugins/hardcoded_password_string.rs

@@ -1,8 +1,8 @@
 use rustpython_ast::{Constant, Expr, ExprKind};
 
 use crate::ast::types::Range;
-use crate::checks::{Check, CheckKind};
 use crate::flake8_bandit::helpers::{matches_password_name, string_literal};
+use crate::registry::{Check, CheckKind};
 
 fn is_password_target(target: &Expr) -> bool {
     let target_name = match &target.node {

src/flake8_blind_except/mod.rs

@@ -8,8 +8,8 @@ mod tests {
     use anyhow::Result;
     use test_case::test_case;
 
-    use crate::checks::CheckCode;
     use crate::linter::test_path;
+    use crate::registry::CheckCode;
     use crate::settings;
 
     #[test_case(CheckCode::BLE001, Path::new("BLE.py"); "BLE001")]

src/flake8_blind_except/plugins.rs

@@ -2,7 +2,7 @@ use rustpython_ast::{Expr, ExprKind, Stmt, StmtKind};
 
 use crate::ast::types::Range;
 use crate::checkers::ast::Checker;
-use crate::checks::{Check, CheckKind};
+use crate::registry::{Check, CheckKind};
 
 /// BLE001
 pub fn blind_except(

src/flake8_boolean_trap/mod.rs

@@ -8,8 +8,8 @@ mod tests {
     use anyhow::Result;
     use test_case::test_case;
 
-    use crate::checks::CheckCode;
     use crate::linter::test_path;
+    use crate::registry::CheckCode;
     use crate::settings;
 
     #[test_case(CheckCode::FBT001, Path::new("FBT.py"); "FBT001")]

src/flake8_boolean_trap/plugins.rs

@@ -3,7 +3,7 @@ use rustpython_parser::ast::{Constant, Expr};
 
 use crate::ast::types::Range;
 use crate::checkers::ast::Checker;
-use crate::checks::{Check, CheckKind};
+use crate::registry::{Check, CheckKind};
 
 const FUNC_NAME_ALLOWLIST: &[&str] = &[
     "assertEqual",

src/flake8_bugbear/mod.rs

@@ -8,8 +8,8 @@ mod tests {
     use anyhow::Result;
     use test_case::test_case;
 
-    use crate::checks::CheckCode;
     use crate::linter::test_path;
+    use crate::registry::CheckCode;
     use crate::{flake8_bugbear, Settings};
 
     #[test_case(CheckCode::B002, Path::new("B002.py"); "B002")]

src/flake8_bugbear/plugins/abstract_base_class.rs

@@ -4,7 +4,7 @@ use rustpython_ast::{Constant, Expr, ExprKind, Keyword, Stmt, StmtKind};
 use crate::ast::helpers::match_module_member;
 use crate::ast::types::Range;
 use crate::checkers::ast::Checker;
-use crate::checks::{Check, CheckCode, CheckKind};
+use crate::registry::{Check, CheckCode, CheckKind};
 
 fn is_abc_class(
     bases: &[Expr],

src/flake8_bugbear/plugins/assert_false.rs

@@ -3,7 +3,7 @@ use rustpython_ast::{Constant, Expr, ExprContext, ExprKind, Location, Stmt, Stmt
 use crate::ast::types::Range;
 use crate::autofix::Fix;
 use crate::checkers::ast::Checker;
-use crate::checks::{Check, CheckKind};
+use crate::registry::{Check, CheckKind};
 use crate::source_code_generator::SourceCodeGenerator;
 
 fn assertion_error(msg: Option<&Expr>) -> Stmt {

src/flake8_bugbear/plugins/assert_raises_exception.rs

@@ -3,7 +3,7 @@ use rustpython_ast::{ExprKind, Stmt, Withitem};
 use crate::ast::helpers::match_module_member;
 use crate::ast::types::Range;
 use crate::checkers::ast::Checker;
-use crate::checks::{Check, CheckKind};
+use crate::registry::{Check, CheckKind};
 
 /// B017
 pub fn assert_raises_exception(checker: &mut Checker, stmt: &Stmt, items: &[Withitem]) {

src/flake8_bugbear/plugins/assignment_to_os_environ.rs

@@ -2,7 +2,7 @@ use rustpython_ast::{Expr, ExprKind};
 
 use crate::ast::types::Range;
 use crate::checkers::ast::Checker;
-use crate::checks::{Check, CheckKind};
+use crate::registry::{Check, CheckKind};
 
 /// B003
 pub fn assignment_to_os_environ(checker: &mut Checker, targets: &[Expr]) {

src/flake8_bugbear/plugins/cached_instance_method.rs

@@ -3,7 +3,7 @@ use rustpython_ast::{Expr, ExprKind};
 use crate::ast::helpers::{collect_call_paths, dealias_call_path, match_call_path};
 use crate::ast::types::{Range, ScopeKind};
 use crate::checkers::ast::Checker;
-use crate::checks::{Check, CheckKind};
+use crate::registry::{Check, CheckKind};
 
 fn is_cache_func(checker: &Checker, expr: &Expr) -> bool {
     let call_path = dealias_call_path(collect_call_paths(expr), &checker.import_aliases);

src/flake8_bugbear/plugins/cannot_raise_literal.rs

@@ -2,7 +2,7 @@ use rustpython_ast::{Expr, ExprKind};
 
 use crate::ast::types::Range;
 use crate::checkers::ast::Checker;
-use crate::checks::{Check, CheckKind};
+use crate::registry::{Check, CheckKind};
 
 /// B016
 pub fn cannot_raise_literal(checker: &mut Checker, expr: &Expr) {

src/flake8_bugbear/plugins/duplicate_exceptions.rs

@@ -6,7 +6,7 @@ use crate::ast::helpers;
 use crate::ast::types::Range;
 use crate::autofix::Fix;
 use crate::checkers::ast::Checker;
-use crate::checks::{Check, CheckCode, CheckKind};
+use crate::registry::{Check, CheckCode, CheckKind};
 use crate::source_code_generator::SourceCodeGenerator;
 
 fn type_pattern(elts: Vec<&Expr>) -> Expr {

src/flake8_bugbear/plugins/f_string_docstring.rs

@@ -2,7 +2,7 @@ use rustpython_ast::{ExprKind, Stmt, StmtKind};
 
 use crate::ast::helpers;
 use crate::checkers::ast::Checker;
-use crate::checks::{Check, CheckKind};
+use crate::registry::{Check, CheckKind};
 
 /// B021
 pub fn f_string_docstring(checker: &mut Checker, body: &[Stmt]) {

src/flake8_bugbear/plugins/function_call_argument_default.rs

@@ -8,8 +8,8 @@ use crate::ast::types::Range;
 use crate::ast::visitor;
 use crate::ast::visitor::Visitor;
 use crate::checkers::ast::Checker;
-use crate::checks::{Check, CheckKind};
 use crate::flake8_bugbear::plugins::mutable_argument_default::is_mutable_func;
+use crate::registry::{Check, CheckKind};
 
 const IMMUTABLE_FUNCS: [(&str, &str); 7] = [
     ("", "tuple"),

src/flake8_bugbear/plugins/function_uses_loop_variable.rs

@@ -6,7 +6,7 @@ use crate::ast::types::{Node, Range};
 use crate::ast::visitor;
 use crate::ast::visitor::Visitor;
 use crate::checkers::ast::Checker;
-use crate::checks::{Check, CheckKind};
+use crate::registry::{Check, CheckKind};
 
 #[derive(Default)]
 struct LoadedNamesVisitor<'a> {

src/flake8_bugbear/plugins/getattr_with_constant.rs

@@ -3,9 +3,9 @@ use rustpython_ast::{Constant, Expr, ExprContext, ExprKind, Location};
 use crate::ast::types::Range;
 use crate::autofix::Fix;
 use crate::checkers::ast::Checker;
-use crate::checks::{Check, CheckKind};
 use crate::python::identifiers::IDENTIFIER_REGEX;
 use crate::python::keyword::KWLIST;
+use crate::registry::{Check, CheckKind};
 use crate::source_code_generator::SourceCodeGenerator;
 
 fn attribute(value: &Expr, attr: &str) -> Expr {

src/flake8_bugbear/plugins/jump_statement_in_finally.rs

@@ -2,7 +2,7 @@ use rustpython_ast::{Stmt, StmtKind};
 
 use crate::ast::types::Range;
 use crate::checkers::ast::Checker;
-use crate::checks::{Check, CheckKind};
+use crate::registry::{Check, CheckKind};
 
 fn walk_stmt(checker: &mut Checker, body: &[Stmt], f: fn(&Stmt) -> bool) {
     for stmt in body {

src/flake8_bugbear/plugins/loop_variable_overrides_iterator.rs

@@ -5,7 +5,7 @@ use crate::ast::types::Range;
 use crate::ast::visitor;
 use crate::ast::visitor::Visitor;
 use crate::checkers::ast::Checker;
-use crate::checks::{Check, CheckKind};
+use crate::registry::{Check, CheckKind};
 
 #[derive(Default)]
 struct NameFinder<'a> {

src/flake8_bugbear/plugins/mutable_argument_default.rs

@@ -4,7 +4,7 @@ use rustpython_ast::{Arguments, Constant, Expr, ExprKind, Operator};
 use crate::ast::helpers::{collect_call_paths, dealias_call_path, match_call_path};
 use crate::ast::types::Range;
 use crate::checkers::ast::Checker;
-use crate::checks::{Check, CheckKind};
+use crate::registry::{Check, CheckKind};
 
 const MUTABLE_FUNCS: &[(&str, &str)] = &[
     ("", "dict"),

src/flake8_bugbear/plugins/raise_without_from_inside_except.rs

@@ -3,8 +3,8 @@ use rustpython_ast::{ExprKind, Stmt, StmtKind};
 use crate::ast::types::Range;
 use crate::ast::visitor::Visitor;
 use crate::checkers::ast::Checker;
-use crate::checks::{Check, CheckKind};
 use crate::python::string::is_lower;
+use crate::registry::{Check, CheckKind};
 
 struct RaiseVisitor {
     checks: Vec<Check>,

src/flake8_bugbear/plugins/redundant_tuple_in_exception_handler.rs

@@ -3,7 +3,7 @@ use rustpython_ast::{Excepthandler, ExcepthandlerKind, ExprKind};
 use crate::ast::types::Range;
 use crate::autofix::Fix;
 use crate::checkers::ast::Checker;
-use crate::checks::{Check, CheckKind};
+use crate::registry::{Check, CheckKind};
 use crate::source_code_generator::SourceCodeGenerator;
 
 /// B013

src/flake8_bugbear/plugins/setattr_with_constant.rs

@@ -5,9 +5,9 @@ use rustpython_ast::{Constant, Expr, ExprContext, ExprKind, Location, Stmt, Stmt
 use crate::ast::types::Range;
 use crate::autofix::Fix;
 use crate::checkers::ast::Checker;
-use crate::checks::{Check, CheckKind};
 use crate::python::identifiers::IDENTIFIER_REGEX;
 use crate::python::keyword::KWLIST;
+use crate::registry::{Check, CheckKind};
 use crate::source_code_generator::SourceCodeGenerator;
 use crate::source_code_style::SourceCodeStyleDetector;
 

src/flake8_bugbear/plugins/star_arg_unpacking_after_keyword_arg.rs

@@ -2,7 +2,7 @@ use rustpython_ast::{Expr, ExprKind, Keyword};
 
 use crate::ast::types::Range;
 use crate::checkers::ast::Checker;
-use crate::checks::{Check, CheckKind};
+use crate::registry::{Check, CheckKind};
 
 /// B026
 pub fn star_arg_unpacking_after_keyword_arg(

src/flake8_bugbear/plugins/strip_with_multi_characters.rs

@@ -3,7 +3,7 @@ use rustpython_ast::{Constant, Expr, ExprKind};
 
 use crate::ast::types::Range;
 use crate::checkers::ast::Checker;
-use crate::checks::{Check, CheckKind};
+use crate::registry::{Check, CheckKind};
 
 /// B005
 pub fn strip_with_multi_characters(checker: &mut Checker, expr: &Expr, func: &Expr, args: &[Expr]) {

src/flake8_bugbear/plugins/unary_prefix_increment.rs

@@ -2,7 +2,7 @@ use rustpython_ast::{Expr, ExprKind, Unaryop};
 
 use crate::ast::types::Range;
 use crate::checkers::ast::Checker;
-use crate::checks::{Check, CheckKind};
+use crate::registry::{Check, CheckKind};
 
 /// B002
 pub fn unary_prefix_increment(checker: &mut Checker, expr: &Expr, op: &Unaryop, operand: &Expr) {

src/flake8_bugbear/plugins/unreliable_callable_check.rs

@@ -2,7 +2,7 @@ use rustpython_ast::{Constant, Expr, ExprKind};
 
 use crate::ast::types::Range;
 use crate::checkers::ast::Checker;
-use crate::checks::{Check, CheckKind};
+use crate::registry::{Check, CheckKind};
 
 /// B004
 pub fn unreliable_callable_check(checker: &mut Checker, expr: &Expr, func: &Expr, args: &[Expr]) {

src/flake8_bugbear/plugins/unused_loop_control_variable.rs

@@ -6,7 +6,7 @@ use crate::ast::visitor;
 use crate::ast::visitor::Visitor;
 use crate::autofix::Fix;
 use crate::checkers::ast::Checker;
-use crate::checks::{Check, CheckKind};
+use crate::registry::{Check, CheckKind};
 
 /// Identify all `ExprKind::Name` nodes in an AST.
 struct NameFinder<'a> {

src/flake8_bugbear/plugins/useless_comparison.rs

@@ -2,7 +2,7 @@ use rustpython_ast::{Expr, ExprKind};
 
 use crate::ast::types::Range;
 use crate::checkers::ast::Checker;
-use crate::checks::{Check, CheckKind};
+use crate::registry::{Check, CheckKind};
 
 pub fn useless_comparison(checker: &mut Checker, expr: &Expr) {
     if matches!(expr.node, ExprKind::Compare { .. }) {

src/flake8_bugbear/plugins/useless_contextlib_suppress.rs

@@ -3,7 +3,7 @@ use rustpython_ast::Expr;
 use crate::ast::helpers::{collect_call_paths, match_call_path};
 use crate::ast::types::Range;
 use crate::checkers::ast::Checker;
-use crate::checks::{Check, CheckKind};
+use crate::registry::{Check, CheckKind};
 
 /// B005
 pub fn useless_contextlib_suppress(checker: &mut Checker, expr: &Expr, args: &[Expr]) {

src/flake8_bugbear/plugins/useless_expression.rs

@@ -2,7 +2,7 @@ use rustpython_ast::{Constant, ExprKind, Stmt, StmtKind};
 
 use crate::ast::types::Range;
 use crate::checkers::ast::Checker;
-use crate::checks::{Check, CheckKind};
+use crate::registry::{Check, CheckKind};
 
 // B018
 pub fn useless_expression(checker: &mut Checker, body: &[Stmt]) {

src/flake8_bugbear/plugins/zip_without_explicit_strict.rs

@@ -2,7 +2,7 @@ use rustpython_ast::{Expr, ExprKind, Keyword};
 
 use crate::ast::types::Range;
 use crate::checkers::ast::Checker;
-use crate::checks::{Check, CheckKind};
+use crate::registry::{Check, CheckKind};
 
 /// B905
 pub fn zip_without_explicit_strict(

src/flake8_builtins/checks.rs

@@ -1,9 +1,9 @@
 use rustpython_ast::Located;
 
 use crate::ast::types::Range;
-use crate::checks::{Check, CheckKind};
 use crate::flake8_builtins::types::ShadowingType;
 use crate::python::builtins::BUILTINS;
+use crate::registry::{Check, CheckKind};
 
 /// Check builtin name shadowing.
 pub fn builtin_shadowing<T>(

src/flake8_builtins/mod.rs

@@ -9,8 +9,8 @@ mod tests {
     use anyhow::Result;
     use test_case::test_case;
 
-    use crate::checks::CheckCode;
     use crate::linter::test_path;
+    use crate::registry::CheckCode;
     use crate::settings;
 
     #[test_case(CheckCode::A001, Path::new("A001.py"); "A001")]

src/flake8_comprehensions/checks.rs

@@ -5,8 +5,8 @@ use rustpython_ast::{
 };
 
 use crate::ast::types::Range;
-use crate::checks::{Check, CheckKind};
 use crate::flake8_comprehensions::fixes;
+use crate::registry::{Check, CheckKind};
 use crate::source_code_locator::SourceCodeLocator;
 
 fn function_name(func: &Expr) -> Option<&str> {

src/flake8_comprehensions/mod.rs

@@ -9,8 +9,8 @@ mod tests {
     use anyhow::Result;
     use test_case::test_case;
 
-    use crate::checks::CheckCode;
     use crate::linter::test_path;
+    use crate::registry::CheckCode;
     use crate::settings;
 
     #[test_case(CheckCode::C400, Path::new("C400.py"); "C400")]

src/flake8_datetimez/mod.rs

@@ -8,8 +8,8 @@ mod tests {
     use anyhow::Result;
     use test_case::test_case;
 
-    use crate::checks::CheckCode;
     use crate::linter::test_path;
+    use crate::registry::CheckCode;
     use crate::settings;
 
     #[test_case(CheckCode::DTZ001, Path::new("DTZ001.py"); "DTZ001")]

src/flake8_datetimez/plugins.rs

@@ -5,7 +5,7 @@ use crate::ast::helpers::{
 };
 use crate::ast::types::Range;
 use crate::checkers::ast::Checker;
-use crate::checks::{Check, CheckKind};
+use crate::registry::{Check, CheckKind};
 
 pub fn call_datetime_without_tzinfo(
     checker: &mut Checker,

src/flake8_debugger/checks.rs

@@ -3,8 +3,8 @@ use rustpython_ast::{Expr, Stmt};
 
 use crate::ast::helpers::{collect_call_paths, dealias_call_path, match_call_path};
 use crate::ast::types::Range;
-use crate::checks::{Check, CheckKind};
 use crate::flake8_debugger::types::DebuggerUsingType;
+use crate::registry::{Check, CheckKind};
 
 const DEBUGGERS: &[(&str, &str)] = &[
     ("pdb", "set_trace"),

src/flake8_debugger/mod.rs

@@ -9,8 +9,8 @@ mod tests {
     use anyhow::Result;
     use test_case::test_case;
 
-    use crate::checks::CheckCode;
     use crate::linter::test_path;
+    use crate::registry::CheckCode;
     use crate::settings;
 
     #[test_case(CheckCode::T100, Path::new("T100.py"); "T100")]

src/flake8_errmsg/mod.rs

@@ -7,8 +7,8 @@ mod tests {
 
     use anyhow::Result;
 
-    use crate::checks::CheckCode;
     use crate::linter::test_path;
+    use crate::registry::CheckCode;
     use crate::{flake8_errmsg, settings};
 
     #[test]

src/flake8_errmsg/plugins.rs

@@ -2,7 +2,7 @@ use rustpython_ast::{Constant, Expr, ExprKind};
 
 use crate::ast::types::Range;
 use crate::checkers::ast::Checker;
-use crate::checks::{Check, CheckCode, CheckKind};
+use crate::registry::{Check, CheckCode, CheckKind};
 
 /// EM101, EM102, EM103
 pub fn string_in_exception(checker: &mut Checker, exc: &Expr) {

src/flake8_implicit_str_concat/checks.rs

@@ -3,7 +3,7 @@ use rustpython_ast::{Constant, Expr, ExprKind, Location, Operator};
 use rustpython_parser::lexer::{LexResult, Tok};
 
 use crate::ast::types::Range;
-use crate::checks::{Check, CheckKind};
+use crate::registry::{Check, CheckKind};
 use crate::source_code_locator::SourceCodeLocator;
 
 /// ISC001, ISC002

src/flake8_implicit_str_concat/mod.rs

@@ -8,8 +8,8 @@ mod tests {
     use anyhow::Result;
     use test_case::test_case;
 
-    use crate::checks::CheckCode;
     use crate::linter::test_path;
+    use crate::registry::CheckCode;
     use crate::settings;
 
     #[test_case(CheckCode::ISC001, Path::new("ISC.py"); "ISC001")]

src/flake8_import_conventions/checks.rs

@@ -2,7 +2,7 @@ use rustc_hash::FxHashMap;
 use rustpython_ast::Stmt;
 
 use crate::ast::types::Range;
-use crate::checks::{Check, CheckKind};
+use crate::registry::{Check, CheckKind};
 
 /// ICN001
 pub fn check_conventional_import(

src/flake8_import_conventions/mod.rs

@@ -9,8 +9,8 @@ mod tests {
     use anyhow::Result;
     use rustc_hash::FxHashMap;
 
-    use crate::checks::CheckCode;
     use crate::linter::test_path;
+    use crate::registry::CheckCode;
     use crate::{flake8_import_conventions, Settings};
 
     #[test]

src/flake8_print/mod.rs

@@ -8,8 +8,8 @@ mod tests {
     use anyhow::Result;
     use test_case::test_case;
 
-    use crate::checks::CheckCode;
     use crate::linter::test_path;
+    use crate::registry::CheckCode;
     use crate::settings;
 
     #[test_case(CheckCode::T201, Path::new("T201.py"); "T201")]

src/flake8_print/plugins/print_call.rs

@@ -5,7 +5,7 @@ use crate::ast::helpers::{collect_call_paths, dealias_call_path, is_const_none,
 use crate::ast::types::Range;
 use crate::autofix::helpers;
 use crate::checkers::ast::Checker;
-use crate::checks::{Check, CheckKind};
+use crate::registry::{Check, CheckKind};
 
 /// T201, T203
 pub fn print_call(checker: &mut Checker, func: &Expr, keywords: &[Keyword]) {

src/flake8_pytest_style/mod.rs

@@ -9,10 +9,10 @@ mod tests {
     use anyhow::Result;
     use test_case::test_case;
 
-    use crate::checks::CheckCode;
     use crate::flake8_pytest_style::settings::Settings;
     use crate::flake8_pytest_style::types;
     use crate::linter::test_path;
+    use crate::registry::CheckCode;
     use crate::settings;
 
     #[test_case(CheckCode::PT001, Path::new("PT001.py"), Settings::default(), "PT001_default"; "PT001_0")]

src/flake8_pytest_style/plugins/assertion.rs

@@ -6,7 +6,7 @@ use super::helpers::is_falsy_constant;
 use crate::ast::types::Range;
 use crate::ast::visitor;
 use crate::ast::visitor::Visitor;
-use crate::checks::{Check, CheckKind};
+use crate::registry::{Check, CheckKind};
 
 /// Visitor that tracks assert statements and checks if they reference
 /// the exception name.

src/flake8_pytest_style/plugins/fail.rs

@@ -3,7 +3,7 @@ use rustpython_ast::{Expr, Keyword};
 use super::helpers::{is_empty_or_null_string, is_pytest_fail, SimpleCallArgs};
 use crate::ast::types::Range;
 use crate::checkers::ast::Checker;
-use crate::checks::{Check, CheckKind};
+use crate::registry::{Check, CheckKind};
 
 pub fn fail_call(checker: &mut Checker, call: &Expr, args: &Vec<Expr>, keywords: &Vec<Keyword>) {
     if is_pytest_fail(call, checker) {

src/flake8_pytest_style/plugins/fixture.rs

@@ -10,7 +10,7 @@ use crate::ast::visitor;
 use crate::ast::visitor::Visitor;
 use crate::autofix::Fix;
 use crate::checkers::ast::Checker;
-use crate::checks::{Check, CheckCode, CheckKind};
+use crate::registry::{Check, CheckCode, CheckKind};
 
 #[derive(Default)]
 /// Visitor that skips functions

src/flake8_pytest_style/plugins/imports.rs

@@ -1,7 +1,7 @@
 use rustpython_ast::Stmt;
 
 use crate::ast::types::Range;
-use crate::checks::{Check, CheckKind};
+use crate::registry::{Check, CheckKind};
 
 fn is_pytest_or_subpackage(imported_name: &str) -> bool {
     imported_name == "pytest" || imported_name.starts_with("pytest.")

src/flake8_pytest_style/plugins/marks.rs

@@ -4,7 +4,7 @@ use super::helpers::{get_mark_decorators, get_mark_name};
 use crate::ast::types::Range;
 use crate::autofix::Fix;
 use crate::checkers::ast::Checker;
-use crate::checks::{Check, CheckCode, CheckKind};
+use crate::registry::{Check, CheckCode, CheckKind};
 
 fn pytest_mark_parentheses(
     checker: &mut Checker,

src/flake8_pytest_style/plugins/parametrize.rs

@@ -4,8 +4,8 @@ use super::helpers::is_pytest_parametrize;
 use crate::ast::types::Range;
 use crate::autofix::Fix;
 use crate::checkers::ast::Checker;
-use crate::checks::{Check, CheckCode, CheckKind};
 use crate::flake8_pytest_style::types;
+use crate::registry::{Check, CheckCode, CheckKind};
 
 fn get_parametrize_decorator<'a>(checker: &Checker, decorators: &'a [Expr]) -> Option<&'a Expr> {
     decorators

src/flake8_pytest_style/plugins/patch.rs

@@ -6,7 +6,7 @@ use crate::ast::helpers::{collect_arg_names, compose_call_path};
 use crate::ast::types::Range;
 use crate::ast::visitor;
 use crate::ast::visitor::Visitor;
-use crate::checks::{Check, CheckKind};
+use crate::registry::{Check, CheckKind};
 
 const PATCH_NAMES: &[&str] = &[
     "mocker.patch",

src/flake8_pytest_style/plugins/raises.rs

@@ -8,7 +8,7 @@ use crate::ast::helpers::{
 };
 use crate::ast::types::Range;
 use crate::checkers::ast::Checker;
-use crate::checks::{Check, CheckCode, CheckKind};
+use crate::registry::{Check, CheckCode, CheckKind};
 
 fn is_pytest_raises(
     func: &Expr,

src/flake8_quotes/checks.rs

@@ -1,8 +1,8 @@
 use rustpython_ast::Location;
 
 use crate::ast::types::Range;
-use crate::checks::{Check, CheckKind};
 use crate::flake8_quotes::settings::{Quote, Settings};
+use crate::registry::{Check, CheckKind};
 use crate::source_code_locator::SourceCodeLocator;
 
 fn good_single(quote: &Quote) -> char {

src/flake8_quotes/mod.rs

@@ -8,9 +8,9 @@ mod tests {
     use anyhow::Result;
     use test_case::test_case;
 
-    use crate::checks::CheckCode;
     use crate::flake8_quotes::settings::Quote;
     use crate::linter::test_path;
+    use crate::registry::CheckCode;
     use crate::{flake8_quotes, Settings};
 
     #[test_case(Path::new("doubles.py"))]

src/flake8_return/mod.rs

@@ -9,8 +9,8 @@ mod tests {
     use anyhow::Result;
     use test_case::test_case;
 
-    use crate::checks::CheckCode;
     use crate::linter::test_path;
+    use crate::registry::CheckCode;
     use crate::Settings;
 
     #[test_case(CheckCode::RET501, Path::new("RET501.py"); "RET501")]

src/flake8_return/plugins.rs

@@ -6,9 +6,9 @@ use crate::ast::visitor::Visitor;
 use crate::ast::whitespace::indentation;
 use crate::autofix::Fix;
 use crate::checkers::ast::Checker;
-use crate::checks::{Branch, CheckCode, CheckKind};
 use crate::flake8_return::helpers::result_exists;
 use crate::flake8_return::visitor::{ReturnVisitor, Stack};
+use crate::registry::{Branch, CheckCode, CheckKind};
 use crate::Check;
 
 /// RET501

src/flake8_simplify/mod.rs

@@ -8,8 +8,8 @@ mod tests {
     use anyhow::Result;
     use test_case::test_case;
 
-    use crate::checks::CheckCode;
     use crate::linter::test_path;
+    use crate::registry::CheckCode;
     use crate::settings;
 
     #[test_case(CheckCode::SIM118, Path::new("SIM118.py"); "SIM118")]

src/flake8_simplify/plugins/key_in_dict.rs

@@ -4,7 +4,7 @@ use crate::ast::helpers::create_expr;
 use crate::ast::types::Range;
 use crate::autofix::Fix;
 use crate::checkers::ast::Checker;
-use crate::checks::{Check, CheckKind};
+use crate::registry::{Check, CheckKind};
 use crate::source_code_generator::SourceCodeGenerator;
 
 /// SIM118

src/flake8_simplify/plugins/yoda_conditions.rs

@@ -2,7 +2,7 @@ use rustpython_ast::{Cmpop, Expr, ExprKind};
 
 use crate::ast::types::Range;
 use crate::checkers::ast::Checker;
-use crate::checks::{Check, CheckKind};
+use crate::registry::{Check, CheckKind};
 
 /// SIM300
 pub fn yoda_conditions(

src/flake8_tidy_imports/checks.rs

@@ -5,8 +5,8 @@ use super::settings::BannedApi;
 use crate::ast::helpers::match_call_path;
 use crate::ast::types::Range;
 use crate::checkers::ast::Checker;
-use crate::checks::{Check, CheckKind};
 use crate::flake8_tidy_imports::settings::Strictness;
+use crate::registry::{Check, CheckKind};
 
 /// TID252
 pub fn banned_relative_import(

src/flake8_tidy_imports/mod.rs

@@ -8,9 +8,9 @@ mod tests {
     use anyhow::Result;
     use rustc_hash::FxHashMap;
 
-    use crate::checks::CheckCode;
     use crate::flake8_tidy_imports::settings::{BannedApi, Strictness};
     use crate::linter::test_path;
+    use crate::registry::CheckCode;
     use crate::{flake8_tidy_imports, Settings};
 
     #[test]

src/flake8_unused_arguments/mod.rs

@@ -11,8 +11,8 @@ mod tests {
     use anyhow::Result;
     use test_case::test_case;
 
-    use crate::checks::CheckCode;
     use crate::linter::test_path;
+    use crate::registry::CheckCode;
     use crate::{flake8_unused_arguments, settings};
 
     #[test_case(CheckCode::ARG001, Path::new("ARG.py"); "ARG001")]

src/flake8_unused_arguments/types.rs

@@ -1,4 +1,4 @@
-use crate::checks::{CheckCode, CheckKind};
+use crate::registry::{CheckCode, CheckKind};
 
 /// An AST node that can contain arguments.
 pub enum Argumentable {

src/fs.rs

@@ -8,7 +8,7 @@ use globset::GlobMatcher;
 use path_absolutize::{path_dedot, Absolutize};
 use rustc_hash::FxHashSet;
 
-use crate::checks::CheckCode;
+use crate::registry::CheckCode;
 
 /// Extract the absolute path and basename (as strings) from a Path.
 pub fn extract_path_names(path: &Path) -> Result<(&str, &str)> {

src/isort/mod.rs

@@ -644,8 +644,8 @@ mod tests {
     use anyhow::Result;
     use test_case::test_case;
 
-    use crate::checks::CheckCode;
     use crate::linter::test_path;
+    use crate::registry::CheckCode;
     use crate::{isort, Settings};
 
     #[test_case(Path::new("add_newline_before_comments.py"))]

src/isort/plugins.rs

@@ -9,9 +9,9 @@ use crate::ast::helpers::{
 use crate::ast::types::Range;
 use crate::ast::whitespace::leading_space;
 use crate::autofix::Fix;
-use crate::checks::CheckKind;
 use crate::isort::track::Block;
 use crate::isort::{comments, format_imports};
+use crate::registry::CheckKind;
 use crate::settings::flags;
 use crate::source_code_style::SourceCodeStyleDetector;
 use crate::{Check, Settings, SourceCodeLocator};

src/lib.rs

@@ -13,7 +13,7 @@
 
 use cfg_if::cfg_if;
 
-use crate::checks::Check;
+use crate::registry::Check;
 use crate::settings::Settings;
 use crate::source_code_locator::SourceCodeLocator;
 
@@ -21,8 +21,6 @@ mod ast;
 pub mod autofix;
 pub mod cache;
 mod checkers;
-pub mod checks;
-pub mod checks_gen;
 pub mod cli;
 mod cst;
 mod directives;
@@ -67,6 +65,8 @@ mod pygrep_hooks;
 mod pylint;
 mod python;
 mod pyupgrade;
+pub mod registry;
+pub mod registry_gen;
 pub mod resolver;
 mod ruff;
 mod rustpython_helpers;