Try out rust-python-parser (nom-based)

2022-10-05 15:57:04 -04:00
282 changed files with 3639 additions and 13859 deletions
--- a/CONTRIBUTING.md
+++ b/CONTRIBUTING.md
@@ -14,12 +14,6 @@ your proposed change.
 ruff is written in Rust (1.63.0). You'll need to install the
 [Rust toolchain](https://www.rust-lang.org/tools/install) for development.

-You'll also need [Insta](https://insta.rs/docs/) to update snapshot tests:
-
-```shell
-cargo install cargo-insta
-```
-
 ### Development

 After cloning the repository, run ruff locally with:
@@ -64,7 +58,7 @@ similar rules are implemented.
 To add a test fixture, create a file under `resources/test/fixtures`, named to match the `CheckCode`
 you defined earlier (e.g., `E402.py`). This file should contain a variety of violations and
 non-violations designed to evaluate and demonstrate the behavior of your lint rule. Run ruff locally
-with (e.g.) `cargo run resources/test/fixtures/E402.py`. Once you're satisfied with the output,
+with (e.g.) `cargo run resources/test/fixtures/E402.py`. Once you're satisified with the output,
 codify the behavior as a snapshot test by adding a new function to the `mod tests` section of
 `src/linter.rs`, like so:

--- a/Cargo.lock
+++ b/Cargo.lock
@@ -16,7 +16,7 @@ checksum = "fcb51a0695d8f838b1ee009b3fbf66bda078cd64590202a864a8f3e8c4315c47"
 dependencies = [
 "getrandom 0.2.7",
 "once_cell",
- "version_check",
+ "version_check 0.9.4",
 ]

 [[package]]
@@ -64,20 +64,6 @@ dependencies = [
 "term",
 ]

-[[package]]
-name = "assert_cmd"
-version = "2.0.4"
-source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "93ae1ddd39efd67689deb1979d80bad3bf7f2b09c6e6117c8d1f2443b5e2f83e"
-dependencies = [
- "bstr",
- "doc-comment",
- "predicates",
- "predicates-core",
- "predicates-tree",
- "wait-timeout",
-]
-
 [[package]]
 name = "async-channel"
 version = "1.7.1"
@@ -333,6 +319,12 @@ version = "0.3.1"
 source = "registry+https://github.com/rust-lang/crates.io-index"
 checksum = "e3b5ca7a04898ad4bcd41c90c5285445ff5b791899bb1b0abdd2a2aa791211d7"

+[[package]]
+name = "bytecount"
+version = "0.3.2"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "f861d9ce359f56dbcb6e0c2a1cb84e52ad732cadb57b806adeb3c7668caccbd8"
+
 [[package]]
 name = "byteorder"
 version = "1.4.3"
@@ -418,9 +410,9 @@ checksum = "fff857943da45f546682664a79488be82e69e43c1a7a2307679ab9afb3a66d2e"

 [[package]]
 name = "clap"
-version = "4.0.15"
+version = "4.0.9"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "6bf8832993da70a4c6d13c581f4463c2bdda27b9bf1c5498dc4365543abe6d6f"
+checksum = "30607dd93c420c6f1f80b544be522a0238a7db35e6a12968d28910983fee0df0"
 dependencies = [
 "atty",
 "bitflags",
@@ -433,9 +425,9 @@ dependencies = [

 [[package]]
 name = "clap_derive"
-version = "4.0.13"
+version = "4.0.9"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "c42f169caba89a7d512b5418b09864543eeb4d497416c917d7137863bd2076ad"
+checksum = "a4a307492e1a34939f79d3b6b9650bd2b971513cd775436bf2b78defeb5af00b"
 dependencies = [
 "heck",
 "proc-macro-error",
@@ -466,16 +458,6 @@ dependencies = [
 "winapi 0.3.9",
 ]

-[[package]]
-name = "codespan-reporting"
-version = "0.11.1"
-source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "3538270d33cc669650c4b093848450d380def10c331d38c768e34cac80576e6e"
-dependencies = [
- "termcolor",
- "unicode-width",
-]
-
 [[package]]
 name = "colored"
 version = "2.0.0"
@@ -598,62 +580,12 @@ dependencies = [
 "syn",
 ]

-[[package]]
-name = "cxx"
-version = "1.0.78"
-source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "19f39818dcfc97d45b03953c1292efc4e80954e1583c4aa770bac1383e2310a4"
-dependencies = [
- "cc",
- "cxxbridge-flags",
- "cxxbridge-macro",
- "link-cplusplus",
-]
-
-[[package]]
-name = "cxx-build"
-version = "1.0.78"
-source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "3e580d70777c116df50c390d1211993f62d40302881e54d4b79727acb83d0199"
-dependencies = [
- "cc",
- "codespan-reporting",
- "once_cell",
- "proc-macro2",
- "quote",
- "scratch",
- "syn",
-]
-
-[[package]]
-name = "cxxbridge-flags"
-version = "1.0.78"
-source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "56a46460b88d1cec95112c8c363f0e2c39afdb237f60583b0b36343bf627ea9c"
-
-[[package]]
-name = "cxxbridge-macro"
-version = "1.0.78"
-source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "747b608fecf06b0d72d440f27acc99288207324b793be2c17991839f3d4995ea"
-dependencies = [
- "proc-macro2",
- "quote",
- "syn",
-]
-
 [[package]]
 name = "diff"
 version = "0.1.13"
 source = "registry+https://github.com/rust-lang/crates.io-index"
 checksum = "56254986775e3233ffa9c4d7d3faaf6d36a2c09d30b20687e9f88bc8bafc16c8"

-[[package]]
-name = "difflib"
-version = "0.4.0"
-source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "6184e33543162437515c2e2b48714794e37845ec9851711914eec9d308f6ebe8"
-
 [[package]]
 name = "digest"
 version = "0.8.1"
@@ -732,12 +664,6 @@ dependencies = [
 "winapi 0.3.9",
 ]

-[[package]]
-name = "doc-comment"
-version = "0.3.3"
-source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "fea41bba32d969b513997752735605054bc0dfa92b4c56bf1189f2e174be7a10"
-
 [[package]]
 name = "either"
 version = "1.8.0"
@@ -987,7 +913,7 @@ source = "registry+https://github.com/rust-lang/crates.io-index"
 checksum = "bff49e947297f3312447abdca79f45f4738097cc82b06e72054d2223f601f1b9"
 dependencies = [
 "typenum",
- "version_check",
+ "version_check 0.9.4",
 ]

 [[package]]
@@ -1008,10 +934,8 @@ source = "registry+https://github.com/rust-lang/crates.io-index"
 checksum = "4eb1a864a501629691edf6c15a593b7a51eebaa1e8468e9ddc623de7c9b58ec6"
 dependencies = [
 "cfg-if 1.0.0",
- "js-sys",
 "libc",
 "wasi 0.11.0+wasi-snapshot-preview1",
- "wasm-bindgen",
 ]

 [[package]]
@@ -1037,9 +961,6 @@ name = "hashbrown"
 version = "0.12.3"
 source = "registry+https://github.com/rust-lang/crates.io-index"
 checksum = "8a9ee70c43aaf417c914396645a0fa852624801b24ebb7ae78fe8272889ac888"
-dependencies = [
- "ahash",
-]

 [[package]]
 name = "heck"
@@ -1076,28 +997,17 @@ checksum = "dfa686283ad6dd069f105e5ab091b04c62850d3e4cf5d67debad1933f55023df"

 [[package]]
 name = "iana-time-zone"
-version = "0.1.51"
+version = "0.1.50"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "f5a6ef98976b22b3b7f2f3a806f858cb862044cfa66805aa3ad84cb3d3b785ed"
+checksum = "fd911b35d940d2bd0bea0f9100068e5b97b51a1cbe13d13382f132e0365257a0"
 dependencies = [
 "android_system_properties",
 "core-foundation-sys",
- "iana-time-zone-haiku",
 "js-sys",
 "wasm-bindgen",
 "winapi 0.3.9",
 ]

-[[package]]
-name = "iana-time-zone-haiku"
-version = "0.1.0"
-source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "fde6edd6cef363e9359ed3c98ba64590ba9eecba2293eb5a723ab32aee8926aa"
-dependencies = [
- "cxx",
- "cxx-build",
-]
-
 [[package]]
 name = "idna"
 version = "0.3.0"
@@ -1181,15 +1091,9 @@ dependencies = [

 [[package]]
 name = "itoa"
-version = "1.0.4"
+version = "1.0.3"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "4217ad341ebadf8d8e724e264f13e593e0648f5b3e94b3896a5df283be015ecc"
-
-[[package]]
-name = "joinery"
-version = "2.1.0"
-source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "72167d68f5fce3b8655487b8038691a3c9984ee769590f93f2a631f4ad64e4f5"
+checksum = "6c8af84674fe1f223a982c933a0ee1086ac4d4052aa0fb8060c12c6ad838e754"

 [[package]]
 name = "js-sys"
@@ -1239,7 +1143,7 @@ dependencies = [
 "string_cache",
 "term",
 "tiny-keccak",
- "unicode-xid",
+ "unicode-xid 0.2.4",
 ]

 [[package]]
@@ -1295,9 +1199,9 @@ dependencies = [

 [[package]]
 name = "libc"
-version = "0.2.135"
+version = "0.2.134"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "68783febc7782c6c5cb401fbda4de5a9898be1762314da0bb2c10ced61f18b0c"
+checksum = "329c933548736bc49fd575ee68c89e8be4d260064184389a5b77517cddd99ffb"

 [[package]]
 name = "libcst"
@@ -1323,15 +1227,6 @@ dependencies = [
 "syn",
 ]

-[[package]]
-name = "link-cplusplus"
-version = "1.0.7"
-source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "9272ab7b96c9046fbc5bc56c06c117cb639fe2d509df0c421cad82d2915cf369"
-dependencies = [
- "cc",
-]
-
 [[package]]
 name = "linked-hash-map"
 version = "0.5.6"
@@ -1477,6 +1372,16 @@ dependencies = [
 "libc",
 ]

+[[package]]
+name = "nom"
+version = "4.2.3"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "2ad2a91a8e869eeb30b9cb3119ae87773a8f4ae617f41b1eb9c154b2905f7bd6"
+dependencies = [
+ "memchr",
+ "version_check 0.1.5",
+]
+
 [[package]]
 name = "nom"
 version = "5.1.2"
@@ -1484,7 +1389,18 @@ source = "registry+https://github.com/rust-lang/crates.io-index"
 checksum = "ffb4262d26ed83a1c0a33a38fe2bb15797329c85770da05e6b828ddb782627af"
 dependencies = [
 "memchr",
- "version_check",
+ "version_check 0.9.4",
+]
+
+[[package]]
+name = "nom_locate"
+version = "0.3.1"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "6a47c112b3861d81f7fbf73892b9271af933af32bd5dee6889aa3c3fa9caed7e"
+dependencies = [
+ "bytecount",
+ "memchr",
+ "nom 4.2.3",
 ]

 [[package]]
@@ -1505,6 +1421,17 @@ dependencies = [
 "winapi 0.3.9",
 ]

+[[package]]
+name = "num-bigint"
+version = "0.2.6"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "090c7f9998ee0ff65aa5b723e4009f7b217707f1fb5ea551329cc4d6231fb304"
+dependencies = [
+ "autocfg",
+ "num-integer",
+ "num-traits",
+]
+
 [[package]]
 name = "num-bigint"
 version = "0.4.3"
@@ -1617,18 +1544,18 @@ checksum = "b1de2e551fb905ac83f73f7aedf2f0cb4a0da7e35efa24a202a936269f1f18e1"

 [[package]]
 name = "path-absolutize"
-version = "3.0.14"
+version = "3.0.13"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "0f1d4993b16f7325d90c18c3c6a3327db7808752db8d208cea0acee0abd52c52"
+checksum = "d3de4b40bd9736640f14c438304c09538159802388febb02c8abaae0846c1f13"
 dependencies = [
 "path-dedot",
 ]

 [[package]]
 name = "path-dedot"
-version = "3.0.18"
+version = "3.0.17"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "9a81540d94551664b72b72829b12bd167c73c9d25fbac0e04fafa8023f7e4901"
+checksum = "d611d5291372b3738a34ebf0d1f849e58b1dcc1101032f76a346eaa1f8ddbb5b"
 dependencies = [
 "once_cell",
 ]
@@ -1796,33 +1723,6 @@ version = "0.1.1"
 source = "registry+https://github.com/rust-lang/crates.io-index"
 checksum = "925383efa346730478fb4838dbe9137d2a47675ad789c546d150a6e1dd4ab31c"

-[[package]]
-name = "predicates"
-version = "2.1.1"
-source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "a5aab5be6e4732b473071984b3164dbbfb7a3674d30ea5ff44410b6bcd960c3c"
-dependencies = [
- "difflib",
- "itertools",
- "predicates-core",
-]
-
-[[package]]
-name = "predicates-core"
-version = "1.0.3"
-source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "da1c2388b1513e1b605fcec39a95e0a9e8ef088f71443ef37099fa9ae6673fcb"
-
-[[package]]
-name = "predicates-tree"
-version = "1.0.5"
-source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "4d86de6de25020a36c6d3643a86d9a6a9f552107c0559c60ea03551b5e16c032"
-dependencies = [
- "predicates-core",
- "termtree",
-]
-
 [[package]]
 name = "proc-macro-error"
 version = "1.0.4"
@@ -1833,7 +1733,7 @@ dependencies = [
 "proc-macro2",
 "quote",
 "syn",
- "version_check",
+ "version_check 0.9.4",
 ]

 [[package]]
@@ -1844,7 +1744,7 @@ checksum = "a1be40180e52ecc98ad80b184934baf3d0d29f979574e439af5a55274b35f869"
 dependencies = [
 "proc-macro2",
 "quote",
- "version_check",
+ "version_check 0.9.4",
 ]

 [[package]]
@@ -1856,6 +1756,21 @@ dependencies = [
 "unicode-ident",
 ]

+[[package]]
+name = "python-parser"
+version = "0.2.0"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "683594d14f9c4db9e899643108918ad9db99aedad77121b5179aa205343c10de"
+dependencies = [
+ "nom 4.2.3",
+ "nom_locate",
+ "num-bigint 0.2.6",
+ "num-traits",
+ "unicode-xid 0.1.0",
+ "unicode_names2 0.2.2",
+ "wtf8",
+]
+
 [[package]]
 name = "quote"
 version = "1.0.21"
@@ -2045,10 +1960,9 @@ dependencies = [

 [[package]]
 name = "ruff"
-version = "0.0.82"
+version = "0.0.56"
 dependencies = [
 "anyhow",
- "assert_cmd",
 "bincode",
 "cacache",
 "chrono",
@@ -2059,16 +1973,15 @@ dependencies = [
 "dirs 4.0.0",
 "fern",
 "filetime",
- "getrandom 0.2.7",
 "glob",
 "insta",
 "itertools",
 "libcst",
 "log",
 "notify",
- "num-bigint",
 "once_cell",
 "path-absolutize",
+ "python-parser",
 "rayon",
 "regex",
 "rustpython-ast",
@@ -2076,11 +1989,6 @@ dependencies = [
 "rustpython-parser",
 "serde",
 "serde_json",
- "strum",
- "strum_macros",
- "test-case",
- "textwrap",
- "titlecase",
 "toml",
 "update-informer",
 "walkdir",
@@ -2101,9 +2009,9 @@ dependencies = [
 [[package]]
 name = "rustpython-ast"
 version = "0.1.0"
-source = "git+https://github.com/charliermarsh/RustPython.git?rev=1b253a12705f84972cd76e8dc1cdaaccb233e5a5#1b253a12705f84972cd76e8dc1cdaaccb233e5a5"
+source = "git+https://github.com/charliermarsh/RustPython.git?rev=4f457893efc381ad5c432576b24bcc7e4a08c641#4f457893efc381ad5c432576b24bcc7e4a08c641"
 dependencies = [
- "num-bigint",
+ "num-bigint 0.4.3",
 "rustpython-common",
 "rustpython-compiler-core",
 ]
@@ -2111,7 +2019,7 @@ dependencies = [
 [[package]]
 name = "rustpython-common"
 version = "0.0.0"
-source = "git+https://github.com/charliermarsh/RustPython.git?rev=1b253a12705f84972cd76e8dc1cdaaccb233e5a5#1b253a12705f84972cd76e8dc1cdaaccb233e5a5"
+source = "git+https://github.com/charliermarsh/RustPython.git?rev=4f457893efc381ad5c432576b24bcc7e4a08c641#4f457893efc381ad5c432576b24bcc7e4a08c641"
 dependencies = [
 "ascii",
 "cfg-if 1.0.0",
@@ -2119,7 +2027,7 @@ dependencies = [
 "lexical-parse-float",
 "libc",
 "lock_api",
- "num-bigint",
+ "num-bigint 0.4.3",
 "num-complex",
 "num-traits",
 "once_cell",
@@ -2134,14 +2042,14 @@ dependencies = [
 [[package]]
 name = "rustpython-compiler-core"
 version = "0.1.2"
-source = "git+https://github.com/charliermarsh/RustPython.git?rev=1b253a12705f84972cd76e8dc1cdaaccb233e5a5#1b253a12705f84972cd76e8dc1cdaaccb233e5a5"
+source = "git+https://github.com/charliermarsh/RustPython.git?rev=4f457893efc381ad5c432576b24bcc7e4a08c641#4f457893efc381ad5c432576b24bcc7e4a08c641"
 dependencies = [
 "bincode",
 "bitflags",
 "bstr",
 "itertools",
 "lz4_flex",
- "num-bigint",
+ "num-bigint 0.4.3",
 "num-complex",
 "serde",
 "static_assertions",
@@ -2151,7 +2059,7 @@ dependencies = [
 [[package]]
 name = "rustpython-parser"
 version = "0.1.2"
-source = "git+https://github.com/charliermarsh/RustPython.git?rev=1b253a12705f84972cd76e8dc1cdaaccb233e5a5#1b253a12705f84972cd76e8dc1cdaaccb233e5a5"
+source = "git+https://github.com/charliermarsh/RustPython.git?rev=4f457893efc381ad5c432576b24bcc7e4a08c641#4f457893efc381ad5c432576b24bcc7e4a08c641"
 dependencies = [
 "ahash",
 "anyhow",
@@ -2159,7 +2067,7 @@ dependencies = [
 "lalrpop",
 "lalrpop-util",
 "log",
- "num-bigint",
+ "num-bigint 0.4.3",
 "num-traits",
 "phf 0.10.1",
 "phf_codegen 0.10.0",
@@ -2169,7 +2077,7 @@ dependencies = [
 "tiny-keccak",
 "unic-emoji-char",
 "unic-ucd-ident",
- "unicode_names2",
+ "unicode_names2 0.5.1",
 ]

 [[package]]
@@ -2199,12 +2107,6 @@ version = "1.1.0"
 source = "registry+https://github.com/rust-lang/crates.io-index"
 checksum = "d29ab0c6d3fc0ee92fe66e2d99f700eab17a8d57d1c1d3b748380fb20baa78cd"

-[[package]]
-name = "scratch"
-version = "1.0.2"
-source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "9c8132065adcfd6e02db789d9285a0deb2f3fcb04002865ab67d5fb103533898"
-
 [[package]]
 name = "sct"
 version = "0.7.0"
@@ -2243,9 +2145,9 @@ dependencies = [

 [[package]]
 name = "serde_json"
-version = "1.0.86"
+version = "1.0.85"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "41feea4228a6f1cd09ec7a3593a682276702cd67b5273544757dae23c096f074"
+checksum = "e55a28e3aaef9d5ce0506d0a14dbba8054ddc7e499ef522dd8b26859ec9d4a44"
 dependencies = [
 "itoa",
 "ryu",
@@ -2348,12 +2250,6 @@ version = "1.10.0"
 source = "registry+https://github.com/rust-lang/crates.io-index"
 checksum = "a507befe795404456341dfab10cef66ead4c041f62b8b11bbb92bffe5d0953e0"

-[[package]]
-name = "smawk"
-version = "0.3.1"
-source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "f67ad224767faa3c7d8b6d91985b78e70a1324408abcb1cfcc2be4c06bc06043"
-
 [[package]]
 name = "socket2"
 version = "0.4.7"
@@ -2410,33 +2306,11 @@ version = "0.10.0"
 source = "registry+https://github.com/rust-lang/crates.io-index"
 checksum = "73473c0e59e6d5812c5dfe2a064a6444949f089e20eec9a2e5506596494e4623"

-[[package]]
-name = "strum"
-version = "0.24.1"
-source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "063e6045c0e62079840579a7e47a355ae92f60eb74daaf156fb1e84ba164e63f"
-dependencies = [
- "strum_macros",
-]
-
-[[package]]
-name = "strum_macros"
-version = "0.24.3"
-source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "1e385be0d24f186b4ce2f9982191e7101bb737312ad61c1f2f984f34bcf85d59"
-dependencies = [
- "heck",
- "proc-macro2",
- "quote",
- "rustversion",
- "syn",
-]
-
 [[package]]
 name = "syn"
-version = "1.0.102"
+version = "1.0.101"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "3fcd952facd492f9be3ef0d0b7032a6e442ee9b361d4acc2b1d0c4aaa5f613a1"
+checksum = "e90cde112c4b9690b8cbe810cba9ddd8bc1d7472e2cae317b69e9438c1cba7d2"
 dependencies = [
 "proc-macro2",
 "quote",
@@ -2495,50 +2369,11 @@ checksum = "76971977e6121664ec1b960d1313aacfa75642adc93b9d4d53b247bd4cb1747e"
 dependencies = [
 "dirs 2.0.2",
 "fnv",
- "nom",
+ "nom 5.1.2",
 "phf 0.8.0",
 "phf_codegen 0.8.0",
 ]

-[[package]]
-name = "termtree"
-version = "0.2.4"
-source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "507e9898683b6c43a9aa55b64259b721b52ba226e0f3779137e50ad114a4c90b"
-
-[[package]]
-name = "test-case"
-version = "2.2.2"
-source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "21d6cf5a7dffb3f9dceec8e6b8ca528d9bd71d36c9f074defb548ce161f598c0"
-dependencies = [
- "test-case-macros",
-]
-
-[[package]]
-name = "test-case-macros"
-version = "2.2.2"
-source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "e45b7bf6e19353ddd832745c8fcf77a17a93171df7151187f26623f2b75b5b26"
-dependencies = [
- "cfg-if 1.0.0",
- "proc-macro-error",
- "proc-macro2",
- "quote",
- "syn",
-]
-
-[[package]]
-name = "textwrap"
-version = "0.15.1"
-source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "949517c0cf1bf4ee812e2e07e08ab448e3ae0d23472aee8a06c985f0c8815b16"
-dependencies = [
- "smawk",
- "unicode-linebreak",
- "unicode-width",
-]
-
 [[package]]
 name = "thiserror"
 version = "1.0.37"
@@ -2594,17 +2429,6 @@ version = "0.1.0"
 source = "registry+https://github.com/rust-lang/crates.io-index"
 checksum = "cda74da7e1a664f795bb1f8a87ec406fb89a02522cf6e50620d016add6dbbf5c"

-[[package]]
-name = "titlecase"
-version = "2.2.1"
-source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "38397a8cdb017cfeb48bf6c154d6de975ac69ffeed35980fde199d2ee0842042"
-dependencies = [
- "joinery",
- "lazy_static",
- "regex",
-]
-
 [[package]]
 name = "toml"
 version = "0.5.9"
@@ -2702,19 +2526,9 @@ checksum = "099b7128301d285f79ddd55b9a83d5e6b9e97c92e0ea0daebee7263e932de992"

 [[package]]
 name = "unicode-ident"
-version = "1.0.5"
+version = "1.0.4"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "6ceab39d59e4c9499d4e5a8ee0e2735b891bb7308ac83dfb4e80cad195c9f6f3"
-
-[[package]]
-name = "unicode-linebreak"
-version = "0.1.4"
-source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "c5faade31a542b8b35855fff6e8def199853b2da8da256da52f52f1316ee3137"
-dependencies = [
- "hashbrown",
- "regex",
-]
+checksum = "dcc811dc4066ac62f84f11307873c4850cb653bfa9b1719cee2bd2204a4bc5dd"

 [[package]]
 name = "unicode-normalization"
@@ -2726,10 +2540,10 @@ dependencies = [
 ]

 [[package]]
-name = "unicode-width"
-version = "0.1.10"
+name = "unicode-xid"
+version = "0.1.0"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "c0edd1e5b14653f783770bce4a4dabb4a5108a5370a5f5d8cfe8710c361f6c8b"
+checksum = "fc72304796d0818e357ead4e000d19c9c174ab23dc11093ac919054d20a6a7fc"

 [[package]]
 name = "unicode-xid"
@@ -2737,6 +2551,12 @@ version = "0.2.4"
 source = "registry+https://github.com/rust-lang/crates.io-index"
 checksum = "f962df74c8c05a667b5ee8bcf162993134c104e96440b663c8daa176dc772d8c"

+[[package]]
+name = "unicode_names2"
+version = "0.2.2"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "0dc6c5da0c8d7200f9488cc346bd30ba62bcd9f79ef937ea6573132e3d507df9"
+
 [[package]]
 name = "unicode_names2"
 version = "0.5.1"
@@ -2799,9 +2619,15 @@ source = "registry+https://github.com/rust-lang/crates.io-index"
 checksum = "2209b78d1249f7e6f3293657c9779fe31ced465df091bbd433a1cf88e916ec55"
 dependencies = [
 "ctor",
- "version_check",
+ "version_check 0.9.4",
 ]

+[[package]]
+name = "version_check"
+version = "0.1.5"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "914b1a6776c4c929a602fafd8bc742e06365d4bcbe48c30f9cca5824f70dc9dd"
+
 [[package]]
 name = "version_check"
 version = "0.9.4"
@@ -2814,15 +2640,6 @@ version = "0.3.0"
 source = "registry+https://github.com/rust-lang/crates.io-index"
 checksum = "f8e76fae08f03f96e166d2dfda232190638c10e0383841252416f9cfe2ae60e6"

-[[package]]
-name = "wait-timeout"
-version = "0.2.0"
-source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "9f200f5b12eb75f8c1ed65abd4b2db8a6e1b138a20de009dacee265a2498f3f6"
-dependencies = [
- "libc",
-]
-
 [[package]]
 name = "waker-fn"
 version = "1.1.0"
@@ -3075,6 +2892,12 @@ dependencies = [
 "winapi-build",
 ]

+[[package]]
+name = "wtf8"
+version = "0.0.3"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "d6b9309a86639c488a8eb2b5331cb5127cc9feb0a94a0db4b5d1ab5b84977956"
+
 [[package]]
 name = "yaml-rust"
 version = "0.4.5"
--- a/Cargo.toml
+++ b/Cargo.toml
@@ -1,6 +1,6 @@
 [package]
 name = "ruff"
-version = "0.0.82"
+version = "0.0.56"
 edition = "2021"

 [lib]
@@ -9,8 +9,10 @@ name = "ruff"
 [dependencies]
 anyhow = { version = "1.0.60" }
 bincode = { version = "1.3.3" }
+cacache = { version = "10.0.1" }
 chrono = { version = "0.4.21" }
 clap = { version = "4.0.1", features = ["derive"] }
+clearscreen = { version = "1.0.10" }
 colored = { version = "2.0.0" }
 common-path = { version = "1.0.0" }
 dirs = { version = "4.0.0" }
@@ -21,37 +23,22 @@ itertools = { version = "0.10.5" }
 libcst = { git = "https://github.com/charliermarsh/LibCST", rev = "32a044c127668df44582f85699358e67803b0d73" }
 log = { version = "0.4.17" }
 notify = { version = "4.0.17" }
-num-bigint = { version = "0.4.3" }
 once_cell = { version = "1.13.1" }
-path-absolutize = { version = "3.0.14", features = ["once_cell_cache", "use_unix_paths_on_wasm"] }
+path-absolutize = { version = "3.0.13", features = ["once_cell_cache"] }
 rayon = { version = "1.5.3" }
 regex = { version = "1.6.0" }
-rustpython-ast = { features = ["unparse"], git = "https://github.com/charliermarsh/RustPython.git", rev = "1b253a12705f84972cd76e8dc1cdaaccb233e5a5" }
-rustpython-common = { git = "https://github.com/charliermarsh/RustPython.git", rev = "1b253a12705f84972cd76e8dc1cdaaccb233e5a5" }
-rustpython-parser = { features = ["lalrpop"], git = "https://github.com/charliermarsh/RustPython.git", rev = "1b253a12705f84972cd76e8dc1cdaaccb233e5a5" }
+rustpython-ast = { features = ["unparse"], git = "https://github.com/charliermarsh/RustPython.git", rev = "4f457893efc381ad5c432576b24bcc7e4a08c641" }
+rustpython-parser = { features = ["lalrpop"], git = "https://github.com/charliermarsh/RustPython.git", rev = "4f457893efc381ad5c432576b24bcc7e4a08c641" }
+rustpython-common = { git = "https://github.com/charliermarsh/RustPython.git", rev = "4f457893efc381ad5c432576b24bcc7e4a08c641" }
 serde = { version = "1.0.143", features = ["derive"] }
 serde_json = { version = "1.0.83" }
-strum = { version = "0.24.1", features = ["strum_macros"] }
-strum_macros = { version = "0.24.3" }
-textwrap = { version = "0.15.1" }
-titlecase = { version = "2.2.1" }
 toml = { version = "0.5.9" }
 update-informer = { version = "0.5.0", default_features = false, features = ["pypi"], optional = true }
 walkdir = { version = "2.3.2" }
-
-[target.'cfg(not(target_family = "wasm"))'.dependencies]
-cacache = { version = "10.0.1" } # uses async-std
-clearscreen = { version = "1.0.10" } # uses which
-
-# https://docs.rs/getrandom/0.2.7/getrandom/#webassembly-support
-# For (future) wasm-pack support
-[target.'cfg(all(target_family = "wasm", target_os = "unknown"))'.dependencies]
-getrandom = { version = "0.2.7", features = ["js"] }
+python-parser = "0.2.0"

 [dev-dependencies]
-assert_cmd = { version = "2.0.4" }
 insta = { version = "1.19.1", features = ["yaml"] }
-test-case = { version = "2.2.2" }

 [features]
 default = ["update-informer"]
--- a/README.md
+++ b/README.md
@@ -1,4 +1,4 @@
-# Ruff
+# ruff

 [![image](https://img.shields.io/pypi/v/ruff.svg)](https://pypi.python.org/pypi/ruff)
 [![image](https://img.shields.io/pypi/l/ruff.svg)](https://pypi.python.org/pypi/ruff)
@@ -20,37 +20,17 @@ An extremely fast Python linter, written in Rust.
 - 🤝 Python 3.10 compatibility
 - 🛠️ `pyproject.toml` support
 - 📦 [ESLint](https://eslint.org/docs/latest/user-guide/command-line-interface#caching)-inspired cache support
- 🔧 [ESLint](https://eslint.org/docs/latest/user-guide/command-line-interface#--fix)-inspired autofix support (e.g., automatically remove unused imports)
- 👀 [TypeScript](https://www.typescriptlang.org/docs/handbook/configuring-watch.html)-inspired `--watch` support, for continuous file monitoring
- ⚖️ [Near-parity](#how-does-ruff-compare-to-flake8) with the built-in Flake8 rule set
- 🔌 Native re-implementations of popular Flake8 plugins, like [`flake8-docstrings`](https://pypi.org/project/flake8-docstrings/) ([`pydocstyle`](https://pypi.org/project/pydocstyle/))
-
-Ruff aims to be orders of magnitude faster than alternative tools while integrating more
-functionality behind a single, common interface. Ruff can be used to replace Flake8 (plus a variety
-of plugins), [`pydocstyle`](https://pypi.org/project/pydocstyle/), [`yesqa`](https://github.com/asottile/yesqa),
-and even a subset of [`pyupgrade`](https://pypi.org/project/pyupgrade/) and [`autoflake`](https://pypi.org/project/autoflake/)
-all while executing tens or hundreds of times faster than any individual tool.
+- 🔧 [ESLint](https://eslint.org/docs/latest/user-guide/command-line-interface#--fix)-inspired `--fix` support
+- 👀 [TypeScript](https://www.typescriptlang.org/docs/handbook/configuring-watch.html)-inspired `--watch` support
+- ⚖️ [Near-complete parity](#Parity-with-Flake8) with the built-in Flake8 rule set

 Read the [launch blog post](https://notes.crmarsh.com/python-tooling-could-be-much-much-faster).

-## Table of Contents
-
-1. [Installation and Usage](#installation-and-usage)
-2. [Configuration](#configuration)
-3. [Supported Rules](#supported-rules)
-4. [Editor Integrations](#editor-integrations)
-5. [FAQ](#faq)
-6. [Development](#development)
-7. [Releases](#releases)
-8. [Benchmarks](#benchmarks)
-9. [License](#license)
-10. [Contributing](#contributing)
-
-## Installation and Usage
+## Installation and usage

 ### Installation

-Available as [`ruff`](https://pypi.org/project/ruff/) on PyPI:
+Available as [ruff](https://pypi.org/project/ruff/) on PyPI:

 ```shell
 pip install ruff
@@ -58,7 +38,7 @@ pip install ruff

 ### Usage

-To run Ruff, try any of the following:
+To run ruff, try any of the following:

 ```shell
 ruff path/to/code/to/check.py
@@ -66,27 +46,27 @@ ruff path/to/code/
 ruff path/to/code/*.py
 ```

-You can run Ruff in `--watch` mode to automatically re-run on-change:
+You can run ruff in `--watch` mode to automatically re-run on-change:

 ```shell
 ruff path/to/code/ --watch
 ```

-Ruff also works with [pre-commit](https://pre-commit.com):
+ruff also works with [pre-commit](https://pre-commit.com):

 ```yaml
 repos:
  - repo: https://github.com/charliermarsh/ruff-pre-commit
-    rev: v0.0.82
+    rev: v0.0.48
    hooks:
      - id: lint
 ```

 ## Configuration

-Ruff is configurable both via `pyproject.toml` and the command line.
+ruff is configurable both via `pyproject.toml` and the command line.

-For example, you could configure Ruff to only enforce a subset of rules with:
+For example, you could configure ruff to only enforce a subset of rules with:

 ```toml
 [tool.ruff]
@@ -114,8 +94,6 @@ Arguments:
  <FILES>...

 Options:
-      --config <CONFIG>
-          Path to the `pyproject.toml` file to use for configuration
  -v, --verbose
          Enable verbose logging
  -q, --quiet
@@ -152,10 +130,6 @@ Options:
          Enable automatic additions of noqa directives to failing lines
      --dummy-variable-rgx <DUMMY_VARIABLE_RGX>
          Regular expression matching the name of dummy variables
-      --target-version <TARGET_VERSION>
-          The minimum Python version that should be supported
-      --stdin-filename <STDIN_FILENAME>
-          The name of the file when passing it through stdin
  -h, --help
          Print help information
  -V, --version
@@ -178,7 +152,7 @@ Exclusions are based on globs, and can be either:
 To omit a lint check entirely, add it to the "ignore" list via `--ignore` or `--extend-ignore`,
 either  on the command-line or in your `project.toml` file.

-To ignore an error in-line, Ruff uses a `noqa` system similar to [Flake8](https://flake8.pycqa.org/en/3.1.1/user/ignoring-errors.html).
+To ignore an error in-line, ruff uses a `noqa` system similar to [Flake8](https://flake8.pycqa.org/en/3.1.1/user/ignoring-errors.html).
 To ignore an individual error, add `# noqa: {code}` to the end of the line, like so:

 ```python
@@ -202,226 +176,129 @@ Lorem ipsum dolor sit amet, consectetur adipiscing elit, sed do eiusmod tempor i
 """  # noqa: E501
 ```

-Ruff supports several workflows to aid in `noqa` management.
+ruff supports several (experimental) workflows to aid in `noqa` management.

-First, Ruff provides a special error code, `M001`, to enforce that your `noqa` directives are
+First, ruff provides a special error code, `M001`, to enforce that your `noqa` directives are
 "valid", in that the errors they _say_ they ignore are actually being triggered on that line (and
 thus suppressed). **You can run `ruff /path/to/file.py --extend-select M001` to flag unused `noqa`
 directives.**

-Second, Ruff can _automatically remove_ unused `noqa` directives via its autofix functionality.
+Second, ruff can _automatically remove_ unused `noqa` directives via its autofix functionality.
 **You can run `ruff /path/to/file.py --extend-select M001 --fix` to automatically remove unused
 `noqa` directives.**

-Third, Ruff can _automatically add_ `noqa` directives to all failing lines. This is useful when
-migrating a new codebase to Ruff. **You can run `ruff /path/to/file.py --add-noqa` to automatically
+Third, ruff can _automatically add_ `noqa` directives to all failing lines. This is useful when
+migrating a new codebase to ruff. **You can run `ruff /path/to/file.py --add-noqa` to automatically
 add `noqa` directives to all failing lines, with the appropriate error codes.**

-## Supported Rules
+### Compatibility with Black

-By default, Ruff enables all `E`, `W`, and `F` error codes, which correspond to those built-in to
-Flake8.
+ruff is compatible with [Black](https://github.com/psf/black) out-of-the-box, as long as
+the `line-length` setting is consistent between the two.
+
+As a project, ruff is designed to be used alongside Black and, as such, will defer implementing
+stylistic lint rules that are obviated by autoformatting.
+
+### Parity with Flake8
+
+ruff's goal is to achieve feature parity with Flake8 when used (1) without plugins, (2) alongside
+Black, and (3) on Python 3 code.
+
+**Under those conditions, ruff implements 44 out of 60 rules.** (ruff is missing: 14 rules related
+to string `.format` calls, 1 rule related to docstring parsing, and 1 rule related to redefined
+variables.)
+
+ruff also implements some of the most popular Flake8 plugins natively, including:
+
+- [`flake8-builtins`](https://pypi.org/project/flake8-builtins/)
+- [`flake8-super`](https://pypi.org/project/flake8-super/)
+- [`flake8-print`](https://pypi.org/project/flake8-print/)
+- [`pyupgrade`](https://pypi.org/project/pyupgrade/) (partial)
+
+Beyond rule-set parity, ruff suffers from the following limitations vis-à-vis Flake8:
+
+1. ruff does not yet support a few Python 3.9 and 3.10 language features, including structural
+   pattern matching and parenthesized context managers.
+2. Flake8 has a plugin architecture and supports writing custom lint rules.
+
+## Rules
+
+The ✅ emoji indicates a rule is enabled by default.

 The 🛠 emoji indicates that a rule is automatically fixable by the `--fix` command-line option.

-### Pyflakes
+| Code | Name | Message |     |     |
+| ---- | ---- | ------- | --- | --- |
+| E402 | ModuleImportNotAtTopOfFile | Module level import not at top of file | ✅ |  |
+| E501 | LineTooLong | Line too long (89 > 88 characters) | ✅ |  |
+| E711 | NoneComparison | Comparison to `None` should be `cond is None` | ✅ |  |
+| E712 | TrueFalseComparison | Comparison to `True` should be `cond is True` | ✅ |  |
+| E713 | NotInTest | Test for membership should be `not in` | ✅ |  |
+| E714 | NotIsTest | Test for object identity should be `is not` | ✅ |  |
+| E721 | TypeComparison | Do not compare types, use `isinstance()` | ✅ |  |
+| E722 | DoNotUseBareExcept | Do not use bare `except` | ✅ |  |
+| E731 | DoNotAssignLambda | Do not assign a lambda expression, use a def | ✅ |  |
+| E741 | AmbiguousVariableName | Ambiguous variable name: `...` | ✅ |  |
+| E742 | AmbiguousClassName | Ambiguous class name: `...` | ✅ |  |
+| E743 | AmbiguousFunctionName | Ambiguous function name: `...` | ✅ |  |
+| E902 | IOError | IOError: `...` | ✅ |  |
+| E999 | SyntaxError | SyntaxError: `...` | ✅ |  |
+| F401 | UnusedImport | `...` imported but unused | ✅ | 🛠 |
+| F402 | ImportShadowedByLoopVar | Import `...` from line 1 shadowed by loop variable | ✅ |  |
+| F403 | ImportStarUsed | `from ... import *` used; unable to detect undefined names | ✅ |  |
+| F404 | LateFutureImport | `from __future__` imports must occur at the beginning of the file | ✅ |  |
+| F405 | ImportStarUsage | `...` may be undefined, or defined from star imports: `...` | ✅ |  |
+| F406 | ImportStarNotPermitted | `from ... import *` only allowed at module level | ✅ |  |
+| F407 | FutureFeatureNotDefined | Future feature `...` is not defined | ✅ |  |
+| F541 | FStringMissingPlaceholders | f-string without any placeholders | ✅ |  |
+| F601 | MultiValueRepeatedKeyLiteral | Dictionary key literal repeated | ✅ |  |
+| F602 | MultiValueRepeatedKeyVariable | Dictionary key `...` repeated | ✅ |  |
+| F621 | ExpressionsInStarAssignment | Too many expressions in star-unpacking assignment | ✅ |  |
+| F622 | TwoStarredExpressions | Two starred expressions in assignment | ✅ |  |
+| F631 | AssertTuple | Assert test is a non-empty tuple, which is always `True` | ✅ |  |
+| F632 | IsLiteral | Use `==` and `!=` to compare constant literals | ✅ |  |
+| F633 | InvalidPrintSyntax | Use of `>>` is invalid with `print` function | ✅ |  |
+| F634 | IfTuple | If test is a tuple, which is always `True` | ✅ |  |
+| F701 | BreakOutsideLoop | `break` outside loop | ✅ |  |
+| F702 | ContinueOutsideLoop | `continue` not properly in loop | ✅ |  |
+| F704 | YieldOutsideFunction | `yield` or `yield from` statement outside of a function/method | ✅ |  |
+| F706 | ReturnOutsideFunction | `return` statement outside of a function/method | ✅ |  |
+| F707 | DefaultExceptNotLast | An `except:` block as not the last exception handler | ✅ |  |
+| F722 | ForwardAnnotationSyntaxError | Syntax error in forward annotation: `...` | ✅ |  |
+| F821 | UndefinedName | Undefined name `...` | ✅ |  |
+| F822 | UndefinedExport | Undefined name `...` in `__all__` | ✅ |  |
+| F823 | UndefinedLocal | Local variable `...` referenced before assignment | ✅ |  |
+| F831 | DuplicateArgumentName | Duplicate argument name in function definition | ✅ |  |
+| F841 | UnusedVariable | Local variable `...` is assigned to but never used | ✅ |  |
+| F901 | RaiseNotImplemented | `raise NotImplemented` should be `raise NotImplementedError` | ✅ |  |
+| A001 | BuiltinVariableShadowing | Variable `...` is shadowing a python builtin |  |  |
+| A002 | BuiltinArgumentShadowing | Argument `...` is shadowing a python builtin |  |  |
+| A003 | BuiltinAttributeShadowing | Class attribute `...` is shadowing a python builtin |  |  |
+| SPR001 | SuperCallWithParameters | Use `super()` instead of `super(__class__, self)` |  | 🛠 |
+| T201 | PrintFound | `print` found |  | 🛠 |
+| T203 | PPrintFound | `pprint` found |  | 🛠 |
+| U001 | UselessMetaclassType | `__metaclass__ = type` is implied |  | 🛠 |
+| R001 | UselessObjectInheritance | Class `...` inherits from object |  | 🛠 |
+| R002 | NoAssertEquals | `assertEquals` is deprecated, use `assertEqual` instead |  | 🛠 |
+| M001 | UnusedNOQA | Unused `noqa` directive |  | 🛠 |

-| Code | Name | Message | Fix |
-| ---- | ---- | ------- | --- |
-| F401 | UnusedImport | `...` imported but unused | 🛠 |
-| F402 | ImportShadowedByLoopVar | Import `...` from line 1 shadowed by loop variable |  |
-| F403 | ImportStarUsed | `from ... import *` used; unable to detect undefined names |  |
-| F404 | LateFutureImport | `from __future__` imports must occur at the beginning of the file |  |
-| F405 | ImportStarUsage | `...` may be undefined, or defined from star imports: `...` |  |
-| F406 | ImportStarNotPermitted | `from ... import *` only allowed at module level |  |
-| F407 | FutureFeatureNotDefined | Future feature `...` is not defined |  |
-| F541 | FStringMissingPlaceholders | f-string without any placeholders |  |
-| F601 | MultiValueRepeatedKeyLiteral | Dictionary key literal repeated |  |
-| F602 | MultiValueRepeatedKeyVariable | Dictionary key `...` repeated |  |
-| F621 | ExpressionsInStarAssignment | Too many expressions in star-unpacking assignment |  |
-| F622 | TwoStarredExpressions | Two starred expressions in assignment |  |
-| F631 | AssertTuple | Assert test is a non-empty tuple, which is always `True` |  |
-| F632 | IsLiteral | Use `==` and `!=` to compare constant literals |  |
-| F633 | InvalidPrintSyntax | Use of `>>` is invalid with `print` function |  |
-| F634 | IfTuple | If test is a tuple, which is always `True` |  |
-| F701 | BreakOutsideLoop | `break` outside loop |  |
-| F702 | ContinueOutsideLoop | `continue` not properly in loop |  |
-| F704 | YieldOutsideFunction | `yield` or `yield from` statement outside of a function/method |  |
-| F706 | ReturnOutsideFunction | `return` statement outside of a function/method |  |
-| F707 | DefaultExceptNotLast | An `except:` block as not the last exception handler |  |
-| F722 | ForwardAnnotationSyntaxError | Syntax error in forward annotation: `...` |  |
-| F821 | UndefinedName | Undefined name `...` |  |
-| F822 | UndefinedExport | Undefined name `...` in `__all__` |  |
-| F823 | UndefinedLocal | Local variable `...` referenced before assignment |  |
-| F831 | DuplicateArgumentName | Duplicate argument name in function definition |  |
-| F841 | UnusedVariable | Local variable `...` is assigned to but never used |  |
-| F901 | RaiseNotImplemented | `raise NotImplemented` should be `raise NotImplementedError` |  |
-
-### pycodestyle
-
-| Code | Name | Message | Fix |
-| ---- | ---- | ------- | --- |
-| E402 | ModuleImportNotAtTopOfFile | Module level import not at top of file |  |
-| E501 | LineTooLong | Line too long (89 > 88 characters) |  |
-| E711 | NoneComparison | Comparison to `None` should be `cond is None` |  |
-| E712 | TrueFalseComparison | Comparison to `True` should be `cond is True` |  |
-| E713 | NotInTest | Test for membership should be `not in` |  |
-| E714 | NotIsTest | Test for object identity should be `is not` |  |
-| E721 | TypeComparison | Do not compare types, use `isinstance()` |  |
-| E722 | DoNotUseBareExcept | Do not use bare `except` |  |
-| E731 | DoNotAssignLambda | Do not assign a lambda expression, use a def |  |
-| E741 | AmbiguousVariableName | Ambiguous variable name: `...` |  |
-| E742 | AmbiguousClassName | Ambiguous class name: `...` |  |
-| E743 | AmbiguousFunctionName | Ambiguous function name: `...` |  |
-| E902 | IOError | IOError: `...` |  |
-| E999 | SyntaxError | SyntaxError: `...` |  |
-| W292 | NoNewLineAtEndOfFile | No newline at end of file |  |
-
-### pydocstyle
-
-| Code | Name | Message | Fix |
-| ---- | ---- | ------- | --- |
-| D100 | PublicModule | Missing docstring in public module |  |
-| D101 | PublicClass | Missing docstring in public class |  |
-| D102 | PublicMethod | Missing docstring in public method |  |
-| D103 | PublicFunction | Missing docstring in public function |  |
-| D104 | PublicPackage | Missing docstring in public package |  |
-| D105 | MagicMethod | Missing docstring in magic method |  |
-| D106 | PublicNestedClass | Missing docstring in public nested class |  |
-| D107 | PublicInit | Missing docstring in `__init__` |  |
-| D200 | FitsOnOneLine | One-line docstring should fit on one line |  |
-| D201 | NoBlankLineBeforeFunction | No blank lines allowed before function docstring (found 1) | 🛠 |
-| D202 | NoBlankLineAfterFunction | No blank lines allowed after function docstring (found 1) | 🛠 |
-| D203 | OneBlankLineBeforeClass | 1 blank line required before class docstring | 🛠 |
-| D204 | OneBlankLineAfterClass | 1 blank line required after class docstring | 🛠 |
-| D205 | BlankLineAfterSummary | 1 blank line required between summary line and description | 🛠 |
-| D206 | IndentWithSpaces | Docstring should be indented with spaces, not tabs |  |
-| D207 | NoUnderIndentation | Docstring is under-indented | 🛠 |
-| D208 | NoOverIndentation | Docstring is over-indented | 🛠 |
-| D209 | NewLineAfterLastParagraph | Multi-line docstring closing quotes should be on a separate line | 🛠 |
-| D210 | NoSurroundingWhitespace | No whitespaces allowed surrounding docstring text | 🛠 |
-| D211 | NoBlankLineBeforeClass | No blank lines allowed before class docstring | 🛠 |
-| D212 | MultiLineSummaryFirstLine | Multi-line docstring summary should start at the first line |  |
-| D213 | MultiLineSummarySecondLine | Multi-line docstring summary should start at the second line |  |
-| D214 | SectionNotOverIndented | Section is over-indented ("Returns") | 🛠 |
-| D215 | SectionUnderlineNotOverIndented | Section underline is over-indented ("Returns") | 🛠 |
-| D300 | UsesTripleQuotes | Use """triple double quotes""" |  |
-| D400 | EndsInPeriod | First line should end with a period |  |
-| D402 | NoSignature | First line should not be the function's signature |  |
-| D403 | FirstLineCapitalized | First word of the first line should be properly capitalized |  |
-| D404 | NoThisPrefix | First word of the docstring should not be `This` |  |
-| D405 | CapitalizeSectionName | Section name should be properly capitalized ("returns") | 🛠 |
-| D406 | NewLineAfterSectionName | Section name should end with a newline ("Returns") | 🛠 |
-| D407 | DashedUnderlineAfterSection | Missing dashed underline after section ("Returns") | 🛠 |
-| D408 | SectionUnderlineAfterName | Section underline should be in the line following the section's name ("Returns") | 🛠 |
-| D409 | SectionUnderlineMatchesSectionLength | Section underline should match the length of its name ("Returns") | 🛠 |
-| D410 | BlankLineAfterSection | Missing blank line after section ("Returns") | 🛠 |
-| D411 | BlankLineBeforeSection | Missing blank line before section ("Returns") | 🛠 |
-| D412 | NoBlankLinesBetweenHeaderAndContent | No blank lines allowed between a section header and its content ("Returns") | 🛠 |
-| D413 | BlankLineAfterLastSection | Missing blank line after last section ("Returns") | 🛠 |
-| D414 | NonEmptySection | Section has no content ("Returns") |  |
-| D415 | EndsInPunctuation | First line should end with a period, question mark, or exclamation point |  |
-| D416 | SectionNameEndsInColon | Section name should end with a colon ("Returns") | 🛠 |
-| D417 | DocumentAllArguments | Missing argument descriptions in the docstring: `x`, `y` |  |
-| D418 | SkipDocstring | Function decorated with @overload shouldn't contain a docstring |  |
-| D419 | NonEmpty | Docstring is empty |  |
-
-### pyupgrade
-
-| Code | Name | Message | Fix |
-| ---- | ---- | ------- | --- |
-| U001 | UselessMetaclassType | `__metaclass__ = type` is implied | 🛠 |
-| U002 | UnnecessaryAbspath | `abspath(__file__)` is unnecessary in Python 3.9 and later | 🛠 |
-| U003 | TypeOfPrimitive | Use `str` instead of `type(...)` | 🛠 |
-| U004 | UselessObjectInheritance | Class `...` inherits from object | 🛠 |
-| U005 | DeprecatedUnittestAlias | `assertEquals` is deprecated, use `assertEqual` instead | 🛠 |
-| U006 | UsePEP585Annotation | Use `list` instead of `List` for type annotations | 🛠 |
-| U007 | UsePEP604Annotation | Use `X \| Y` for type annotations | 🛠 |
-| U008 | SuperCallWithParameters | Use `super()` instead of `super(__class__, self)` | 🛠 |
-
-### pep8-naming
-
-| Code | Name | Message | Fix |
-| ---- | ---- | ------- | --- |
-| N801 | InvalidClassName | Class name `...` should use CapWords convention  |  |
-| N802 | InvalidFunctionName | Function name `...` should be lowercase |  |
-| N803 | InvalidArgumentName | Argument name `...` should be lowercase |  |
-| N804 | InvalidFirstArgumentNameForClassMethod | First argument of a class method should be named `cls` |  |
-| N805 | InvalidFirstArgumentNameForMethod | First argument of a method should be named `self` |  |
-| N807 | DunderFunctionName | function name should not start and end with '__' |  |
-| N811 | ConstantImportedAsNonConstant | constant '...' imported as non constant '...' |  |
-| N812 | LowercaseImportedAsNonLowercase | lowercase '...' imported as non lowercase '...' |  |
-| N813 | CamelcaseImportedAsLowercase | camelcase '...' imported as lowercase '...' |  |
-| N814 | CamelcaseImportedAsConstant | camelcase '...' imported as constant '...' |  |
-| N817 | CamelcaseImportedAsAcronym | camelcase '...' imported as acronym '...' |  |
-
-### flake8-comprehensions
-
-| Code | Name | Message | Fix |
-| ---- | ---- | ------- | --- |
-| C400 | UnnecessaryGeneratorList | Unnecessary generator (rewrite as a `list` comprehension) |  |
-| C401 | UnnecessaryGeneratorSet | Unnecessary generator (rewrite as a `set` comprehension) |  |
-| C402 | UnnecessaryGeneratorDict | Unnecessary generator (rewrite as a `dict` comprehension) |  |
-| C403 | UnnecessaryListComprehensionSet | Unnecessary `list` comprehension (rewrite as a `set` comprehension) |  |
-| C404 | UnnecessaryListComprehensionDict | Unnecessary `list` comprehension (rewrite as a `dict` comprehension) |  |
-| C405 | UnnecessaryLiteralSet | Unnecessary `(list\|tuple)` literal (rewrite as a `set` literal) |  |
-| C406 | UnnecessaryLiteralDict | Unnecessary `(list\|tuple)` literal (rewrite as a `dict` literal) |  |
-| C408 | UnnecessaryCollectionCall | Unnecessary `(dict\|list\|tuple)` call (rewrite as a literal) |  |
-| C409 | UnnecessaryLiteralWithinTupleCall | Unnecessary `(list\|tuple)` literal passed to `tuple()` (remove the outer call to `tuple()`) |  |
-| C410 | UnnecessaryLiteralWithinListCall | Unnecessary `(list\|tuple)` literal passed to `list()` (rewrite as a `list` literal) |  |
-| C411 | UnnecessaryListCall | Unnecessary `list` call (remove the outer call to `list()`) |  |
-| C413 | UnnecessaryCallAroundSorted | Unnecessary `(list\|reversed)` call around `sorted()` |  |
-| C414 | UnnecessaryDoubleCastOrProcess | Unnecessary `(list\|reversed\|set\|sorted\|tuple)` call within `(list\|set\|sorted\|tuple)()` |  |
-| C415 | UnnecessarySubscriptReversal | Unnecessary subscript reversal of iterable within `(reversed\|set\|sorted)()` |  |
-| C416 | UnnecessaryComprehension |  Unnecessary `(list\|set)` comprehension (rewrite using `(list\|set)()`) |  |
-| C417 | UnnecessaryMap | Unnecessary `map` usage (rewrite using a `(list\|set\|dict)` comprehension) |  |
-
-### flake8-bugbear
-
-| Code | Name | Message | Fix |
-| ---- | ---- | ------- | --- |
-| B011 | DoNotAssertFalse | Do not `assert False` (`python -O` removes these calls), raise `AssertionError()` | 🛠 |
-| B014 | DuplicateHandlerException | Exception handler with duplicate exception: `ValueError` | 🛠 |
-| B025 | DuplicateTryBlockException | try-except block with duplicate exception `Exception` |  |
-
-### flake8-builtins
-
-| Code | Name | Message | Fix |
-| ---- | ---- | ------- | --- |
-| A001 | BuiltinVariableShadowing | Variable `...` is shadowing a python builtin |  |
-| A002 | BuiltinArgumentShadowing | Argument `...` is shadowing a python builtin |  |
-| A003 | BuiltinAttributeShadowing | Class attribute `...` is shadowing a python builtin |  |
-
-### flake8-print
-
-| Code | Name | Message | Fix |
-| ---- | ---- | ------- | --- |
-| T201 | PrintFound | `print` found | 🛠 |
-| T203 | PPrintFound | `pprint` found | 🛠 |
-
-### Meta rules
-
-| Code | Name | Message | Fix |
-| ---- | ---- | ------- | --- |
-| M001 | UnusedNOQA | Unused `noqa` directive | 🛠 |
-
-## Editor Integrations
+## Integrations

 ### PyCharm

-Ruff can be installed as an [External Tool](https://www.jetbrains.com/help/pycharm/configuring-third-party-tools.html)
+ruff can be installed as an [External Tool](https://www.jetbrains.com/help/pycharm/configuring-third-party-tools.html)
 in PyCharm. Open the Preferences pane, then navigate to "Tools", then "External Tools". From there,
 add a new tool with the following configuration:

-![Install Ruff as an External Tool](https://user-images.githubusercontent.com/1309177/193155720-336e43f0-1a8d-46b4-bc12-e60f9ae01f7e.png)
+![Install ruff as an External Tool](https://user-images.githubusercontent.com/1309177/193155720-336e43f0-1a8d-46b4-bc12-e60f9ae01f7e.png)

-Ruff should then appear as a runnable action:
+ruff should then appear as a runnable action:

-![Ruff as a runnable action](https://user-images.githubusercontent.com/1309177/193156026-732b0aaf-3dd9-4549-9b4d-2de6d2168a33.png)
+![ruff as a runnable action](https://user-images.githubusercontent.com/1309177/193156026-732b0aaf-3dd9-4549-9b4d-2de6d2168a33.png)

 ### GitHub Actions

-GitHub Actions has everything you need to run Ruff out-of-the-box:
+GitHub Actions has everything you need to run ruff out-of-the-box:

 ```yaml
 name: CI
@@ -430,184 +307,22 @@ jobs:
  build:
    runs-on: ubuntu-latest
    steps:
-      - uses: actions/checkout@v3
+      - uses: actions/checkout@v2
      - name: Install Python
-        uses: actions/setup-python@v4
+        uses: actions/setup-python@v2
        with:
          python-version: "3.10"
      - name: Install dependencies
        run: |
          python -m pip install --upgrade pip
          pip install ruff
-      - name: Run Ruff
+      - name: Run ruff
        run: ruff .
 ```

-## FAQ
-
-### Is Ruff compatible with Black?
-
-Yes. Ruff is compatible with [Black](https://github.com/psf/black) out-of-the-box, as long as
-the `line-length` setting is consistent between the two.
-
-As a project, Ruff is designed to be used alongside Black and, as such, will defer implementing
-stylistic lint rules that are obviated by autoformatting.
-
-### How does Ruff compare to Flake8?
-
-Ruff can be used as a (near) drop-in replacement for Flake8 when used (1) without or with a small
-number of plugins, (2) alongside Black, and (3) on Python 3 code.
-
-Under those conditions Ruff is missing 14 rules related to string `.format` calls, 1 rule related
-to docstring parsing, and 1 rule related to redefined variables.
-
-Ruff re-implements some of the most popular Flake8 plugins and related code quality tools natively,
-including:
-
- [`pydocstyle`](https://pypi.org/project/pydocstyle/)
- [`yesqa`](https://github.com/asottile/yesqa)
- [`flake8-docstrings`](https://pypi.org/project/flake8-docstrings/)
- [`flake8-builtins`](https://pypi.org/project/flake8-builtins/)
- [`flake8-super`](https://pypi.org/project/flake8-super/)
- [`flake8-print`](https://pypi.org/project/flake8-print/)
- [`flake8-comprehensions`](https://pypi.org/project/flake8-comprehensions/)
- [`flake8-bugbear`](https://pypi.org/project/flake8-bugbear/) (3/32)
- [`pyupgrade`](https://pypi.org/project/pyupgrade/) (8/34)
- [`autoflake`](https://pypi.org/project/autoflake/) (1/7)
-
-Beyond rule-set parity, Ruff suffers from the following limitations vis-à-vis Flake8:
-
-1. Ruff does not yet support a few Python 3.9 and 3.10 language features, including structural
-   pattern matching and parenthesized context managers.
-2. Flake8 has a plugin architecture and supports writing custom lint rules. (To date, popular Flake8
-   plugins have been re-implemented within Ruff directly.)
-
-### Which tools does Ruff replace?
-
-Today, Ruff can be used to replace Flake8 when used with any of the following plugins:
-
- [`flake8-docstrings`](https://pypi.org/project/flake8-docstrings/)
- [`flake8-builtins`](https://pypi.org/project/flake8-builtins/)
- [`flake8-super`](https://pypi.org/project/flake8-super/)
- [`flake8-print`](https://pypi.org/project/flake8-print/)
- [`flake8-comprehensions`](https://pypi.org/project/flake8-comprehensions/)
- [`flake8-bugbear`](https://pypi.org/project/flake8-bugbear/) (3/32)
-
-Ruff also implements the functionality that you get from [`yesqa`](https://github.com/asottile/yesqa),
-and a subset of the rules implemented in [`pyupgrade`](https://pypi.org/project/pyupgrade/) (8/34).
-
-If you're looking to use Ruff, but rely on an unsupported Flake8 plugin, free to file an Issue.
-
-### Do I need to install Rust to use Ruff?
-
-Nope! Ruff is available as [`ruff`](https://pypi.org/project/ruff/) on PyPI:
-
-```shell
-pip install ruff
-```
-
-Ruff ships with wheels for all major platforms, which enables `pip` to install Ruff without relying
-on Rust at all.
-
-### Can I write my own plugins for Ruff?
-
-Ruff does not yet support third-party plugins, though a plugin system is within-scope for the
-project. See [#283](https://github.com/charliermarsh/ruff/issues/2830) for more.
-
-### Does Ruff support NumPy- or Google-style docstrings?
-
-Yes! To enable a specific docstring convention, start by enabling all `pydocstyle` error codes, and
-then selectively disabling based on your [preferred convention](https://www.pydocstyle.org/en/latest/error_codes.html#default-conventions).
-
-For example, if you're coming from `flake8-docstrings`, the following configuration is equivalent to
-`--docstring-convention numpy`:
-
-```toml
-[tool.ruff]
-extend-select = [
-    "D100",
-    "D101",
-    "D102",
-    "D103",
-    "D104",
-    "D105",
-    "D106",
-    "D200",
-    "D201",
-    "D202",
-    "D204",
-    "D205",
-    "D206",
-    "D207",
-    "D208",
-    "D209",
-    "D210",
-    "D211",
-    "D214",
-    "D215",
-    "D300",
-    "D400",
-    "D402",
-    "D403",
-    "D404",
-    "D405",
-    "D406",
-    "D407",
-    "D408",
-    "D409",
-    "D410",
-    "D411",
-    "D412",
-    "D413",
-    "D418",
-    "D419",
-]
-```
-
-Similarly, the following is equivalent to `--docstring-convention google`:
-
-```toml
-[tool.ruff]
-extend-select = [
-    "D100",
-    "D101",
-    "D102",
-    "D103",
-    "D104",
-    "D105",
-    "D106",
-    "D107",
-    "D200",
-    "D201",
-    "D202",
-    "D205",
-    "D206",
-    "D207",
-    "D208",
-    "D209",
-    "D210",
-    "D211",
-    "D212",
-    "D214",
-    "D300",
-    "D402",
-    "D403",
-    "D405",
-    "D410",
-    "D411",
-    "D412",
-    "D414",
-    "D415",
-    "D416",
-    "D417",
-    "D418",
-    "D419",
-]
-```
-
 ## Development

-Ruff is written in Rust (1.63.0). You'll need to install the [Rust toolchain](https://www.rust-lang.org/tools/install)
+ruff is written in Rust (1.63.0). You'll need to install the [Rust toolchain](https://www.rust-lang.org/tools/install)
 for development.

 Assuming you have `cargo` installed, you can run:
@@ -619,13 +334,13 @@ cargo clippy
 cargo test
 ```

-## Releases
+## Deployment

-Ruff is distributed on [PyPI](https://pypi.org/project/ruff/), and published via [`maturin`](https://github.com/PyO3/maturin).
+ruff is distributed on [PyPI](https://pypi.org/project/ruff/), and published via [`maturin`](https://github.com/PyO3/maturin).

 See: `.github/workflows/release.yaml`.

-## Benchmarks
+## Benchmarking

 First, clone [CPython](https://github.com/python/cpython). It's a large and diverse Python codebase,
 which makes it a good target for benchmarking.
@@ -702,9 +417,9 @@ hyperfine --ignore-failure --warmup 5 \

 In order, these evaluate:

- Ruff
+- ruff
 - Pylint
- Pyflakes
+- PyFlakes
 - autoflake
 - pycodestyle
 - Flake8
--- a/examples/generate_rules_table.rs
+++ b/examples/generate_rules_table.rs
@@ -1,29 +1,27 @@
-//! Generate a Markdown-compatible table of supported lint rules.
-
-use strum::IntoEnumIterator;
-
-use ruff::checks::{CheckCategory, CheckCode};
+/// Generate a Markdown-compatible table of supported lint rules.
+use ruff::checks::{CheckCode, ALL_CHECK_CODES, DEFAULT_CHECK_CODES};

 fn main() {
-    for check_category in CheckCategory::iter() {
-        println!("### {}", check_category.title());
-        println!();
+    let mut check_codes: Vec<CheckCode> = ALL_CHECK_CODES.to_vec();
+    check_codes.sort();

-        println!("| Code | Name | Message | Fix |");
-        println!("| ---- | ---- | ------- | --- |");
-        for check_code in CheckCode::iter() {
-            if check_code.category() == check_category {
-                let check_kind = check_code.kind();
-                let fix_token = if check_kind.fixable() { "🛠" } else { "" };
-                println!(
-                    "| {} | {} | {} | {} |",
-                    check_kind.code().as_ref(),
-                    check_kind.as_ref(),
-                    check_kind.body().replace("|", r"\|"),
-                    fix_token
-                );
-            }
-        }
-        println!();
+    println!("| Code | Name | Message |     |     |");
+    println!("| ---- | ---- | ------- | --- | --- |");
+    for check_code in check_codes {
+        let check_kind = check_code.kind();
+        let default_token = if DEFAULT_CHECK_CODES.contains(&check_code) {
+            "✅"
+        } else {
+            ""
+        };
+        let fix_token = if check_kind.fixable() { "🛠" } else { "" };
+        println!(
+            "| {} | {} | {} | {} | {} |",
+            check_kind.code().as_str(),
+            check_kind.name(),
+            check_kind.body(),
+            default_token,
+            fix_token
+        );
    }
 }
--- a/examples/parse_python.rs
+++ b/examples/parse_python.rs
@@ -0,0 +1,23 @@
+/// Print the AST for a given Python file.
+use std::path::PathBuf;
+
+use anyhow::Result;
+use clap::Parser;
+
+use ruff::fs;
+
+#[derive(Debug, Parser)]
+struct Cli {
+    #[arg(required = true)]
+    file: PathBuf,
+}
+
+fn main() -> Result<()> {
+    let cli = Cli::parse();
+
+    let contents = fs::read_file(&cli.file)?;
+    let ast = python_parser::file_input(python_parser::make_strspan(&contents))?.1;
+    println!("{:?}", ast);
+
+    Ok(())
+}
--- a/examples/print_ast.rs
+++ b/examples/print_ast.rs
@@ -1,5 +1,4 @@
-//! Print the AST for a given Python file.
-
+/// Print the AST for a given Python file.
 use std::path::PathBuf;

 use anyhow::Result;
--- a/examples/print_tokens.rs
+++ b/examples/print_tokens.rs
@@ -1,5 +1,4 @@
-//! Print the token stream for a given Python file.
-
+/// Print the token stream for a given Python file.
 use std::path::PathBuf;

 use anyhow::Result;
--- a/output.py
+++ b/output.py
@@ -0,0 +1,37 @@
+from collections import Counter
+
+
+def f() -> None:
+    """Docstring goes here."""
+    for x in range(5):
+        print(x)
+    else:
+        print("Nope!")
+
+
+a = {
+    "a",
+    "b",
+    "c",
+    "a",
+    "b",
+    "c",
+    "a",
+    "b",
+    "c",
+    "a",
+    "b",
+    "c",
+    "a",
+    "b",
+    "c",
+    "a",
+    "b",
+    "c",
+    "a",
+    "b",
+    "c",
+}
+
+cls(title=title, before_text=before_text, after_text=after_text, before_description=before_description, after_description=after_description, transform_type=transform_type)
+
--- a/resources/test/fixtures/B011.py
+++ b/resources/test/fixtures/B011.py
@@ -1,10 +0,0 @@
-"""
-Should emit:
-B011 - on line 8
-B011 - on line 10
-"""
-
-assert 1 != 2
-assert False
-assert 1 != 2, "message"
-assert False, "message"
--- a/resources/test/fixtures/B014.py
+++ b/resources/test/fixtures/B014.py
@@ -1,76 +0,0 @@
-"""
-Should emit:
-B014 - on lines 11, 17, 28, 42, 49, 56, and 74.
-"""
-
-import binascii
-import re
-
-try:
-    pass
-except (Exception, TypeError):
-    # TypeError is a subclass of Exception, so it doesn't add anything
-    pass
-
-try:
-    pass
-except (OSError, OSError) as err:
-    # Duplicate exception types are useless
-    pass
-
-
-class MyError(Exception):
-    pass
-
-
-try:
-    pass
-except (MyError, MyError):
-    # Detect duplicate non-builtin errors
-    pass
-
-
-try:
-    pass
-except (MyError, Exception) as e:
-    # Don't assume that we're all subclasses of Exception
-    pass
-
-
-try:
-    pass
-except (MyError, BaseException) as e:
-    # But we *can* assume that everything is a subclass of BaseException
-    pass
-
-
-try:
-    pass
-except (re.error, re.error):
-    # Duplicate exception types as attributes
-    pass
-
-
-try:
-    pass
-except (IOError, EnvironmentError, OSError):
-    # Detect if a primary exception and any its aliases are present.
-    #
-    # Since Python 3.3, IOError, EnvironmentError, WindowsError, mmap.error,
-    # socket.error and select.error are aliases of OSError. See PEP 3151 for
-    # more info.
-    pass
-
-
-try:
-    pass
-except (MyException, NotImplemented):
-    # NotImplemented is not an exception, let's not crash on it.
-    pass
-
-
-try:
-    pass
-except (ValueError, binascii.Error):
-    # binascii.Error is a subclass of ValueError.
-    pass
--- a/resources/test/fixtures/B025.py
+++ b/resources/test/fixtures/B025.py
@@ -1,38 +0,0 @@
-"""
-Should emit:
-B025 - on lines 15, 22, 31
-"""
-
-import pickle
-
-try:
-    a = 1
-except ValueError:
-    a = 2
-finally:
-    a = 3
-
-try:
-    a = 1
-except ValueError:
-    a = 2
-except ValueError:
-    a = 2
-
-try:
-    a = 1
-except pickle.PickleError:
-    a = 2
-except ValueError:
-    a = 2
-except pickle.PickleError:
-    a = 2
-
-try:
-    a = 1
-except (ValueError, TypeError):
-    a = 2
-except ValueError:
-    a = 2
-except (OSError, TypeError):
-    a = 2
--- a/resources/test/fixtures/C400.py
+++ b/resources/test/fixtures/C400.py
@@ -1 +0,0 @@
-x = list(x for x in range(3))
--- a/resources/test/fixtures/C401.py
+++ b/resources/test/fixtures/C401.py
@@ -1 +0,0 @@
-x = set(x for x in range(3))
--- a/resources/test/fixtures/C402.py
+++ b/resources/test/fixtures/C402.py
@@ -1 +0,0 @@
-d = dict((x, x) for x in range(3))
--- a/resources/test/fixtures/C403.py
+++ b/resources/test/fixtures/C403.py
@@ -1 +0,0 @@
-s = set([x for x in range(3)])
--- a/resources/test/fixtures/C404.py
+++ b/resources/test/fixtures/C404.py
@@ -1 +0,0 @@
-d = dict([(i, i) for i in range(3)])
--- a/resources/test/fixtures/C405.py
+++ b/resources/test/fixtures/C405.py
@@ -1,5 +0,0 @@
-s1 = set([1, 2])
-s2 = set((1, 2))
-s3 = set([])
-s4 = set(())
-s5 = set()
--- a/resources/test/fixtures/C406.py
+++ b/resources/test/fixtures/C406.py
@@ -1,5 +0,0 @@
-d1 = dict([(1, 2)])
-d2 = dict(((1, 2),))
-d3 = dict([])
-d4 = dict(())
-d5 = dict()
--- a/resources/test/fixtures/C408.py
+++ b/resources/test/fixtures/C408.py
@@ -1,5 +0,0 @@
-t = tuple()
-l = list()
-d1 = dict()
-d2 = dict(a=1)
-d3 = dict(**d2)
--- a/resources/test/fixtures/C409.py
+++ b/resources/test/fixtures/C409.py
@@ -1,3 +0,0 @@
-t1 = tuple([1, 2])
-t2 = tuple((1, 2))
-t3 = tuple([])
--- a/resources/test/fixtures/C410.py
+++ b/resources/test/fixtures/C410.py
@@ -1,4 +0,0 @@
-l1 = list([1, 2])
-l2 = list((1, 2))
-l3 = list([])
-l4 = list(())
--- a/resources/test/fixtures/C411.py
+++ b/resources/test/fixtures/C411.py
@@ -1,2 +0,0 @@
-x = [1, 2, 3]
-list([i for i in x])
--- a/resources/test/fixtures/C413.py
+++ b/resources/test/fixtures/C413.py
@@ -1,4 +0,0 @@
-x = [2, 3, 1]
-list(sorted(x))
-reversed(sorted(x))
-reversed(sorted(x, reverse=True))
--- a/resources/test/fixtures/C414.py
+++ b/resources/test/fixtures/C414.py
@@ -1,14 +0,0 @@
-x = [1, 2, 3]
-list(list(x))
-list(tuple(x))
-tuple(list(x))
-tuple(tuple(x))
-set(set(x))
-set(list(x))
-set(tuple(x))
-set(sorted(x))
-set(reversed(x))
-sorted(list(x))
-sorted(tuple(x))
-sorted(sorted(x))
-sorted(reversed(x))
--- a/resources/test/fixtures/C415.py
+++ b/resources/test/fixtures/C415.py
@@ -1,9 +0,0 @@
-lst = [2, 1, 3]
-a = set(lst[::-1])
-b = reversed(lst[::-1])
-c = sorted(lst[::-1])
-d = sorted(lst[::-1], reverse=True)
-e = set(lst[2:-1])
-f = set(lst[:1:-1])
-g = set(lst[::1])
-h = set(lst[::-2])
--- a/resources/test/fixtures/C416.py
+++ b/resources/test/fixtures/C416.py
@@ -1,6 +0,0 @@
-x = [1, 2, 3]
-[i for i in x]
-{i for i in x}
-
-[i for i in x if i > 1]
-[i for i in x for j in x]
--- a/resources/test/fixtures/C417.py
+++ b/resources/test/fixtures/C417.py
@@ -1,6 +0,0 @@
-nums = [1, 2, 3]
-map(lambda x: x + 1, nums)
-map(lambda x: str(x), nums)
-list(map(lambda x: x * 2, nums))
-set(map(lambda x: x % 2 == 0, nums))
-dict(map(lambda v: (v, v**2), nums))
--- a/resources/test/fixtures/D.py
+++ b/resources/test/fixtures/D.py
@@ -1,534 +0,0 @@
-# No docstring, so we can test D100
-from functools import wraps
-import os
-from .expected import Expectation
-from typing import overload
-
-
-expectation = Expectation()
-expect = expectation.expect
-
-expect('class_', 'D101: Missing docstring in public class')
-
-
-class class_:
-
-    expect('meta', 'D419: Docstring is empty')
-
-    class meta:
-        """"""
-
-    @expect('D102: Missing docstring in public method')
-    def method(self=None):
-        pass
-
-    def _ok_since_private(self=None):
-        pass
-
-    @overload
-    def overloaded_method(self, a: int) -> str:
-        ...
-
-    @overload
-    def overloaded_method(self, a: str) -> str:
-        """Foo bar documentation."""
-        ...
-
-    def overloaded_method(a):
-        """Foo bar documentation."""
-        return str(a)
-
-    expect('overloaded_method',
-           "D418: Function/ Method decorated with @overload"
-           " shouldn't contain a docstring")
-
-    @property
-    def foo(self):
-        """The foo of the thing, which isn't in imperitive mood."""
-        return "hello"
-
-    @expect('D102: Missing docstring in public method')
-    def __new__(self=None):
-        pass
-
-    @expect('D107: Missing docstring in __init__')
-    def __init__(self=None):
-        pass
-
-    @expect('D105: Missing docstring in magic method')
-    def __str__(self=None):
-        pass
-
-    @expect('D102: Missing docstring in public method')
-    def __call__(self=None, x=None, y=None, z=None):
-        pass
-
-
-@expect('D419: Docstring is empty')
-def function():
-    """ """
-    def ok_since_nested():
-        pass
-
-    @expect('D419: Docstring is empty')
-    def nested():
-        ''
-
-
-def function_with_nesting():
-    """Foo bar documentation."""
-    @overload
-    def nested_overloaded_func(a: int) -> str:
-        ...
-
-    @overload
-    def nested_overloaded_func(a: str) -> str:
-        """Foo bar documentation."""
-        ...
-
-    def nested_overloaded_func(a):
-        """Foo bar documentation."""
-        return str(a)
-
-
-expect('nested_overloaded_func',
-       "D418: Function/ Method decorated with @overload"
-       " shouldn't contain a docstring")
-
-
-@overload
-def overloaded_func(a: int) -> str:
-    ...
-
-
-@overload
-def overloaded_func(a: str) -> str:
-    """Foo bar documentation."""
-    ...
-
-
-def overloaded_func(a):
-    """Foo bar documentation."""
-    return str(a)
-
-
-expect('overloaded_func',
-       "D418: Function/ Method decorated with @overload"
-       " shouldn't contain a docstring")
-
-
-@expect('D200: One-line docstring should fit on one line with quotes '
-        '(found 3)')
-@expect('D212: Multi-line docstring summary should start at the first line')
-def asdlkfasd():
-    """
-    Wrong.
-    """
-
-
-@expect('D201: No blank lines allowed before function docstring (found 1)')
-def leading_space():
-
-    """Leading space."""
-
-
-@expect('D202: No blank lines allowed after function docstring (found 1)')
-def trailing_space():
-    """Leading space."""
-
-    pass
-
-
-@expect('D201: No blank lines allowed before function docstring (found 1)')
-@expect('D202: No blank lines allowed after function docstring (found 1)')
-def trailing_and_leading_space():
-
-    """Trailing and leading space."""
-
-    pass
-
-
-expect('LeadingSpaceMissing',
-       'D203: 1 blank line required before class docstring (found 0)')
-
-
-class LeadingSpaceMissing:
-    """Leading space missing."""
-
-
-expect('WithLeadingSpace',
-       'D211: No blank lines allowed before class docstring (found 1)')
-
-
-class WithLeadingSpace:
-
-    """With leading space."""
-
-
-expect('TrailingSpace',
-       'D204: 1 blank line required after class docstring (found 0)')
-expect('TrailingSpace',
-       'D211: No blank lines allowed before class docstring (found 1)')
-
-
-class TrailingSpace:
-
-    """TrailingSpace."""
-    pass
-
-
-expect('LeadingAndTrailingSpaceMissing',
-       'D203: 1 blank line required before class docstring (found 0)')
-expect('LeadingAndTrailingSpaceMissing',
-       'D204: 1 blank line required after class docstring (found 0)')
-
-
-class LeadingAndTrailingSpaceMissing:
-    """Leading and trailing space missing."""
-    pass
-
-
-@expect('D205: 1 blank line required between summary line and description '
-        '(found 0)')
-@expect('D213: Multi-line docstring summary should start at the second line')
-def multi_line_zero_separating_blanks():
-    """Summary.
-    Description.
-
-    """
-
-
-@expect('D205: 1 blank line required between summary line and description '
-        '(found 2)')
-@expect('D213: Multi-line docstring summary should start at the second line')
-def multi_line_two_separating_blanks():
-    """Summary.
-
-
-    Description.
-
-    """
-
-
-@expect('D213: Multi-line docstring summary should start at the second line')
-def multi_line_one_separating_blanks():
-    """Summary.
-
-    Description.
-
-    """
-
-
-@expect('D207: Docstring is under-indented')
-@expect('D213: Multi-line docstring summary should start at the second line')
-def asdfsdf():
-    """Summary.
-
-Description.
-
-    """
-
-
-@expect('D207: Docstring is under-indented')
-@expect('D213: Multi-line docstring summary should start at the second line')
-def asdsdfsdffsdf():
-    """Summary.
-
-    Description.
-
-"""
-
-
-@expect('D208: Docstring is over-indented')
-@expect('D213: Multi-line docstring summary should start at the second line')
-def asdfsdsdf24():
-    """Summary.
-
-       Description.
-
-    """
-
-
-@expect('D208: Docstring is over-indented')
-@expect('D213: Multi-line docstring summary should start at the second line')
-def asdfsdsdfsdf24():
-    """Summary.
-
-    Description.
-
-        """
-
-
-@expect('D208: Docstring is over-indented')
-@expect('D213: Multi-line docstring summary should start at the second line')
-def asdfsdfsdsdsdfsdf24():
-    """Summary.
-
-        Description.
-
-    """
-
-
-@expect('D209: Multi-line docstring closing quotes should be on a separate '
-        'line')
-@expect('D213: Multi-line docstring summary should start at the second line')
-def asdfljdf24():
-    """Summary.
-
-    Description."""
-
-
-@expect('D210: No whitespaces allowed surrounding docstring text')
-def endswith():
-    """Whitespace at the end. """
-
-
-@expect('D210: No whitespaces allowed surrounding docstring text')
-def around():
-    """ Whitespace at everywhere. """
-
-
-@expect('D210: No whitespaces allowed surrounding docstring text')
-@expect('D213: Multi-line docstring summary should start at the second line')
-def multiline():
-    """ Whitespace at the beginning.
-
-    This is the end.
-    """
-
-
-@expect('D300: Use """triple double quotes""" (found \'\'\'-quotes)')
-def triple_single_quotes_raw():
-    r'''Summary.'''
-
-
-@expect('D300: Use """triple double quotes""" (found \'\'\'-quotes)')
-def triple_single_quotes_raw_uppercase():
-    R'''Summary.'''
-
-
-@expect('D300: Use """triple double quotes""" (found \'-quotes)')
-def single_quotes_raw():
-    r'Summary.'
-
-
-@expect('D300: Use """triple double quotes""" (found \'-quotes)')
-def single_quotes_raw_uppercase():
-    R'Summary.'
-
-
-@expect('D300: Use """triple double quotes""" (found \'-quotes)')
-@expect('D301: Use r""" if any backslashes in a docstring')
-def single_quotes_raw_uppercase_backslash():
-    R'Sum\mary.'
-
-
-@expect('D301: Use r""" if any backslashes in a docstring')
-def double_quotes_backslash():
-    """Sum\\mary."""
-
-
-@expect('D301: Use r""" if any backslashes in a docstring')
-def double_quotes_backslash_uppercase():
-    R"""Sum\\mary."""
-
-
-@expect('D213: Multi-line docstring summary should start at the second line')
-def exceptions_of_D301():
-    """Exclude some backslashes from D301.
-
-    In particular, line continuations \
-    and unicode literals \u0394 and \N{GREEK CAPITAL LETTER DELTA}.
-    They are considered to be intentionally unescaped.
-    """
-
-
-@expect("D400: First line should end with a period (not 'y')")
-@expect("D415: First line should end with a period, question mark, "
-        "or exclamation point (not 'y')")
-def lwnlkjl():
-    """Summary"""
-
-
-@expect("D401: First line should be in imperative mood "
-        "(perhaps 'Return', not 'Returns')")
-def liouiwnlkjl():
-    """Returns foo."""
-
-
-@expect("D401: First line should be in imperative mood; try rephrasing "
-        "(found 'Constructor')")
-def sdgfsdg23245():
-    """Constructor for a foo."""
-
-
-@expect("D401: First line should be in imperative mood; try rephrasing "
-        "(found 'Constructor')")
-def sdgfsdg23245777():
-    """Constructor."""
-
-
-@expect('D402: First line should not be the function\'s "signature"')
-def foobar():
-    """Signature: foobar()."""
-
-
-@expect('D213: Multi-line docstring summary should start at the second line')
-def new_209():
-    """First line.
-
-    More lines.
-    """
-    pass
-
-
-@expect('D213: Multi-line docstring summary should start at the second line')
-def old_209():
-    """One liner.
-
-    Multi-line comments. OK to have extra blank line
-
-    """
-
-
-@expect("D103: Missing docstring in public function")
-def oneliner_d102(): return
-
-
-@expect("D400: First line should end with a period (not 'r')")
-@expect("D415: First line should end with a period, question mark,"
-        " or exclamation point (not 'r')")
-def oneliner_withdoc(): """One liner"""
-
-
-def ignored_decorator(func):   # noqa: D400,D401,D415
-    """Runs something"""
-    func()
-    pass
-
-
-def decorator_for_test(func):   # noqa: D400,D401,D415
-    """Runs something"""
-    func()
-    pass
-
-
-@ignored_decorator
-def oneliner_ignored_decorator(): """One liner"""
-
-
-@decorator_for_test
-@expect("D400: First line should end with a period (not 'r')")
-@expect("D415: First line should end with a period, question mark,"
-        " or exclamation point (not 'r')")
-def oneliner_with_decorator_expecting_errors(): """One liner"""
-
-
-@decorator_for_test
-def valid_oneliner_with_decorator(): """One liner."""
-
-
-@expect("D207: Docstring is under-indented")
-@expect('D213: Multi-line docstring summary should start at the second line')
-def docstring_start_in_same_line(): """First Line.
-
-    Second Line
-    """
-
-
-def function_with_lambda_arg(x=lambda y: y):
-    """Wrap the given lambda."""
-
-
-@expect('D213: Multi-line docstring summary should start at the second line')
-def a_following_valid_function(x=None):
-    """Check for a bug where the previous function caused an assertion.
-
-    The assertion was caused in the next function, so this one is necessary.
-
-    """
-
-
-def outer_function():
-    """Do something."""
-    def inner_function():
-        """Do inner something."""
-        return 0
-
-
-@expect("D400: First line should end with a period (not 'g')")
-@expect("D401: First line should be in imperative mood "
-        "(perhaps 'Run', not 'Runs')")
-@expect("D415: First line should end with a period, question mark, "
-        "or exclamation point (not 'g')")
-def docstring_bad():
-    """Runs something"""
-    pass
-
-
-def docstring_bad_ignore_all():  # noqa
-    """Runs something"""
-    pass
-
-
-def docstring_bad_ignore_one():  # noqa: D400,D401,D415
-    """Runs something"""
-    pass
-
-
-@expect("D401: First line should be in imperative mood "
-        "(perhaps 'Run', not 'Runs')")
-def docstring_ignore_some_violations_but_catch_D401():  # noqa: E501,D400,D415
-    """Runs something"""
-    pass
-
-
-@expect(
-    "D401: First line should be in imperative mood "
-    "(perhaps 'Initiate', not 'Initiates')"
-)
-def docstring_initiates():
-    """Initiates the process."""
-
-
-@expect(
-    "D401: First line should be in imperative mood "
-    "(perhaps 'Initialize', not 'Initializes')"
-)
-def docstring_initializes():
-    """Initializes the process."""
-
-
-@wraps(docstring_bad_ignore_one)
-def bad_decorated_function():
-    """Bad (E501) but decorated"""
-    pass
-
-
-def valid_google_string():  # noqa: D400
-    """Test a valid something!"""
-
-
-@expect("D415: First line should end with a period, question mark, "
-        "or exclamation point (not 'g')")
-def bad_google_string():  # noqa: D400
-    """Test a valid something"""
-
-
-# This is reproducing a bug where AttributeError is raised when parsing class
-# parameters as functions for Google / Numpy conventions.
-class Blah:  # noqa: D203,D213
-    """A Blah.
-
-    Parameters
-    ----------
-    x : int
-
-    """
-
-    def __init__(self, x):
-        pass
-
-
-expect(os.path.normcase(__file__ if __file__[-1] != 'c' else __file__[:-1]),
-       'D100: Missing docstring in public module')
--- a/resources/test/fixtures/F401_0.py
+++ b/resources/test/fixtures/F401_0.py
@@ -61,28 +61,7 @@ Y = TypeVar("Y", bound="Dict")
 Z = TypeVar("Z", "List", "Set")

 a = list["Fruit"]
-b = Union["""Nut""", None]
+b = Union["Nut", None]
 c = cast("Vegetable", b)

 Field = lambda default=MISSING: field(default=default)
-
-
-# Test: access a sub-importation via an alias.
-import pyarrow as pa
-import pyarrow.csv
-
-print(pa.csv.read_csv("test.csv"))
-
-
-# Test: referencing an import via TypeAlias.
-import sys
-
-import numpy as np
-
-if sys.version_info >= (3, 10):
-    from typing import TypeAlias
-else:
-    from typing_extensions import TypeAlias
-
-
-CustomInt: TypeAlias = "np.int8 | np.int16"
--- a/resources/test/fixtures/F401_1.py
+++ b/resources/test/fixtures/F401_1.py
@@ -1,5 +0,0 @@
-"""Access a sub-importation via an alias."""
-import pyarrow as pa
-import pyarrow.csv
-
-print(pa.csv.read_csv("test.csv"))
--- a/resources/test/fixtures/F401_2.py
+++ b/resources/test/fixtures/F401_2.py
@@ -1,12 +0,0 @@
-"""Test: referencing an import via TypeAlias."""
-import sys
-
-import numpy as np
-
-if sys.version_info >= (3, 10):
-    from typing import TypeAlias
-else:
-    from typing_extensions import TypeAlias
-
-
-CustomInt: TypeAlias = "np.int8 | np.int16"
--- a/resources/test/fixtures/F401_3.py
+++ b/resources/test/fixtures/F401_3.py
@@ -1,14 +0,0 @@
-"""Test: referencing an import via TypeAlias (with future annotations)."""
-from __future__ import annotations
-
-import sys
-
-import numpy as np
-
-if sys.version_info >= (3, 10):
-    from typing import TypeAlias
-else:
-    from typing_extensions import TypeAlias
-
-
-CustomInt: TypeAlias = np.int8 | np.int16
--- a/resources/test/fixtures/F401_4.py
+++ b/resources/test/fixtures/F401_4.py
@@ -1,14 +0,0 @@
-"""Test: referencing an import via TypeAlias (with future annotations and quotes)."""
-from __future__ import annotations
-
-import sys
-
-import numpy as np
-
-if sys.version_info >= (3, 10):
-    from typing import TypeAlias
-else:
-    from typing_extensions import TypeAlias
-
-
-CustomInt: TypeAlias = "np.int8 | np.int16"
--- a/resources/test/fixtures/F404.py
+++ b/resources/test/fixtures/F404.py
@@ -1,3 +1,4 @@
+from __future__ import print_function
 """Docstring"""
 from __future__ import absolute_import

--- a/resources/test/fixtures/F821.py
+++ b/resources/test/fixtures/F821.py
@@ -89,45 +89,3 @@ A = (
    f'B'
    f'{B}'
 )
-
-
-from typing import Annotated, Literal
-
-
-def arbitrary_callable() -> None:
-    ...
-
-
-class PEP593Test:
-    field: Annotated[
-        int,
-        "base64",
-        arbitrary_callable(),
-        123,
-        (1, 2, 3),
-    ]
-    field_with_stringified_type: Annotated[
-        "PEP593Test",
-        123,
-    ]
-    field_with_undefined_stringified_type: Annotated[
-        "PEP593Test123",
-        123,
-    ]
-    field_with_nested_subscript: Annotated[
-        dict[Literal["foo"], str],
-        123,
-    ]
-    field_with_undefined_nested_subscript: Annotated[
-        dict["foo", "bar"],  # Expected to fail as undefined.
-        123,
-    ]
-
-
-def in_ipython_notebook() -> bool:
-    try:
-        # autoimported by notebooks
-        get_ipython()  # type: ignore[name-defined]
-    except NameError:
-        return False  # not in notebook
-    return True
--- a/resources/test/fixtures/M001.py
+++ b/resources/test/fixtures/M001.py
@@ -15,9 +15,6 @@ def f() -> None:
    # Invalid
    d = 1  # noqa: F841, E501

-    # Invalid (and unimplemented)
-    d = 1  # noqa: F841, W191
-

 # Valid
 _ = """Lorem ipsum dolor sit amet.
--- a/resources/test/fixtures/N801.py
+++ b/resources/test/fixtures/N801.py
@@ -1,34 +0,0 @@
-class bad:
-    pass
-
-
-class _bad:
-    pass
-
-
-class bad_class:
-    pass
-
-
-class Bad_Class:
-    pass
-
-
-class BAD_CLASS:
-    pass
-
-
-class Good:
-    pass
-
-
-class _Good:
-    pass
-
-
-class GoodClass:
-    pass
-
-
-class GOOD:
-    pass
--- a/resources/test/fixtures/N802.py
+++ b/resources/test/fixtures/N802.py
@@ -1,26 +0,0 @@
-def Bad():
-    pass
-
-
-def _Bad():
-    pass
-
-
-def BAD():
-    pass
-
-
-def BAD_FUNC():
-    pass
-
-
-def good():
-    pass
-
-
-def _good():
-    pass
-
-
-def good_func():
-    pass
--- a/resources/test/fixtures/N803.py
+++ b/resources/test/fixtures/N803.py
@@ -1,7 +0,0 @@
-def func(a, A):
-    return a, A
-
-
-class Class:
-    def method(self, a, A):
-        return a, A
--- a/resources/test/fixtures/N804.py
+++ b/resources/test/fixtures/N804.py
@@ -1,19 +0,0 @@
-class Class:
-    @classmethod
-    def bad_class_method(this):
-        pass
-
-    @classmethod
-    def good_class_method(cls):
-        pass
-
-    def method(self):
-        pass
-
-    @staticmethod
-    def static_method(x):
-        return x
-
-
-def func(x):
-    return x
--- a/resources/test/fixtures/N805.py
+++ b/resources/test/fixtures/N805.py
@@ -1,26 +0,0 @@
-import random
-
-
-class Class:
-    def bad_method(this):
-        pass
-
-    if random.random(0, 2) == 0:
-
-        def extra_bad_method(this):
-            pass
-
-    def good_method(self):
-        pass
-
-    @classmethod
-    def class_method(cls):
-        pass
-
-    @staticmethod
-    def static_method(x):
-        return x
-
-
-def func(x):
-    return x
--- a/resources/test/fixtures/N807.py
+++ b/resources/test/fixtures/N807.py
@@ -1,15 +0,0 @@
-def __bad__():
-    pass
-
-
-def __good():
-    pass
-
-
-def good__():
-    pass
-
-
-class Class:
-    def __good__(self):
-        pass
--- a/resources/test/fixtures/N811.py
+++ b/resources/test/fixtures/N811.py
@@ -1 +0,0 @@
-from mod import BAD as bad
--- a/resources/test/fixtures/N812.py
+++ b/resources/test/fixtures/N812.py
@@ -1 +0,0 @@
-from mod import bad as Bad
--- a/resources/test/fixtures/N813.py
+++ b/resources/test/fixtures/N813.py
@@ -1 +0,0 @@
-from mod import CamelCase as camelcase
--- a/resources/test/fixtures/N814.py
+++ b/resources/test/fixtures/N814.py
@@ -1 +0,0 @@
-from mod import CamelCase as CAMELCASE
--- a/resources/test/fixtures/N817.py
+++ b/resources/test/fixtures/N817.py
@@ -1 +0,0 @@
-from mod import CamelCase as CC
--- a/resources/test/fixtures/R001.py
+++ b/resources/test/fixtures/R001.py
--- a/resources/test/fixtures/R002.py
+++ b/resources/test/fixtures/R002.py
@@ -0,0 +1,3 @@
+self.assertEquals (1, 2)
+self.assertEquals(1, 2)
+self.assertEqual(3, 4)
--- a/resources/test/fixtures/SPR001.py
+++ b/resources/test/fixtures/SPR001.py
--- a/resources/test/fixtures/U002.py
+++ b/resources/test/fixtures/U002.py
@@ -1,15 +0,0 @@
-from os.path import abspath
-
-x = abspath(__file__)
-
-
-import os
-
-
-y = os.path.abspath(__file__)
-
-
-from os import path
-
-
-z = path.abspath(__file__)
--- a/resources/test/fixtures/U003.py
+++ b/resources/test/fixtures/U003.py
@@ -1,5 +0,0 @@
-type('')
-type(b'')
-type(0)
-type(0.)
-type(0j)
--- a/resources/test/fixtures/U005.py
+++ b/resources/test/fixtures/U005.py
@@ -1,10 +0,0 @@
-import unittest
-
-
-class Suite(unittest.TestCase):
-    def test(self) -> None:
-        self.assertEquals (1, 2)
-        self.assertEquals(1, 2)
-        self.assertEqual(3, 4)
-        self.failUnlessAlmostEqual(1, 1.1)
-        self.assertNotRegexpMatches("a", "b")
--- a/resources/test/fixtures/U006.py
+++ b/resources/test/fixtures/U006.py
@@ -1,12 +0,0 @@
-from typing import List
-
-
-def f(x: List[str]) -> None:
-    ...
-
-
-import typing
-
-
-def f(x: typing.List[str]) -> None:
-    ...
--- a/resources/test/fixtures/U007.py
+++ b/resources/test/fixtures/U007.py
@@ -1,40 +0,0 @@
-from typing import Optional
-
-
-def f(x: Optional[str]) -> None:
-    ...
-
-
-import typing
-
-
-def f(x: typing.Optional[str]) -> None:
-    ...
-
-
-from typing import Union
-
-
-def f(x: Union[str, int, Union[float, bytes]]) -> None:
-    ...
-
-
-import typing
-
-
-def f(x: typing.Union[str, int]) -> None:
-    ...
-
-
-from typing import Union
-
-
-def f(x: "Union[str, int, Union[float, bytes]]") -> None:
-    ...
-
-
-import typing
-
-
-def f(x: "typing.Union[str, int]") -> None:
-    ...
--- a/resources/test/fixtures/W292_0.py
+++ b/resources/test/fixtures/W292_0.py
@@ -1,2 +0,0 @@
-def fn() -> None:
-    pass
--- a/resources/test/fixtures/W292_1.py
+++ b/resources/test/fixtures/W292_1.py
@@ -1,2 +0,0 @@
-def fn() -> None:
-    pass  # noqa: W292
--- a/resources/test/fixtures/W292_2.py
+++ b/resources/test/fixtures/W292_2.py
@@ -1,2 +0,0 @@
-def fn() -> None:
-    pass
--- a/resources/test/fixtures/canonical_google_examples.py
+++ b/resources/test/fixtures/canonical_google_examples.py
@@ -1,108 +0,0 @@
-"""A one line summary of the module or program, terminated by a period.
-
-Leave one blank line.  The rest of this docstring should contain an
-overall description of the module or program.  Optionally, it may also
-contain a brief description of exported classes and functions and/or usage
-examples.
-
-  Typical usage example:
-
-  foo = ClassFoo()
-  bar = foo.FunctionBar()
-"""
-# above: "2.8.2 Modules" section example
-# https://google.github.io/styleguide/pyguide.html#382-modules
-
-# Examples from the official "Google Python Style Guide" documentation:
-#     * As HTML: https://google.github.io/styleguide/pyguide.html
-#     * Source Markdown:
-#         https://github.com/google/styleguide/blob/gh-pages/pyguide.md
-
-import os
-from .expected import Expectation
-
-expectation = Expectation()
-expect = expectation.expect
-
-# module docstring expected violations:
-expectation.expected.add((
-    os.path.normcase(__file__),
-    "D213: Multi-line docstring summary should start at the second line"))
-
-
-# "3.8.3 Functions and Methods" section example
-# https://google.github.io/styleguide/pyguide.html#383-functions-and-methods
-@expect("D213: Multi-line docstring summary should start at the second line",
-        arg_count=3)
-@expect("D401: First line should be in imperative mood "
-        "(perhaps 'Fetch', not 'Fetches')", arg_count=3)
-@expect("D406: Section name should end with a newline "
-        "('Raises', not 'Raises:')", arg_count=3)
-@expect("D406: Section name should end with a newline "
-        "('Returns', not 'Returns:')", arg_count=3)
-@expect("D407: Missing dashed underline after section ('Raises')", arg_count=3)
-@expect("D407: Missing dashed underline after section ('Returns')",
-        arg_count=3)
-@expect("D413: Missing blank line after last section ('Raises')", arg_count=3)
-def fetch_bigtable_rows(big_table, keys, other_silly_variable=None):
-    """Fetches rows from a Bigtable.
-
-    Retrieves rows pertaining to the given keys from the Table instance
-    represented by big_table.  Silly things may happen if
-    other_silly_variable is not None.
-
-    Args:
-        big_table: An open Bigtable Table instance.
-        keys: A sequence of strings representing the key of each table row
-            to fetch.
-        other_silly_variable: Another optional variable, that has a much
-            longer name than the other args, and which does nothing.
-
-    Returns:
-        A dict mapping keys to the corresponding table row data
-        fetched. Each row is represented as a tuple of strings. For
-        example:
-
-        {'Serak': ('Rigel VII', 'Preparer'),
-         'Zim': ('Irk', 'Invader'),
-         'Lrrr': ('Omicron Persei 8', 'Emperor')}
-
-        If a key from the keys argument is missing from the dictionary,
-        then that row was not found in the table.
-
-    Raises:
-        IOError: An error occurred accessing the bigtable.Table object.
-    """
-
-
-# "3.8.4 Classes" section example
-# https://google.github.io/styleguide/pyguide.html#384-classes
-@expect("D203: 1 blank line required before class docstring (found 0)")
-@expect("D213: Multi-line docstring summary should start at the second line")
-@expect("D406: Section name should end with a newline "
-        "('Attributes', not 'Attributes:')")
-@expect("D407: Missing dashed underline after section ('Attributes')")
-@expect("D413: Missing blank line after last section ('Attributes')")
-class SampleClass:
-    """Summary of class here.
-
-    Longer class information....
-    Longer class information....
-
-    Attributes:
-        likes_spam: A boolean indicating if we like SPAM or not.
-        eggs: An integer count of the eggs we have laid.
-    """
-
-    @expect("D401: First line should be in imperative mood "
-            "(perhaps 'Init', not 'Inits')", arg_count=2)
-    def __init__(self, likes_spam=False):
-        """Inits SampleClass with blah."""
-        if self:  # added to avoid NameError when run via @expect decorator
-            self.likes_spam = likes_spam
-            self.eggs = 0
-
-    @expect("D401: First line should be in imperative mood "
-            "(perhaps 'Perform', not 'Performs')", arg_count=1)
-    def public_method(self):
-        """Performs operation blah."""
--- a/resources/test/fixtures/canonical_numpy_examples.py
+++ b/resources/test/fixtures/canonical_numpy_examples.py
@@ -1,163 +0,0 @@
-"""This is the docstring for the example.py module.  Modules names should
-have short, all-lowercase names.  The module name may have underscores if
-this improves readability.
-
-Every module should have a docstring at the very top of the file.  The
-module's docstring may extend over multiple lines.  If your docstring does
-extend over multiple lines, the closing three quotation marks must be on
-a line by itself, preferably preceded by a blank line.
-
-"""
-
-# Example source file from the official "numpydoc docstring guide"
-# documentation (with the modification of commenting out all the original
-# ``import`` lines, plus adding this note and ``Expectation`` code):
-#     * As HTML: https://numpydoc.readthedocs.io/en/latest/example.html
-#     * Source Python:
-#         https://github.com/numpy/numpydoc/blob/master/doc/example.py
-
-# from __future__ import division, absolute_import, print_function
-#
-# import os  # standard library imports first
-#
-# Do NOT import using *, e.g. from numpy import *
-#
-# Import the module using
-#
-#   import numpy
-#
-# instead or import individual functions as needed, e.g
-#
-#  from numpy import array, zeros
-#
-# If you prefer the use of abbreviated module names, we suggest the
-# convention used by NumPy itself::
-#
-# import numpy as np
-# import matplotlib as mpl
-# import matplotlib.pyplot as plt
-#
-# These abbreviated names are not to be used in docstrings; users must
-# be able to paste and execute docstrings after importing only the
-# numpy module itself, unabbreviated.
-
-import os
-from .expected import Expectation
-
-expectation = Expectation()
-expect = expectation.expect
-
-# module docstring expected violations:
-expectation.expected.add((
-    os.path.normcase(__file__),
-    "D205: 1 blank line required between summary line and description "
-    "(found 0)"))
-expectation.expected.add((
-    os.path.normcase(__file__),
-    "D213: Multi-line docstring summary should start at the second line"))
-expectation.expected.add((
-    os.path.normcase(__file__),
-    "D400: First line should end with a period (not 'd')"))
-expectation.expected.add((
-    os.path.normcase(__file__),
-    "D404: First word of the docstring should not be `This`"))
-expectation.expected.add((
-    os.path.normcase(__file__),
-    "D415: First line should end with a period, question mark, or exclamation "
-    "point (not 'd')"))
-
-
-@expect("D213: Multi-line docstring summary should start at the second line",
-        arg_count=3)
-@expect("D401: First line should be in imperative mood; try rephrasing "
-        "(found 'A')", arg_count=3)
-@expect("D413: Missing blank line after last section ('Examples')",
-        arg_count=3)
-def foo(var1, var2, long_var_name='hi'):
-    r"""A one-line summary that does not use variable names.
-
-    Several sentences providing an extended description. Refer to
-    variables using back-ticks, e.g. `var`.
-
-    Parameters
-    ----------
-    var1 : array_like
-        Array_like means all those objects -- lists, nested lists, etc. --
-        that can be converted to an array.  We can also refer to
-        variables like `var1`.
-    var2 : int
-        The type above can either refer to an actual Python type
-        (e.g. ``int``), or describe the type of the variable in more
-        detail, e.g. ``(N,) ndarray`` or ``array_like``.
-    long_var_name : {'hi', 'ho'}, optional
-        Choices in brackets, default first when optional.
-
-    Returns
-    -------
-    type
-        Explanation of anonymous return value of type ``type``.
-    describe : type
-        Explanation of return value named `describe`.
-    out : type
-        Explanation of `out`.
-    type_without_description
-
-    Other Parameters
-    ----------------
-    only_seldom_used_keywords : type
-        Explanation
-    common_parameters_listed_above : type
-        Explanation
-
-    Raises
-    ------
-    BadException
-        Because you shouldn't have done that.
-
-    See Also
-    --------
-    numpy.array : Relationship (optional).
-    numpy.ndarray : Relationship (optional), which could be fairly long, in
-                    which case the line wraps here.
-    numpy.dot, numpy.linalg.norm, numpy.eye
-
-    Notes
-    -----
-    Notes about the implementation algorithm (if needed).
-
-    This can have multiple paragraphs.
-
-    You may include some math:
-
-    .. math:: X(e^{j\omega } ) = x(n)e^{ - j\omega n}
-
-    And even use a Greek symbol like :math:`\omega` inline.
-
-    References
-    ----------
-    Cite the relevant literature, e.g. [1]_.  You may also cite these
-    references in the notes section above.
-
-    .. [1] O. McNoleg, "The integration of GIS, remote sensing,
-       expert systems and adaptive co-kriging for environmental habitat
-       modelling of the Highland Haggis using object-oriented, fuzzy-logic
-       and neural-network techniques," Computers & Geosciences, vol. 22,
-       pp. 585-588, 1996.
-
-    Examples
-    --------
-    These are written in doctest format, and should illustrate how to
-    use the function.
-
-    >>> a = [1, 2, 3]
-    >>> print([x + 3 for x in a])
-    [4, 5, 6]
-    >>> print("a\nb")
-    a
-    b
-    """
-    # After closing class docstring, there should be one blank line to
-    # separate following codes (according to PEP257).
-    # But for function, method and module, there should be no blank lines
-    # after closing the docstring.
-    pass
--- a/resources/test/fixtures/sections.py
+++ b/resources/test/fixtures/sections.py
@@ -1,497 +0,0 @@
-"""A valid module docstring."""
-
-from .expected import Expectation
-
-expectation = Expectation()
-expect = expectation.expect
-
-
-_D213 = 'D213: Multi-line docstring summary should start at the second line'
-_D400 = "D400: First line should end with a period (not '!')"
-
-
-@expect(_D213)
-@expect("D405: Section name should be properly capitalized "
-        "('Returns', not 'returns')")
-def not_capitalized():  # noqa: D416
-    """Toggle the gizmo.
-
-    returns
-    -------
-    A value of some sort.
-
-    """
-
-
-@expect(_D213)
-@expect("D406: Section name should end with a newline "
-        "('Returns', not 'Returns:')")
-def superfluous_suffix():  # noqa: D416
-    """Toggle the gizmo.
-
-    Returns:
-    -------
-    A value of some sort.
-
-    """
-
-
-@expect(_D213)
-@expect("D407: Missing dashed underline after section ('Returns')")
-def no_underline():  # noqa: D416
-    """Toggle the gizmo.
-
-    Returns
-    A value of some sort.
-
-    """
-
-
-@expect(_D213)
-@expect("D407: Missing dashed underline after section ('Returns')")
-@expect("D414: Section has no content ('Returns')")
-def no_underline_and_no_description():  # noqa: D416
-    """Toggle the gizmo.
-
-    Returns
-
-    """
-
-
-@expect(_D213)
-@expect("D410: Missing blank line after section ('Returns')")
-@expect("D414: Section has no content ('Returns')")
-@expect("D411: Missing blank line before section ('Yields')")
-@expect("D414: Section has no content ('Yields')")
-def consecutive_sections():  # noqa: D416
-    """Toggle the gizmo.
-
-    Returns
-    -------
-    Yields
-    ------
-
-    Raises
-    ------
-    Questions.
-
-    """
-
-
-@expect(_D213)
-@expect("D408: Section underline should be in the line following the "
-        "section's name ('Returns')")
-def blank_line_before_underline():  # noqa: D416
-    """Toggle the gizmo.
-
-    Returns
-
-    -------
-    A value of some sort.
-
-    """
-
-
-@expect(_D213)
-@expect("D409: Section underline should match the length of its name "
-        "(Expected 7 dashes in section 'Returns', got 2)")
-def bad_underline_length():  # noqa: D416
-    """Toggle the gizmo.
-
-    Returns
-    --
-    A value of some sort.
-
-    """
-
-
-@expect(_D213)
-@expect("D413: Missing blank line after last section ('Returns')")
-def no_blank_line_after_last_section():  # noqa: D416
-    """Toggle the gizmo.
-
-    Returns
-    -------
-    A value of some sort.
-    """
-
-
-@expect(_D213)
-@expect("D411: Missing blank line before section ('Returns')")
-def no_blank_line_before_section():  # noqa: D416
-    """Toggle the gizmo.
-
-    The function's description.
-    Returns
-    -------
-    A value of some sort.
-
-    """
-
-
-@expect(_D213)
-@expect("D214: Section is over-indented ('Returns')")
-def section_overindented():  # noqa: D416
-    """Toggle the gizmo.
-
-        Returns
-    -------
-    A value of some sort.
-
-    """
-
-
-@expect(_D213)
-@expect("D215: Section underline is over-indented (in section 'Returns')")
-def section_underline_overindented():  # noqa: D416
-    """Toggle the gizmo.
-
-    Returns
-        -------
-    A value of some sort.
-
-    """
-
-
-@expect(_D213)
-@expect("D215: Section underline is over-indented (in section 'Returns')")
-@expect("D413: Missing blank line after last section ('Returns')")
-@expect("D414: Section has no content ('Returns')")
-def section_underline_overindented_and_contentless():  # noqa: D416
-    """Toggle the gizmo.
-
-    Returns
-        -------
-    """
-
-
-@expect(_D213)
-def ignore_non_actual_section():  # noqa: D416
-    """Toggle the gizmo.
-
-    This is the function's description, which will also specify what it
-    returns
-
-    """
-
-
-@expect(_D213)
-@expect("D401: First line should be in imperative mood "
-        "(perhaps 'Return', not 'Returns')")
-@expect("D400: First line should end with a period (not 's')")
-@expect("D415: First line should end with a period, question "
-        "mark, or exclamation point (not 's')")
-@expect("D205: 1 blank line required between summary line and description "
-        "(found 0)")
-def section_name_in_first_line():  # noqa: D416
-    """Returns
-    -------
-    A value of some sort.
-
-    """
-
-
-@expect(_D213)
-@expect("D405: Section name should be properly capitalized "
-        "('Short Summary', not 'Short summary')")
-@expect("D412: No blank lines allowed between a section header and its "
-        "content ('Short Summary')")
-@expect("D409: Section underline should match the length of its name "
-        "(Expected 7 dashes in section 'Returns', got 6)")
-@expect("D410: Missing blank line after section ('Returns')")
-@expect("D411: Missing blank line before section ('Raises')")
-@expect("D406: Section name should end with a newline "
-        "('Raises', not 'Raises:')")
-@expect("D407: Missing dashed underline after section ('Raises')")
-def multiple_sections():  # noqa: D416
-    """Toggle the gizmo.
-
-    Short summary
-    -------------
-
-    This is the function's description, which will also specify what it
-    returns.
-
-    Returns
-    ------
-    Many many wonderful things.
-    Raises:
-    My attention.
-
-    """
-
-
-@expect(_D213)
-def false_positive_section_prefix():  # noqa: D416
-    """Toggle the gizmo.
-
-    Parameters
-    ----------
-    attributes_are_fun: attributes for the function.
-
-    """
-
-
-@expect(_D213)
-def section_names_as_parameter_names():  # noqa: D416
-    """Toggle the gizmo.
-
-    Parameters
-    ----------
-    notes : list
-        A list of wonderful notes.
-    examples: list
-        A list of horrible examples.
-
-    """
-
-
-@expect(_D213)
-@expect("D414: Section has no content ('Returns')")
-def valid_google_style_section():  # noqa: D406, D407
-    """Toggle the gizmo.
-
-    Args:
-        note: A random string.
-
-    Returns:
-
-    Raises:
-        RandomError: A random error that occurs randomly.
-
-    """
-
-
-@expect(_D213)
-@expect("D416: Section name should end with a colon "
-        "('Args:', not 'Args')")
-def missing_colon_google_style_section():  # noqa: D406, D407
-    """Toggle the gizmo.
-
-    Args
-        note: A random string.
-
-    """
-
-
-@expect("D417: Missing argument descriptions in the docstring "
-        "(argument(s) y are missing descriptions in "
-        "'bar' docstring)", func_name="bar")
-def _test_nested_functions():
-    x = 1
-
-    def bar(y=2):  # noqa: D207, D213, D406, D407
-        """Nested function test for docstrings.
-
-        Will this work when referencing x?
-
-        Args:
-            x: Test something
-that is broken.
-
-        """
-        print(x)
-
-
-@expect(_D213)
-@expect("D417: Missing argument descriptions in the docstring "
-        "(argument(s) y are missing descriptions in "
-        "'test_missing_google_args' docstring)")
-def test_missing_google_args(x=1, y=2, _private=3):  # noqa: D406, D407
-    """Toggle the gizmo.
-
-    Args:
-        x (int): The greatest integer.
-
-    """
-
-
-class TestGoogle:  # noqa: D203
-    """Test class."""
-
-    def test_method(self, test, another_test, _):  # noqa: D213, D407
-        """Test a valid args section.
-
-        Args:
-            test: A parameter.
-            another_test: Another parameter.
-
-        """
-
-    @expect("D417: Missing argument descriptions in the docstring "
-            "(argument(s) test, y, z are missing descriptions in "
-            "'test_missing_args' docstring)", arg_count=5)
-    def test_missing_args(self, test, x, y, z=3, _private_arg=3):  # noqa: D213, D407
-        """Test a valid args section.
-
-        Args:
-            x: Another parameter.
-
-        """
-
-    @classmethod
-    @expect("D417: Missing argument descriptions in the docstring "
-            "(argument(s) test, y, z are missing descriptions in "
-            "'test_missing_args_class_method' docstring)", arg_count=5)
-    def test_missing_args_class_method(cls, test, x, y, _, z=3):  # noqa: D213, D407
-        """Test a valid args section.
-
-        Args:
-            x: Another parameter. The parameter below is missing description.
-            y:
-
-        """
-
-    @staticmethod
-    @expect("D417: Missing argument descriptions in the docstring "
-            "(argument(s) a, y, z are missing descriptions in "
-            "'test_missing_args_static_method' docstring)", arg_count=4)
-    def test_missing_args_static_method(a, x, y, _test, z=3):  # noqa: D213, D407
-        """Test a valid args section.
-
-        Args:
-            x: Another parameter.
-
-        """
-
-    @staticmethod
-    @expect("D417: Missing argument descriptions in the docstring "
-            "(argument(s) a, b are missing descriptions in "
-            "'test_missing_docstring' docstring)", arg_count=2)
-    def test_missing_docstring(a, b):  # noqa: D213, D407
-        """Test a valid args section.
-
-        Args:
-            a:
-
-        """
-
-    @staticmethod
-    def test_hanging_indent(skip, verbose):  # noqa: D213, D407
-        """Do stuff.
-
-        Args:
-            skip (:attr:`.Skip`):
-                Lorem ipsum dolor sit amet, consectetur adipiscing elit.
-                Etiam at tellus a tellus faucibus maximus. Curabitur tellus
-                mauris, semper id vehicula ac, feugiat ut tortor.
-            verbose (bool):
-                If True, print out as much infromation as possible.
-                If False, print out concise "one-liner" information.
-
-        """
-
-
-@expect(_D213)
-@expect("D417: Missing argument descriptions in the docstring "
-        "(argument(s) y are missing descriptions in "
-        "'test_missing_numpy_args' docstring)")
-def test_missing_numpy_args(_private_arg=0, x=1, y=2):  # noqa: D406, D407
-    """Toggle the gizmo.
-
-    Parameters
-    ----------
-    x : int
-        The greatest integer in the history \
-of the entire world.
-
-    """
-
-
-class TestNumpy:  # noqa: D203
-    """Test class."""
-
-    def test_method(self, test, another_test, z, _, x=1, y=2, _private_arg=1):  # noqa: D213, D407
-        """Test a valid args section.
-
-        Some long string with a \
-line continuation.
-
-        Parameters
-        ----------
-        test, another_test
-            Some parameters without type.
-        z : some parameter with a very long type description that requires a \
-line continuation.
-            But no further description.
-        x, y : int
-            Some integer parameters.
-
-        """
-
-    @expect("D417: Missing argument descriptions in the docstring "
-            "(argument(s) test, y, z are missing descriptions in "
-            "'test_missing_args' docstring)", arg_count=5)
-    def test_missing_args(self, test, x, y, z=3, t=1, _private=0):  # noqa: D213, D407
-        """Test a valid args section.
-
-        Parameters
-        ----------
-        x, t : int
-            Some parameters.
-
-
-        """
-
-    @classmethod
-    @expect("D417: Missing argument descriptions in the docstring "
-            "(argument(s) test, y, z are missing descriptions in "
-            "'test_missing_args_class_method' docstring)", arg_count=4)
-    def test_missing_args_class_method(cls, test, x, y, z=3):  # noqa: D213, D407
-        """Test a valid args section.
-
-        Parameters
-        ----------
-        z
-        x
-            Another parameter. The parameters y, test below are
-            missing descriptions. The parameter z above is also missing
-            a description.
-        y
-        test
-
-        """
-
-    @staticmethod
-    @expect("D417: Missing argument descriptions in the docstring "
-            "(argument(s) a, z are missing descriptions in "
-            "'test_missing_args_static_method' docstring)", arg_count=3)
-    def test_missing_args_static_method(a, x, y, z=3, t=1):  # noqa: D213, D407
-        """Test a valid args section.
-
-        Parameters
-        ----------
-        x, y
-            Another parameter.
-        t : int
-            Yet another parameter.
-
-        """
-
-    @staticmethod
-    def test_mixing_numpy_and_google(danger):  # noqa: D213
-        """Repro for #388.
-
-        Parameters
-        ----------
-        danger
-            Zoneeeeee!
-
-        """
-
-
-class TestIncorrectIndent:  # noqa: D203
-    """Test class."""
-
-    @expect("D417: Missing argument descriptions in the docstring "
-            "(argument(s) y are missing descriptions in "
-            "'test_incorrect_indent' docstring)", arg_count=3)
-    def test_incorrect_indent(self, x=1, y=2):  # noqa: D207, D213, D407
-        """Reproducing issue #437.
-
-Testing this incorrectly indented docstring.
-
-        Args:
-            x: Test argument.
-
-        """
--- a/scripts/.flake8
+++ b/scripts/.flake8
@@ -1,19 +0,0 @@
-[flake8]
-exclude =
-    # Defaults
-    .svn,
-    CVS,
-    .bzr,
-    .hg,
-    .git,
-    __pycache__,
-    .tox,
-    .idea,
-    .mypy_cache,
-    .venv,
-    node_modules,
-    # Custom
-    _state_machine.py,
-    test_fstring.py,
-    bad_coding2.py,
-    badsyntax_*.py
--- a/setup.py
+++ b/setup.py
@@ -1,23 +0,0 @@
-import sys
-
-from setuptools import setup
-
-sys.stderr.write(
-    """
-===============================
-Unsupported installation method
-===============================
-ruff no longer supports installation with `python setup.py install`.
-Please use `python -m pip install .` instead.
-"""
-)
-sys.exit(1)
-
-
-# The below code will never execute, however GitHub is particularly
-# picky about where it finds Python packaging metadata.
-# See: https://github.com/github/feedback/discussions/6456
-#
-# To be removed once GitHub catches up.
-
-setup(name="ruff", install_requires=[])
--- a/src/ast.rs
+++ b/src/ast.rs
@@ -1,4 +1,4 @@
-pub mod helpers;
+pub mod checks;
 pub mod operations;
 pub mod relocate;
 pub mod types;
--- a/src/ast/checks.rs
+++ b/src/ast/checks.rs
@@ -0,0 +1,792 @@
+use std::collections::BTreeSet;
+
+use itertools::izip;
+use regex::Regex;
+use rustpython_parser::ast::{
+    Arg, ArgData, Arguments, Cmpop, Constant, Excepthandler, ExcepthandlerKind, Expr, ExprKind,
+    Stmt, StmtKind, Unaryop,
+};
+
+use crate::ast::types::{
+    Binding, BindingKind, CheckLocator, FunctionScope, Range, Scope, ScopeKind,
+};
+use crate::checks::{Check, CheckKind, RejectedCmpop};
+use crate::python::builtins::BUILTINS;
+
+/// Check IfTuple compliance.
+pub fn check_if_tuple(test: &Expr, location: Range) -> Option<Check> {
+    if let ExprKind::Tuple { elts, .. } = &test.node {
+        if !elts.is_empty() {
+            return Some(Check::new(CheckKind::IfTuple, location));
+        }
+    }
+    None
+}
+
+/// Check AssertTuple compliance.
+pub fn check_assert_tuple(test: &Expr, location: Range) -> Option<Check> {
+    if let ExprKind::Tuple { elts, .. } = &test.node {
+        if !elts.is_empty() {
+            return Some(Check::new(CheckKind::AssertTuple, location));
+        }
+    }
+    None
+}
+
+/// Check NotInTest and NotIsTest compliance.
+pub fn check_not_tests(
+    op: &Unaryop,
+    operand: &Expr,
+    check_not_in: bool,
+    check_not_is: bool,
+    locator: &dyn CheckLocator,
+) -> Vec<Check> {
+    let mut checks: Vec<Check> = vec![];
+
+    if matches!(op, Unaryop::Not) {
+        if let ExprKind::Compare { ops, .. } = &operand.node {
+            for op in ops {
+                match op {
+                    Cmpop::In => {
+                        if check_not_in {
+                            checks.push(Check::new(
+                                CheckKind::NotInTest,
+                                locator.locate_check(Range::from_located(operand)),
+                            ));
+                        }
+                    }
+                    Cmpop::Is => {
+                        if check_not_is {
+                            checks.push(Check::new(
+                                CheckKind::NotIsTest,
+                                locator.locate_check(Range::from_located(operand)),
+                            ));
+                        }
+                    }
+                    _ => {}
+                }
+            }
+        }
+    }
+
+    checks
+}
+
+/// Check UnusedVariable compliance.
+pub fn check_unused_variables(
+    scope: &Scope,
+    locator: &dyn CheckLocator,
+    dummy_variable_rgx: &Regex,
+) -> Vec<Check> {
+    let mut checks: Vec<Check> = vec![];
+
+    if matches!(
+        scope.kind,
+        ScopeKind::Function(FunctionScope { uses_locals: true })
+    ) {
+        return checks;
+    }
+
+    for (name, binding) in scope.values.iter() {
+        if binding.used.is_none()
+            && matches!(binding.kind, BindingKind::Assignment)
+            && !dummy_variable_rgx.is_match(name)
+            && name != "__tracebackhide__"
+            && name != "__traceback_info__"
+            && name != "__traceback_supplement__"
+        {
+            checks.push(Check::new(
+                CheckKind::UnusedVariable(name.to_string()),
+                locator.locate_check(binding.range),
+            ));
+        }
+    }
+
+    checks
+}
+
+/// Check DoNotAssignLambda compliance.
+pub fn check_do_not_assign_lambda(value: &Expr, location: Range) -> Option<Check> {
+    if let ExprKind::Lambda { .. } = &value.node {
+        Some(Check::new(CheckKind::DoNotAssignLambda, location))
+    } else {
+        None
+    }
+}
+
+/// Check UselessMetaclassType compliance.
+pub fn check_useless_metaclass_type(
+    targets: &Vec<Expr>,
+    value: &Expr,
+    location: Range,
+) -> Option<Check> {
+    if targets.len() == 1 {
+        if let ExprKind::Name { id, .. } = targets.first().map(|expr| &expr.node).unwrap() {
+            if id == "__metaclass__" {
+                if let ExprKind::Name { id, .. } = &value.node {
+                    if id == "type" {
+                        return Some(Check::new(CheckKind::UselessMetaclassType, location));
+                    }
+                }
+            }
+        }
+    }
+    None
+}
+
+fn is_ambiguous_name(name: &str) -> bool {
+    name == "l" || name == "I" || name == "O"
+}
+
+/// Check AmbiguousVariableName compliance.
+pub fn check_ambiguous_variable_name(name: &str, location: Range) -> Option<Check> {
+    if is_ambiguous_name(name) {
+        Some(Check::new(
+            CheckKind::AmbiguousVariableName(name.to_string()),
+            location,
+        ))
+    } else {
+        None
+    }
+}
+
+/// Check AmbiguousClassName compliance.
+pub fn check_ambiguous_class_name(name: &str, location: Range) -> Option<Check> {
+    if is_ambiguous_name(name) {
+        Some(Check::new(
+            CheckKind::AmbiguousClassName(name.to_string()),
+            location,
+        ))
+    } else {
+        None
+    }
+}
+
+/// Check AmbiguousFunctionName compliance.
+pub fn check_ambiguous_function_name(name: &str, location: Range) -> Option<Check> {
+    if is_ambiguous_name(name) {
+        Some(Check::new(
+            CheckKind::AmbiguousFunctionName(name.to_string()),
+            location,
+        ))
+    } else {
+        None
+    }
+}
+
+/// Check UselessObjectInheritance compliance.
+pub fn check_useless_object_inheritance(
+    name: &str,
+    bases: &[Expr],
+    scope: &Scope,
+) -> Option<Check> {
+    for expr in bases {
+        if let ExprKind::Name { id, .. } = &expr.node {
+            if id == "object" {
+                match scope.values.get(id) {
+                    None
+                    | Some(Binding {
+                        kind: BindingKind::Builtin,
+                        ..
+                    }) => {
+                        return Some(Check::new(
+                            CheckKind::UselessObjectInheritance(name.to_string()),
+                            Range::from_located(expr),
+                        ));
+                    }
+                    _ => {}
+                }
+            }
+        }
+    }
+
+    None
+}
+
+/// Check DefaultExceptNotLast compliance.
+pub fn check_default_except_not_last(handlers: &Vec<Excepthandler>) -> Option<Check> {
+    for (idx, handler) in handlers.iter().enumerate() {
+        let ExcepthandlerKind::ExceptHandler { type_, .. } = &handler.node;
+        if type_.is_none() && idx < handlers.len() - 1 {
+            return Some(Check::new(
+                CheckKind::DefaultExceptNotLast,
+                Range::from_located(handler),
+            ));
+        }
+    }
+
+    None
+}
+
+/// Check RaiseNotImplemented compliance.
+pub fn check_raise_not_implemented(expr: &Expr) -> Option<Check> {
+    match &expr.node {
+        ExprKind::Call { func, .. } => {
+            if let ExprKind::Name { id, .. } = &func.node {
+                if id == "NotImplemented" {
+                    return Some(Check::new(
+                        CheckKind::RaiseNotImplemented,
+                        Range::from_located(expr),
+                    ));
+                }
+            }
+        }
+        ExprKind::Name { id, .. } => {
+            if id == "NotImplemented" {
+                return Some(Check::new(
+                    CheckKind::RaiseNotImplemented,
+                    Range::from_located(expr),
+                ));
+            }
+        }
+        _ => {}
+    }
+
+    None
+}
+
+/// Check DuplicateArgumentName compliance.
+pub fn check_duplicate_arguments(arguments: &Arguments) -> Vec<Check> {
+    let mut checks: Vec<Check> = vec![];
+
+    // Collect all the arguments into a single vector.
+    let mut all_arguments: Vec<&Arg> = arguments
+        .args
+        .iter()
+        .chain(arguments.posonlyargs.iter())
+        .chain(arguments.kwonlyargs.iter())
+        .collect();
+    if let Some(arg) = &arguments.vararg {
+        all_arguments.push(arg);
+    }
+    if let Some(arg) = &arguments.kwarg {
+        all_arguments.push(arg);
+    }
+
+    // Search for duplicates.
+    let mut idents: BTreeSet<&str> = BTreeSet::new();
+    for arg in all_arguments {
+        let ident = &arg.node.arg;
+        if idents.contains(ident.as_str()) {
+            checks.push(Check::new(
+                CheckKind::DuplicateArgumentName,
+                Range::from_located(arg),
+            ));
+        }
+        idents.insert(ident);
+    }
+
+    checks
+}
+
+/// Check AssertEquals compliance.
+pub fn check_assert_equals(expr: &Expr) -> Option<Check> {
+    if let ExprKind::Attribute { value, attr, .. } = &expr.node {
+        if attr == "assertEquals" {
+            if let ExprKind::Name { id, .. } = &value.node {
+                if id == "self" {
+                    return Some(Check::new(
+                        CheckKind::NoAssertEquals,
+                        Range::from_located(expr),
+                    ));
+                }
+            }
+        }
+    }
+    None
+}
+
+#[derive(Debug, PartialEq)]
+enum DictionaryKey<'a> {
+    Constant(&'a Constant),
+    Variable(&'a String),
+}
+
+fn convert_to_value(expr: &Expr) -> Option<DictionaryKey> {
+    match &expr.node {
+        ExprKind::Constant { value, .. } => Some(DictionaryKey::Constant(value)),
+        ExprKind::Name { id, .. } => Some(DictionaryKey::Variable(id)),
+        _ => None,
+    }
+}
+
+/// Check MultiValueRepeatedKeyLiteral and MultiValueRepeatedKeyVariable compliance.
+pub fn check_repeated_keys(
+    keys: &Vec<Expr>,
+    check_repeated_literals: bool,
+    check_repeated_variables: bool,
+    locator: &dyn CheckLocator,
+) -> Vec<Check> {
+    let mut checks: Vec<Check> = vec![];
+
+    let num_keys = keys.len();
+    for i in 0..num_keys {
+        let k1 = &keys[i];
+        let v1 = convert_to_value(k1);
+        for k2 in keys.iter().take(num_keys).skip(i + 1) {
+            let v2 = convert_to_value(k2);
+            match (&v1, &v2) {
+                (Some(DictionaryKey::Constant(v1)), Some(DictionaryKey::Constant(v2))) => {
+                    if check_repeated_literals && v1 == v2 {
+                        checks.push(Check::new(
+                            CheckKind::MultiValueRepeatedKeyLiteral,
+                            locator.locate_check(Range::from_located(k2)),
+                        ))
+                    }
+                }
+                (Some(DictionaryKey::Variable(v1)), Some(DictionaryKey::Variable(v2))) => {
+                    if check_repeated_variables && v1 == v2 {
+                        checks.push(Check::new(
+                            CheckKind::MultiValueRepeatedKeyVariable((*v2).to_string()),
+                            locator.locate_check(Range::from_located(k2)),
+                        ))
+                    }
+                }
+                _ => {}
+            }
+        }
+    }
+
+    checks
+}
+
+/// Check TrueFalseComparison and NoneComparison compliance.
+pub fn check_literal_comparisons(
+    left: &Expr,
+    ops: &Vec<Cmpop>,
+    comparators: &Vec<Expr>,
+    check_none_comparisons: bool,
+    check_true_false_comparisons: bool,
+    locator: &dyn CheckLocator,
+) -> Vec<Check> {
+    let mut checks: Vec<Check> = vec![];
+
+    let op = ops.first().unwrap();
+    let comparator = left;
+
+    // Check `left`.
+    if check_none_comparisons
+        && matches!(
+            comparator.node,
+            ExprKind::Constant {
+                value: Constant::None,
+                kind: None
+            }
+        )
+    {
+        if matches!(op, Cmpop::Eq) {
+            checks.push(Check::new(
+                CheckKind::NoneComparison(RejectedCmpop::Eq),
+                locator.locate_check(Range::from_located(comparator)),
+            ));
+        }
+        if matches!(op, Cmpop::NotEq) {
+            checks.push(Check::new(
+                CheckKind::NoneComparison(RejectedCmpop::NotEq),
+                locator.locate_check(Range::from_located(comparator)),
+            ));
+        }
+    }
+
+    if check_true_false_comparisons {
+        if let ExprKind::Constant {
+            value: Constant::Bool(value),
+            kind: None,
+        } = comparator.node
+        {
+            if matches!(op, Cmpop::Eq) {
+                checks.push(Check::new(
+                    CheckKind::TrueFalseComparison(value, RejectedCmpop::Eq),
+                    locator.locate_check(Range::from_located(comparator)),
+                ));
+            }
+            if matches!(op, Cmpop::NotEq) {
+                checks.push(Check::new(
+                    CheckKind::TrueFalseComparison(value, RejectedCmpop::NotEq),
+                    locator.locate_check(Range::from_located(comparator)),
+                ));
+            }
+        }
+    }
+
+    // Check each comparator in order.
+    for (op, comparator) in izip!(ops, comparators) {
+        if check_none_comparisons
+            && matches!(
+                comparator.node,
+                ExprKind::Constant {
+                    value: Constant::None,
+                    kind: None
+                }
+            )
+        {
+            if matches!(op, Cmpop::Eq) {
+                checks.push(Check::new(
+                    CheckKind::NoneComparison(RejectedCmpop::Eq),
+                    locator.locate_check(Range::from_located(comparator)),
+                ));
+            }
+            if matches!(op, Cmpop::NotEq) {
+                checks.push(Check::new(
+                    CheckKind::NoneComparison(RejectedCmpop::NotEq),
+                    locator.locate_check(Range::from_located(comparator)),
+                ));
+            }
+        }
+
+        if check_true_false_comparisons {
+            if let ExprKind::Constant {
+                value: Constant::Bool(value),
+                kind: None,
+            } = comparator.node
+            {
+                if matches!(op, Cmpop::Eq) {
+                    checks.push(Check::new(
+                        CheckKind::TrueFalseComparison(value, RejectedCmpop::Eq),
+                        locator.locate_check(Range::from_located(comparator)),
+                    ));
+                }
+                if matches!(op, Cmpop::NotEq) {
+                    checks.push(Check::new(
+                        CheckKind::TrueFalseComparison(value, RejectedCmpop::NotEq),
+                        locator.locate_check(Range::from_located(comparator)),
+                    ));
+                }
+            }
+        }
+    }
+
+    checks
+}
+
+fn is_constant(expr: &Expr) -> bool {
+    match &expr.node {
+        ExprKind::Constant { .. } => true,
+        ExprKind::Tuple { elts, .. } => elts.iter().all(is_constant),
+        _ => false,
+    }
+}
+
+fn is_singleton(expr: &Expr) -> bool {
+    matches!(
+        expr.node,
+        ExprKind::Constant {
+            value: Constant::None | Constant::Bool(_) | Constant::Ellipsis,
+            ..
+        }
+    )
+}
+
+fn is_constant_non_singleton(expr: &Expr) -> bool {
+    is_constant(expr) && !is_singleton(expr)
+}
+
+/// Check IsLiteral compliance.
+pub fn check_is_literal(
+    left: &Expr,
+    ops: &Vec<Cmpop>,
+    comparators: &Vec<Expr>,
+    location: Range,
+) -> Vec<Check> {
+    let mut checks: Vec<Check> = vec![];
+
+    let mut left = left;
+    for (op, right) in izip!(ops, comparators) {
+        if matches!(op, Cmpop::Is | Cmpop::IsNot)
+            && (is_constant_non_singleton(left) || is_constant_non_singleton(right))
+        {
+            checks.push(Check::new(CheckKind::IsLiteral, location));
+        }
+        left = right;
+    }
+
+    checks
+}
+
+/// Check TypeComparison compliance.
+pub fn check_type_comparison(
+    ops: &Vec<Cmpop>,
+    comparators: &Vec<Expr>,
+    location: Range,
+) -> Vec<Check> {
+    let mut checks: Vec<Check> = vec![];
+
+    for (op, right) in izip!(ops, comparators) {
+        if matches!(op, Cmpop::Is | Cmpop::IsNot | Cmpop::Eq | Cmpop::NotEq) {
+            match &right.node {
+                ExprKind::Call { func, args, .. } => {
+                    if let ExprKind::Name { id, .. } = &func.node {
+                        // Ex) type(False)
+                        if id == "type" {
+                            if let Some(arg) = args.first() {
+                                // Allow comparison for types which are not obvious.
+                                if !matches!(arg.node, ExprKind::Name { .. }) {
+                                    checks.push(Check::new(CheckKind::TypeComparison, location));
+                                }
+                            }
+                        }
+                    }
+                }
+                ExprKind::Attribute { value, .. } => {
+                    if let ExprKind::Name { id, .. } = &value.node {
+                        // Ex) types.IntType
+                        if id == "types" {
+                            checks.push(Check::new(CheckKind::TypeComparison, location));
+                        }
+                    }
+                }
+                _ => {}
+            }
+        }
+    }
+
+    checks
+}
+
+/// Check TwoStarredExpressions and TooManyExpressionsInStarredAssignment compliance.
+pub fn check_starred_expressions(
+    elts: &[Expr],
+    check_too_many_expressions: bool,
+    check_two_starred_expressions: bool,
+    location: Range,
+) -> Option<Check> {
+    let mut has_starred: bool = false;
+    let mut starred_index: Option<usize> = None;
+    for (index, elt) in elts.iter().enumerate() {
+        if matches!(elt.node, ExprKind::Starred { .. }) {
+            if has_starred && check_two_starred_expressions {
+                return Some(Check::new(CheckKind::TwoStarredExpressions, location));
+            }
+            has_starred = true;
+            starred_index = Some(index);
+        }
+    }
+
+    if check_too_many_expressions {
+        if let Some(starred_index) = starred_index {
+            if starred_index >= 1 << 8 || elts.len() - starred_index > 1 << 24 {
+                return Some(Check::new(CheckKind::ExpressionsInStarAssignment, location));
+            }
+        }
+    }
+
+    None
+}
+
+/// Check BreakOutsideLoop compliance.
+pub fn check_break_outside_loop(
+    stmt: &Stmt,
+    parents: &[&Stmt],
+    parent_stack: &[usize],
+    locator: &dyn CheckLocator,
+) -> Option<Check> {
+    let mut allowed: bool = false;
+    let mut parent = stmt;
+    for index in parent_stack.iter().rev() {
+        let child = parent;
+        parent = parents[*index];
+        match &parent.node {
+            StmtKind::For { orelse, .. }
+            | StmtKind::AsyncFor { orelse, .. }
+            | StmtKind::While { orelse, .. } => {
+                if !orelse.contains(child) {
+                    allowed = true;
+                    break;
+                }
+            }
+
+            StmtKind::FunctionDef { .. }
+            | StmtKind::AsyncFunctionDef { .. }
+            | StmtKind::ClassDef { .. } => {
+                break;
+            }
+            _ => {}
+        }
+    }
+
+    if !allowed {
+        Some(Check::new(
+            CheckKind::BreakOutsideLoop,
+            locator.locate_check(Range::from_located(stmt)),
+        ))
+    } else {
+        None
+    }
+}
+
+/// Check ContinueOutsideLoop compliance.
+pub fn check_continue_outside_loop(
+    stmt: &Stmt,
+    parents: &[&Stmt],
+    parent_stack: &[usize],
+    locator: &dyn CheckLocator,
+) -> Option<Check> {
+    let mut allowed: bool = false;
+    let mut parent = stmt;
+    for index in parent_stack.iter().rev() {
+        let child = parent;
+        parent = parents[*index];
+        match &parent.node {
+            StmtKind::For { orelse, .. }
+            | StmtKind::AsyncFor { orelse, .. }
+            | StmtKind::While { orelse, .. } => {
+                if !orelse.contains(child) {
+                    allowed = true;
+                    break;
+                }
+            }
+
+            StmtKind::FunctionDef { .. }
+            | StmtKind::AsyncFunctionDef { .. }
+            | StmtKind::ClassDef { .. } => {
+                break;
+            }
+            _ => {}
+        }
+    }
+
+    if !allowed {
+        Some(Check::new(
+            CheckKind::ContinueOutsideLoop,
+            locator.locate_check(Range::from_located(stmt)),
+        ))
+    } else {
+        None
+    }
+}
+
+// flake8-builtins
+pub enum ShadowingType {
+    Variable,
+    Argument,
+    Attribute,
+}
+
+/// Check builtin name shadowing
+pub fn check_builtin_shadowing(
+    name: &str,
+    location: Range,
+    node_type: ShadowingType,
+) -> Option<Check> {
+    if BUILTINS.contains(&name) {
+        Some(Check::new(
+            match node_type {
+                ShadowingType::Variable => CheckKind::BuiltinVariableShadowing(name.to_string()),
+                ShadowingType::Argument => CheckKind::BuiltinArgumentShadowing(name.to_string()),
+                ShadowingType::Attribute => CheckKind::BuiltinAttributeShadowing(name.to_string()),
+            },
+            location,
+        ))
+    } else {
+        None
+    }
+}
+
+/// Returns `true` if a call is an argumented `super` invocation.
+pub fn is_super_call_with_arguments(func: &Expr, args: &Vec<Expr>) -> bool {
+    // Check: is this a `super` call?
+    if let ExprKind::Name { id, .. } = &func.node {
+        id == "super" && !args.is_empty()
+    } else {
+        false
+    }
+}
+
+// flake8-super
+/// Check that `super()` has no args
+pub fn check_super_args(
+    scope: &Scope,
+    parents: &[&Stmt],
+    expr: &Expr,
+    func: &Expr,
+    args: &Vec<Expr>,
+) -> Option<Check> {
+    if !is_super_call_with_arguments(func, args) {
+        return None;
+    }
+
+    // Check: are we in a Function scope?
+    if !matches!(scope.kind, ScopeKind::Function { .. }) {
+        return None;
+    }
+
+    let mut parents = parents.iter().rev();
+
+    // For a `super` invocation to be unnecessary, the first argument needs to match the enclosing
+    // class, and the second argument needs to match the first argument to the enclosing function.
+    if let [first_arg, second_arg] = args.as_slice() {
+        // Find the enclosing function definition (if any).
+        if let Some(StmtKind::FunctionDef {
+            args: parent_args, ..
+        }) = parents
+            .find(|stmt| matches!(stmt.node, StmtKind::FunctionDef { .. }))
+            .map(|stmt| &stmt.node)
+        {
+            // Extract the name of the first argument to the enclosing function.
+            if let Some(ArgData {
+                arg: parent_arg, ..
+            }) = parent_args.args.first().map(|expr| &expr.node)
+            {
+                // Find the enclosing class definition (if any).
+                if let Some(StmtKind::ClassDef {
+                    name: parent_name, ..
+                }) = parents
+                    .find(|stmt| matches!(stmt.node, StmtKind::ClassDef { .. }))
+                    .map(|stmt| &stmt.node)
+                {
+                    if let (
+                        ExprKind::Name {
+                            id: first_arg_id, ..
+                        },
+                        ExprKind::Name {
+                            id: second_arg_id, ..
+                        },
+                    ) = (&first_arg.node, &second_arg.node)
+                    {
+                        if first_arg_id == parent_name && second_arg_id == parent_arg {
+                            return Some(Check::new(
+                                CheckKind::SuperCallWithParameters,
+                                Range::from_located(expr),
+                            ));
+                        }
+                    }
+                }
+            }
+        }
+    }
+
+    None
+}
+
+// flake8-print
+/// Check whether a function call is a `print` or `pprint` invocation
+pub fn check_print_call(
+    expr: &Expr,
+    func: &Expr,
+    check_print: bool,
+    check_pprint: bool,
+) -> Option<Check> {
+    if let ExprKind::Name { id, .. } = &func.node {
+        if check_print && id == "print" {
+            return Some(Check::new(CheckKind::PrintFound, Range::from_located(expr)));
+        } else if check_pprint && id == "pprint" {
+            return Some(Check::new(
+                CheckKind::PPrintFound,
+                Range::from_located(expr),
+            ));
+        }
+    }
+
+    if let ExprKind::Attribute { value, attr, .. } = &func.node {
+        if let ExprKind::Name { id, .. } = &value.node {
+            if check_pprint && id == "pprint" && attr == "pprint" {
+                return Some(Check::new(
+                    CheckKind::PPrintFound,
+                    Range::from_located(expr),
+                ));
+            }
+        }
+    }
+
+    None
+}
--- a/src/ast/helpers.rs
+++ b/src/ast/helpers.rs
@@ -1,145 +0,0 @@
-use once_cell::sync::Lazy;
-use regex::Regex;
-use rustpython_ast::{Excepthandler, ExcepthandlerKind, Expr, ExprKind, Location, StmtKind};
-
-use crate::python::typing;
-
-fn compose_call_path_inner<'a>(expr: &'a Expr, parts: &mut Vec<&'a str>) {
-    match &expr.node {
-        ExprKind::Call { func, .. } => {
-            compose_call_path_inner(func, parts);
-        }
-        ExprKind::Attribute { value, attr, .. } => {
-            compose_call_path_inner(value, parts);
-            parts.push(attr);
-        }
-        ExprKind::Name { id, .. } => {
-            parts.push(id);
-        }
-        _ => {}
-    }
-}
-
-pub fn compose_call_path(expr: &Expr) -> Option<String> {
-    let mut segments = vec![];
-    compose_call_path_inner(expr, &mut segments);
-    if segments.is_empty() {
-        None
-    } else {
-        Some(segments.join("."))
-    }
-}
-
-pub fn match_name_or_attr(expr: &Expr, target: &str) -> bool {
-    match &expr.node {
-        ExprKind::Attribute { attr, .. } => target == attr,
-        ExprKind::Name { id, .. } => target == id,
-        _ => false,
-    }
-}
-
-pub enum SubscriptKind {
-    AnnotatedSubscript,
-    PEP593AnnotatedSubscript,
-}
-
-pub fn match_annotated_subscript(expr: &Expr) -> Option<SubscriptKind> {
-    match &expr.node {
-        ExprKind::Attribute { attr, .. } => {
-            if typing::is_annotated_subscript(attr) {
-                Some(SubscriptKind::AnnotatedSubscript)
-            } else if typing::is_pep593_annotated_subscript(attr) {
-                Some(SubscriptKind::PEP593AnnotatedSubscript)
-            } else {
-                None
-            }
-        }
-        ExprKind::Name { id, .. } => {
-            if typing::is_annotated_subscript(id) {
-                Some(SubscriptKind::AnnotatedSubscript)
-            } else if typing::is_pep593_annotated_subscript(id) {
-                Some(SubscriptKind::PEP593AnnotatedSubscript)
-            } else {
-                None
-            }
-        }
-        _ => None,
-    }
-}
-
-static DUNDER_REGEX: Lazy<Regex> = Lazy::new(|| Regex::new(r"__[^\s]+__").unwrap());
-
-pub fn is_assignment_to_a_dunder(node: &StmtKind) -> bool {
-    // Check whether it's an assignment to a dunder, with or without a type annotation.
-    // This is what pycodestyle (as of 2.9.1) does.
-    match node {
-        StmtKind::Assign {
-            targets,
-            value: _,
-            type_comment: _,
-        } => {
-            if targets.len() != 1 {
-                return false;
-            }
-            match &targets[0].node {
-                ExprKind::Name { id, ctx: _ } => DUNDER_REGEX.is_match(id),
-                _ => false,
-            }
-        }
-        StmtKind::AnnAssign {
-            target,
-            annotation: _,
-            value: _,
-            simple: _,
-        } => match &target.node {
-            ExprKind::Name { id, ctx: _ } => DUNDER_REGEX.is_match(id),
-            _ => false,
-        },
-        _ => false,
-    }
-}
-
-/// Extract the names of all handled exceptions.
-pub fn extract_handler_names(handlers: &[Excepthandler]) -> Vec<String> {
-    let mut handler_names = vec![];
-    for handler in handlers {
-        match &handler.node {
-            ExcepthandlerKind::ExceptHandler { type_, .. } => {
-                if let Some(type_) = type_ {
-                    if let ExprKind::Tuple { elts, .. } = &type_.node {
-                        for type_ in elts {
-                            if let Some(name) = compose_call_path(type_) {
-                                handler_names.push(name);
-                            }
-                        }
-                    } else if let Some(name) = compose_call_path(type_) {
-                        handler_names.push(name);
-                    }
-                }
-            }
-        }
-    }
-    handler_names
-}
-
-/// Returns `true` if a call is an argumented `super` invocation.
-pub fn is_super_call_with_arguments(func: &Expr, args: &[Expr]) -> bool {
-    // Check: is this a `super` call?
-    if let ExprKind::Name { id, .. } = &func.node {
-        id == "super" && !args.is_empty()
-    } else {
-        false
-    }
-}
-
-/// Convert a location within a file (relative to `base`) to an absolute position.
-pub fn to_absolute(relative: &Location, base: &Location) -> Location {
-    if relative.row() == 1 {
-        Location::new(
-            relative.row() + base.row() - 1,
-            relative.column() + base.column() - 1,
-        )
-    } else {
-        Location::new(relative.row() + base.row() - 1, relative.column())
-    }
-}
--- a/src/ast/operations.rs
+++ b/src/ast/operations.rs
@@ -121,7 +121,7 @@ pub fn is_unpacking_assignment(stmt: &Stmt) -> bool {
 /// Struct used to efficiently slice source code at (row, column) Locations.
 pub struct SourceCodeLocator<'a> {
    content: &'a str,
-    offsets: Vec<Vec<usize>>,
+    offsets: Vec<usize>,
    initialized: bool,
 }

@@ -134,52 +134,32 @@ impl<'a> SourceCodeLocator<'a> {
        }
    }

-    fn init(&mut self) {
+    pub fn slice_source_code_at(&mut self, location: &Location) -> &'a str {
        if !self.initialized {
            let mut offset = 0;
-            for line in self.content.lines() {
-                let mut newline = 0;
-                let mut line_offsets: Vec<usize> = vec![];
-                for (i, _char) in line.char_indices() {
-                    line_offsets.push(offset + i);
-                    newline = i + 1;
-                }
-                line_offsets.push(offset + newline);
-                self.offsets.push(line_offsets);
-                offset += newline + 1;
+            for i in self.content.lines() {
+                self.offsets.push(offset);
+                offset += i.len();
+                offset += 1;
            }
-            self.offsets.push(vec![offset]);
            self.initialized = true;
        }
-    }
-
-    pub fn slice_source_code_at(&mut self, location: &Location) -> &'a str {
-        self.init();
-        let offset = self.offsets[location.row() - 1][location.column() - 1];
+        let offset = self.offsets[location.row() - 1] + location.column() - 1;
        &self.content[offset..]
    }

    pub fn slice_source_code_range(&mut self, range: &Range) -> &'a str {
-        self.init();
-        let start = self.offsets[range.location.row() - 1][range.location.column() - 1];
-        let end = self.offsets[range.end_location.row() - 1][range.end_location.column() - 1];
+        if !self.initialized {
+            let mut offset = 0;
+            for i in self.content.lines() {
+                self.offsets.push(offset);
+                offset += i.len();
+                offset += 1;
+            }
+            self.initialized = true;
+        }
+        let start = self.offsets[range.location.row() - 1] + range.location.column() - 1;
+        let end = self.offsets[range.end_location.row() - 1] + range.end_location.column() - 1;
        &self.content[start..end]
    }
-
-    pub fn partition_source_code_at(
-        &mut self,
-        outer: &Range,
-        inner: &Range,
-    ) -> (&'a str, &'a str, &'a str) {
-        self.init();
-        let outer_start = self.offsets[outer.location.row() - 1][outer.location.column() - 1];
-        let outer_end = self.offsets[outer.end_location.row() - 1][outer.end_location.column() - 1];
-        let inner_start = self.offsets[inner.location.row() - 1][inner.location.column() - 1];
-        let inner_end = self.offsets[inner.end_location.row() - 1][inner.end_location.column() - 1];
-        (
-            &self.content[outer_start..inner_start],
-            &self.content[inner_start..inner_end],
-            &self.content[inner_end..outer_end],
-        )
-    }
 }
--- a/src/ast/relocate.rs
+++ b/src/ast/relocate.rs
@@ -4,14 +4,14 @@ use crate::ast::types::Range;

 fn relocate_keyword(keyword: &mut Keyword, location: Range) {
    keyword.location = location.location;
-    keyword.end_location = Some(location.end_location);
+    keyword.end_location = location.end_location;
    relocate_expr(&mut keyword.node.value, location);
 }

 /// Change an expression's location (recursively) to match a desired, fixed location.
 pub fn relocate_expr(expr: &mut Expr, location: Range) {
    expr.location = location.location;
-    expr.end_location = Some(location.end_location);
+    expr.end_location = location.end_location;
    match &mut expr.node {
        ExprKind::BoolOp { values, .. } => {
            for expr in values {
--- a/src/ast/types.rs
+++ b/src/ast/types.rs
@@ -18,9 +18,7 @@ impl Range {
    pub fn from_located<T>(located: &Located<T>) -> Self {
        Range {
            location: located.location,
-            end_location: located
-                .end_location
-                .expect("AST nodes should have end_location."),
+            end_location: located.end_location,
        }
    }
 }
@@ -76,9 +74,9 @@ pub enum BindingKind {
    Export(Vec<String>),
    FutureImportation,
    StarImportation,
-    Importation(String, String, BindingContext),
-    FromImportation(String, String, BindingContext),
-    SubmoduleImportation(String, String, BindingContext),
+    Importation(String, BindingContext),
+    FromImportation(String, BindingContext),
+    SubmoduleImportation(String, BindingContext),
 }

 #[derive(Clone, Debug)]
--- a/src/ast/visitor.rs
+++ b/src/ast/visitor.rs
@@ -4,8 +4,6 @@ use rustpython_parser::ast::{
    PatternKind, Stmt, StmtKind, Unaryop, Withitem,
 };

-use crate::ast::helpers::match_name_or_attr;
-
 pub trait Visitor<'a> {
    fn visit_stmt(&mut self, stmt: &'a Stmt) {
        walk_stmt(self, stmt);
@@ -150,11 +148,7 @@ pub fn walk_stmt<'a, V: Visitor<'a> + ?Sized>(visitor: &mut V, stmt: &'a Stmt) {
        } => {
            visitor.visit_annotation(annotation);
            if let Some(expr) = value {
-                if match_name_or_attr(annotation, "TypeAlias") {
-                    visitor.visit_annotation(expr);
-                } else {
-                    visitor.visit_expr(expr);
-                }
+                visitor.visit_expr(expr);
            }
            visitor.visit_expr(target);
        }
--- a/src/autofix.rs
+++ b/src/autofix.rs
@@ -0,0 +1,2 @@
+pub mod fixer;
+pub mod fixes;
--- a/src/autofix/fixer.rs
+++ b/src/autofix/fixer.rs
@@ -1,10 +1,11 @@
-use crate::autofix::Fix;
-use crate::autofix::Patch;
+use std::fs;
+use std::path::Path;
+
+use anyhow::Result;
 use itertools::Itertools;
 use rustpython_parser::ast::Location;
-use std::collections::BTreeSet;

-use crate::checks::Check;
+use crate::checks::{Check, Fix};

 #[derive(Hash)]
 pub enum Mode {
@@ -23,59 +24,51 @@ impl From<bool> for Mode {
 }

 /// Auto-fix errors in a file, and write the fixed source code to disk.
-pub fn fix_file(checks: &mut [Check], contents: &str) -> Option<String> {
+pub fn fix_file(checks: &mut [Check], contents: &str, path: &Path) -> Result<()> {
    if checks.iter().all(|check| check.fix.is_none()) {
-        return None;
+        return Ok(());
    }

-    Some(apply_fixes(
+    let output = apply_fixes(
        checks.iter_mut().filter_map(|check| check.fix.as_mut()),
        contents,
-    ))
+    );
+
+    fs::write(path, output).map_err(|e| e.into())
 }

 /// Apply a series of fixes.
 fn apply_fixes<'a>(fixes: impl Iterator<Item = &'a mut Fix>, contents: &str) -> String {
    let lines: Vec<&str> = contents.lines().collect();

-    let mut output: String = Default::default();
-    let mut last_pos: Location = Default::default();
-    let mut applied: BTreeSet<&Patch> = Default::default();
-
-    for fix in fixes.sorted_by_key(|fix| fix.patch.location) {
-        // If we already applied an identical fix as part of another correction, skip any
-        // re-application.
-        if applied.contains(&fix.patch) {
-            fix.applied = true;
-            continue;
-        }
+    let mut output = "".to_string();
+    let mut last_pos: Location = Location::new(0, 0);

+    for fix in fixes.sorted_by_key(|fix| fix.location) {
        // Best-effort approach: if this fix overlaps with a fix we've already applied, skip it.
-        if last_pos > fix.patch.location {
+        if last_pos > fix.location {
            continue;
        }

-        if fix.patch.location.row() > last_pos.row() {
+        if fix.location.row() > last_pos.row() {
            if last_pos.row() > 0 || last_pos.column() > 0 {
                output.push_str(&lines[last_pos.row() - 1][last_pos.column() - 1..]);
                output.push('\n');
            }
-            for line in &lines[last_pos.row()..fix.patch.location.row() - 1] {
+            for line in &lines[last_pos.row()..fix.location.row() - 1] {
                output.push_str(line);
                output.push('\n');
            }
-            output
-                .push_str(&lines[fix.patch.location.row() - 1][..fix.patch.location.column() - 1]);
-            output.push_str(&fix.patch.content);
+            output.push_str(&lines[fix.location.row() - 1][..fix.location.column() - 1]);
+            output.push_str(&fix.content);
        } else {
            output.push_str(
-                &lines[last_pos.row() - 1][last_pos.column() - 1..fix.patch.location.column() - 1],
+                &lines[last_pos.row() - 1][last_pos.column() - 1..fix.location.column() - 1],
            );
-            output.push_str(&fix.patch.content);
+            output.push_str(&fix.content);
        }
-        last_pos = fix.patch.end_location;

-        applied.insert(&fix.patch);
+        last_pos = fix.end_location;
        fix.applied = true;
    }

@@ -102,8 +95,7 @@ mod tests {
    use rustpython_parser::ast::Location;

    use crate::autofix::fixer::apply_fixes;
-    use crate::autofix::Fix;
-    use crate::autofix::Patch;
+    use crate::checks::Fix;

    #[test]
    fn empty_file() -> Result<()> {
@@ -119,11 +111,9 @@ mod tests {
    #[test]
    fn apply_single_replacement() -> Result<()> {
        let mut fixes = vec![Fix {
-            patch: Patch {
-                content: "Bar".to_string(),
-                location: Location::new(1, 9),
-                end_location: Location::new(1, 15),
-            },
+            content: "Bar".to_string(),
+            location: Location::new(1, 9),
+            end_location: Location::new(1, 15),
            applied: false,
        }];
        let actual = apply_fixes(
@@ -145,11 +135,9 @@ mod tests {
    #[test]
    fn apply_single_removal() -> Result<()> {
        let mut fixes = vec![Fix {
-            patch: Patch {
-                content: "".to_string(),
-                location: Location::new(1, 8),
-                end_location: Location::new(1, 16),
-            },
+            content: "".to_string(),
+            location: Location::new(1, 8),
+            end_location: Location::new(1, 16),
            applied: false,
        }];
        let actual = apply_fixes(
@@ -172,19 +160,15 @@ mod tests {
    fn apply_double_removal() -> Result<()> {
        let mut fixes = vec![
            Fix {
-                patch: Patch {
-                    content: "".to_string(),
-                    location: Location::new(1, 8),
-                    end_location: Location::new(1, 17),
-                },
+                content: "".to_string(),
+                location: Location::new(1, 8),
+                end_location: Location::new(1, 17),
                applied: false,
            },
            Fix {
-                patch: Patch {
-                    content: "".to_string(),
-                    location: Location::new(1, 17),
-                    end_location: Location::new(1, 24),
-                },
+                content: "".to_string(),
+                location: Location::new(1, 17),
+                end_location: Location::new(1, 24),
                applied: false,
            },
        ];
@@ -208,19 +192,15 @@ mod tests {
    fn ignore_overlapping_fixes() -> Result<()> {
        let mut fixes = vec![
            Fix {
-                patch: Patch {
-                    content: "".to_string(),
-                    location: Location::new(1, 8),
-                    end_location: Location::new(1, 16),
-                },
+                content: "".to_string(),
+                location: Location::new(1, 8),
+                end_location: Location::new(1, 16),
                applied: false,
            },
            Fix {
-                patch: Patch {
-                    content: "ignored".to_string(),
-                    location: Location::new(1, 10),
-                    end_location: Location::new(1, 12),
-                },
+                content: "ignored".to_string(),
+                location: Location::new(1, 10),
+                end_location: Location::new(1, 12),
                applied: false,
            },
        ];
--- a/src/autofix/fixes.rs
+++ b/src/autofix/fixes.rs
@@ -0,0 +1,392 @@
+use anyhow::Result;
+use itertools::Itertools;
+use libcst_native::ImportNames::Aliases;
+use libcst_native::NameOrAttribute::N;
+use libcst_native::{Codegen, Expression, SmallStatement, Statement};
+use rustpython_parser::ast::{ExcepthandlerKind, Expr, Keyword, Location, Stmt, StmtKind};
+use rustpython_parser::lexer;
+use rustpython_parser::token::Tok;
+
+use crate::ast::operations::SourceCodeLocator;
+use crate::ast::types::Range;
+use crate::checks::Fix;
+
+/// Convert a location within a file (relative to `base`) to an absolute position.
+fn to_absolute(relative: &Location, base: &Location) -> Location {
+    if relative.row() == 1 {
+        Location::new(
+            relative.row() + base.row() - 1,
+            relative.column() + base.column() - 1,
+        )
+    } else {
+        Location::new(relative.row() + base.row() - 1, relative.column())
+    }
+}
+
+/// Generate a fix to remove a base from a ClassDef statement.
+pub fn remove_class_def_base(
+    locator: &mut SourceCodeLocator,
+    stmt_at: &Location,
+    expr_at: Location,
+    bases: &[Expr],
+    keywords: &[Keyword],
+) -> Option<Fix> {
+    let content = locator.slice_source_code_at(stmt_at);
+
+    // Case 1: `object` is the only base.
+    if bases.len() == 1 && keywords.is_empty() {
+        let mut fix_start = None;
+        let mut fix_end = None;
+        let mut count: usize = 0;
+        for (start, tok, end) in lexer::make_tokenizer(content).flatten() {
+            if matches!(tok, Tok::Lpar) {
+                if count == 0 {
+                    fix_start = Some(to_absolute(&start, stmt_at));
+                }
+                count += 1;
+            }
+
+            if matches!(tok, Tok::Rpar) {
+                count -= 1;
+                if count == 0 {
+                    fix_end = Some(to_absolute(&end, stmt_at));
+                    break;
+                }
+            }
+        }
+
+        return match (fix_start, fix_end) {
+            (Some(start), Some(end)) => Some(Fix {
+                content: "".to_string(),
+                location: start,
+                end_location: end,
+                applied: false,
+            }),
+            _ => None,
+        };
+    }
+
+    if bases
+        .iter()
+        .map(|node| node.location)
+        .chain(keywords.iter().map(|node| node.location))
+        .any(|location| location > expr_at)
+    {
+        // Case 2: `object` is _not_ the last node.
+        let mut fix_start: Option<Location> = None;
+        let mut fix_end: Option<Location> = None;
+        let mut seen_comma = false;
+        for (start, tok, end) in lexer::make_tokenizer(content).flatten() {
+            let start = to_absolute(&start, stmt_at);
+            if seen_comma {
+                if matches!(tok, Tok::Newline) {
+                    fix_end = Some(end);
+                } else {
+                    fix_end = Some(start);
+                }
+                break;
+            }
+            if start == expr_at {
+                fix_start = Some(start);
+            }
+            if fix_start.is_some() && matches!(tok, Tok::Comma) {
+                seen_comma = true;
+            }
+        }
+
+        match (fix_start, fix_end) {
+            (Some(start), Some(end)) => Some(Fix {
+                content: "".to_string(),
+                location: start,
+                end_location: end,
+                applied: false,
+            }),
+            _ => None,
+        }
+    } else {
+        // Case 3: `object` is the last node, so we have to find the last token that isn't a comma.
+        let mut fix_start: Option<Location> = None;
+        let mut fix_end: Option<Location> = None;
+        for (start, tok, end) in lexer::make_tokenizer(content).flatten() {
+            let start = to_absolute(&start, stmt_at);
+            let end = to_absolute(&end, stmt_at);
+            if start == expr_at {
+                fix_end = Some(end);
+                break;
+            }
+            if matches!(tok, Tok::Comma) {
+                fix_start = Some(start);
+            }
+        }
+
+        match (fix_start, fix_end) {
+            (Some(start), Some(end)) => Some(Fix {
+                content: "".to_string(),
+                location: start,
+                end_location: end,
+                applied: false,
+            }),
+            _ => None,
+        }
+    }
+}
+
+pub fn remove_super_arguments(locator: &mut SourceCodeLocator, expr: &Expr) -> Option<Fix> {
+    let range = Range::from_located(expr);
+    let contents = locator.slice_source_code_range(&range);
+
+    let mut tree = match libcst_native::parse_module(contents, None) {
+        Ok(m) => m,
+        Err(_) => return None,
+    };
+
+    if let Some(Statement::Simple(body)) = tree.body.first_mut() {
+        if let Some(SmallStatement::Expr(body)) = body.body.first_mut() {
+            if let Expression::Call(body) = &mut body.value {
+                body.args = vec![];
+                body.whitespace_before_args = Default::default();
+                body.whitespace_after_func = Default::default();
+
+                let mut state = Default::default();
+                tree.codegen(&mut state);
+
+                return Some(Fix {
+                    content: state.to_string(),
+                    location: range.location,
+                    end_location: range.end_location,
+                    applied: false,
+                });
+            }
+        }
+    }
+
+    None
+}
+
+/// Determine if a body contains only a single statement, taking into account deleted.
+fn has_single_child(body: &[Stmt], deleted: &[&Stmt]) -> bool {
+    body.iter().filter(|child| !deleted.contains(child)).count() == 1
+}
+
+/// Determine if a child is the only statement in its body.
+fn is_lone_child(child: &Stmt, parent: &Stmt, deleted: &[&Stmt]) -> Result<bool> {
+    match &parent.node {
+        StmtKind::FunctionDef { body, .. }
+        | StmtKind::AsyncFunctionDef { body, .. }
+        | StmtKind::ClassDef { body, .. }
+        | StmtKind::With { body, .. }
+        | StmtKind::AsyncWith { body, .. } => {
+            if body.iter().contains(child) {
+                Ok(has_single_child(body, deleted))
+            } else {
+                Err(anyhow::anyhow!("Unable to find child in parent body."))
+            }
+        }
+        StmtKind::For { body, orelse, .. }
+        | StmtKind::AsyncFor { body, orelse, .. }
+        | StmtKind::While { body, orelse, .. }
+        | StmtKind::If { body, orelse, .. } => {
+            if body.iter().contains(child) {
+                Ok(has_single_child(body, deleted))
+            } else if orelse.iter().contains(child) {
+                Ok(has_single_child(orelse, deleted))
+            } else {
+                Err(anyhow::anyhow!("Unable to find child in parent body."))
+            }
+        }
+        StmtKind::Try {
+            body,
+            handlers,
+            orelse,
+            finalbody,
+        } => {
+            if body.iter().contains(child) {
+                Ok(has_single_child(body, deleted))
+            } else if orelse.iter().contains(child) {
+                Ok(has_single_child(orelse, deleted))
+            } else if finalbody.iter().contains(child) {
+                Ok(has_single_child(finalbody, deleted))
+            } else if let Some(body) = handlers.iter().find_map(|handler| match &handler.node {
+                ExcepthandlerKind::ExceptHandler { body, .. } => {
+                    if body.iter().contains(child) {
+                        Some(body)
+                    } else {
+                        None
+                    }
+                }
+            }) {
+                Ok(has_single_child(body, deleted))
+            } else {
+                Err(anyhow::anyhow!("Unable to find child in parent body."))
+            }
+        }
+        _ => Err(anyhow::anyhow!("Unable to find child in parent body.")),
+    }
+}
+
+pub fn remove_stmt(stmt: &Stmt, parent: Option<&Stmt>, deleted: &[&Stmt]) -> Result<Fix> {
+    if parent
+        .map(|parent| is_lone_child(stmt, parent, deleted))
+        .map_or(Ok(None), |v| v.map(Some))?
+        .unwrap_or_default()
+    {
+        // If removing this node would lead to an invalid syntax tree, replace
+        // it with a `pass`.
+        Ok(Fix {
+            location: stmt.location,
+            end_location: stmt.end_location,
+            content: "pass".to_string(),
+            applied: false,
+        })
+    } else {
+        // Otherwise, nuke the entire line.
+        // TODO(charlie): This logic assumes that there are no multi-statement physical lines.
+        Ok(Fix {
+            location: Location::new(stmt.location.row(), 1),
+            end_location: Location::new(stmt.end_location.row() + 1, 1),
+            content: "".to_string(),
+            applied: false,
+        })
+    }
+}
+
+/// Generate a Fix to remove any unused imports from an `import` statement.
+pub fn remove_unused_imports(
+    locator: &mut SourceCodeLocator,
+    full_names: &[&str],
+    stmt: &Stmt,
+    parent: Option<&Stmt>,
+    deleted: &[&Stmt],
+) -> Result<Fix> {
+    let mut tree = match libcst_native::parse_module(
+        locator.slice_source_code_range(&Range::from_located(stmt)),
+        None,
+    ) {
+        Ok(m) => m,
+        Err(_) => return Err(anyhow::anyhow!("Failed to extract CST from source.")),
+    };
+
+    let body = if let Some(Statement::Simple(body)) = tree.body.first_mut() {
+        body
+    } else {
+        return Err(anyhow::anyhow!("Expected node to be: Statement::Simple."));
+    };
+    let body = if let Some(SmallStatement::Import(body)) = body.body.first_mut() {
+        body
+    } else {
+        return Err(anyhow::anyhow!(
+            "Expected node to be: SmallStatement::ImportFrom."
+        ));
+    };
+    let aliases = &mut body.names;
+
+    // Preserve the trailing comma (or not) from the last entry.
+    let trailing_comma = aliases.last().and_then(|alias| alias.comma.clone());
+
+    // Identify unused imports from within the `import from`.
+    let mut removable = vec![];
+    for (index, alias) in aliases.iter().enumerate() {
+        if let N(import_name) = &alias.name {
+            if full_names.contains(&import_name.value) {
+                removable.push(index);
+            }
+        }
+    }
+    // TODO(charlie): This is quadratic.
+    for index in removable.iter().rev() {
+        aliases.remove(*index);
+    }
+
+    if let Some(alias) = aliases.last_mut() {
+        alias.comma = trailing_comma;
+    }
+
+    if aliases.is_empty() {
+        remove_stmt(stmt, parent, deleted)
+    } else {
+        let mut state = Default::default();
+        tree.codegen(&mut state);
+
+        Ok(Fix {
+            content: state.to_string(),
+            location: stmt.location,
+            end_location: stmt.end_location,
+            applied: false,
+        })
+    }
+}
+
+/// Generate a Fix to remove any unused imports from an `import from` statement.
+pub fn remove_unused_import_froms(
+    locator: &mut SourceCodeLocator,
+    full_names: &[&str],
+    stmt: &Stmt,
+    parent: Option<&Stmt>,
+    deleted: &[&Stmt],
+) -> Result<Fix> {
+    let mut tree = match libcst_native::parse_module(
+        locator.slice_source_code_range(&Range::from_located(stmt)),
+        None,
+    ) {
+        Ok(m) => m,
+        Err(_) => return Err(anyhow::anyhow!("Failed to extract CST from source.")),
+    };
+
+    let body = if let Some(Statement::Simple(body)) = tree.body.first_mut() {
+        body
+    } else {
+        return Err(anyhow::anyhow!("Expected node to be: Statement::Simple."));
+    };
+    let body = if let Some(SmallStatement::ImportFrom(body)) = body.body.first_mut() {
+        body
+    } else {
+        return Err(anyhow::anyhow!(
+            "Expected node to be: SmallStatement::ImportFrom."
+        ));
+    };
+    let aliases = if let Aliases(aliases) = &mut body.names {
+        aliases
+    } else {
+        return Err(anyhow::anyhow!("Expected node to be: Aliases."));
+    };
+
+    // Preserve the trailing comma (or not) from the last entry.
+    let trailing_comma = aliases.last().and_then(|alias| alias.comma.clone());
+
+    // Identify unused imports from within the `import from`.
+    let mut removable = vec![];
+    for (index, alias) in aliases.iter().enumerate() {
+        if let N(name) = &alias.name {
+            let import_name = if let Some(N(module_name)) = &body.module {
+                format!("{}.{}", module_name.value, name.value)
+            } else {
+                name.value.to_string()
+            };
+            if full_names.contains(&import_name.as_str()) {
+                removable.push(index);
+            }
+        }
+    }
+    // TODO(charlie): This is quadratic.
+    for index in removable.iter().rev() {
+        aliases.remove(*index);
+    }
+
+    if let Some(alias) = aliases.last_mut() {
+        alias.comma = trailing_comma;
+    }
+
+    if aliases.is_empty() {
+        remove_stmt(stmt, parent, deleted)
+    } else {
+        let mut state = Default::default();
+        tree.codegen(&mut state);
+
+        Ok(Fix {
+            content: state.to_string(),
+            location: stmt.location,
+            end_location: stmt.end_location,
+            applied: false,
+        })
+    }
+}
--- a/src/autofix/helpers.rs
+++ b/src/autofix/helpers.rs
@@ -1,89 +0,0 @@
-use anyhow::Result;
-use itertools::Itertools;
-use rustpython_parser::ast::{ExcepthandlerKind, Location, Stmt, StmtKind};
-
-use crate::autofix::Fix;
-
-/// Determine if a body contains only a single statement, taking into account deleted.
-fn has_single_child(body: &[Stmt], deleted: &[&Stmt]) -> bool {
-    body.iter().filter(|child| !deleted.contains(child)).count() == 1
-}
-
-/// Determine if a child is the only statement in its body.
-fn is_lone_child(child: &Stmt, parent: &Stmt, deleted: &[&Stmt]) -> Result<bool> {
-    match &parent.node {
-        StmtKind::FunctionDef { body, .. }
-        | StmtKind::AsyncFunctionDef { body, .. }
-        | StmtKind::ClassDef { body, .. }
-        | StmtKind::With { body, .. }
-        | StmtKind::AsyncWith { body, .. } => {
-            if body.iter().contains(child) {
-                Ok(has_single_child(body, deleted))
-            } else {
-                Err(anyhow::anyhow!("Unable to find child in parent body."))
-            }
-        }
-        StmtKind::For { body, orelse, .. }
-        | StmtKind::AsyncFor { body, orelse, .. }
-        | StmtKind::While { body, orelse, .. }
-        | StmtKind::If { body, orelse, .. } => {
-            if body.iter().contains(child) {
-                Ok(has_single_child(body, deleted))
-            } else if orelse.iter().contains(child) {
-                Ok(has_single_child(orelse, deleted))
-            } else {
-                Err(anyhow::anyhow!("Unable to find child in parent body."))
-            }
-        }
-        StmtKind::Try {
-            body,
-            handlers,
-            orelse,
-            finalbody,
-        } => {
-            if body.iter().contains(child) {
-                Ok(has_single_child(body, deleted))
-            } else if orelse.iter().contains(child) {
-                Ok(has_single_child(orelse, deleted))
-            } else if finalbody.iter().contains(child) {
-                Ok(has_single_child(finalbody, deleted))
-            } else if let Some(body) = handlers.iter().find_map(|handler| match &handler.node {
-                ExcepthandlerKind::ExceptHandler { body, .. } => {
-                    if body.iter().contains(child) {
-                        Some(body)
-                    } else {
-                        None
-                    }
-                }
-            }) {
-                Ok(has_single_child(body, deleted))
-            } else {
-                Err(anyhow::anyhow!("Unable to find child in parent body."))
-            }
-        }
-        _ => Err(anyhow::anyhow!("Unable to find child in parent body.")),
-    }
-}
-
-pub fn remove_stmt(stmt: &Stmt, parent: Option<&Stmt>, deleted: &[&Stmt]) -> Result<Fix> {
-    if parent
-        .map(|parent| is_lone_child(stmt, parent, deleted))
-        .map_or(Ok(None), |v| v.map(Some))?
-        .unwrap_or_default()
-    {
-        // If removing this node would lead to an invalid syntax tree, replace
-        // it with a `pass`.
-        Ok(Fix::replacement(
-            "pass".to_string(),
-            stmt.location,
-            stmt.end_location.unwrap(),
-        ))
-    } else {
-        // Otherwise, nuke the entire line.
-        // TODO(charlie): This logic assumes that there are no multi-statement physical lines.
-        Ok(Fix::deletion(
-            Location::new(stmt.location.row(), 1),
-            Location::new(stmt.end_location.unwrap().row() + 1, 1),
-        ))
-    }
-}
--- a/src/autofix/mod.rs
+++ b/src/autofix/mod.rs
@@ -1,53 +0,0 @@
-use rustpython_ast::Location;
-use serde::{Deserialize, Serialize};
-
-pub mod fixer;
-pub mod helpers;
-
-#[derive(Debug, PartialEq, Eq, PartialOrd, Ord, Serialize, Deserialize)]
-pub struct Patch {
-    pub content: String,
-    pub location: Location,
-    pub end_location: Location,
-}
-
-#[derive(Debug, PartialEq, Eq, Serialize, Deserialize)]
-pub struct Fix {
-    pub patch: Patch,
-    pub applied: bool,
-}
-
-impl Fix {
-    pub fn deletion(start: Location, end: Location) -> Self {
-        Self {
-            patch: Patch {
-                content: "".to_string(),
-                location: start,
-                end_location: end,
-            },
-            applied: false,
-        }
-    }
-
-    pub fn replacement(content: String, start: Location, end: Location) -> Self {
-        Self {
-            patch: Patch {
-                content,
-                location: start,
-                end_location: end,
-            },
-            applied: false,
-        }
-    }
-
-    pub fn insertion(content: String, at: Location) -> Self {
-        Self {
-            patch: Patch {
-                content,
-                location: at,
-                end_location: at,
-            },
-            applied: false,
-        }
-    }
-}
--- a/src/cache.rs
+++ b/src/cache.rs
@@ -1,9 +1,3 @@
-// cacache uses asyncd-std which has no wasm support, so currently no caching support on wasm
-#![cfg_attr(
-    target_family = "wasm",
-    allow(unused_imports, unused_variables, dead_code)
-)]
-
 use std::collections::hash_map::DefaultHasher;
 use std::fs::{create_dir_all, File, Metadata};
 use std::hash::{Hash, Hasher};
@@ -11,7 +5,6 @@ use std::io::Write;
 use std::path::Path;

 use anyhow::Result;
-#[cfg(not(target_family = "wasm"))]
 use cacache::Error::EntryNotFound;
 use filetime::FileTime;
 use log::error;
@@ -114,7 +107,6 @@ pub fn get(
        return None;
    };

-    #[cfg(not(target_family = "wasm"))] // cacache needs async-std which doesn't support wasm
    match cacache::read_sync(cache_dir(), cache_key(path, settings, autofix)) {
        Ok(encoded) => match bincode::deserialize::<CheckResult>(&encoded[..]) {
            Ok(CheckResult {
@@ -145,14 +137,12 @@ pub fn set(
        return;
    };

-    #[cfg(not(target_family = "wasm"))] // modification date not supported on wasm
    let check_result = CheckResultRef {
        metadata: &CacheMetadata {
            mtime: FileTime::from_last_modification_time(metadata).unix_seconds(),
        },
        messages,
    };
-    #[cfg(not(target_family = "wasm"))] // cacache needs async-std which doesn't support wasm
    if let Err(e) = cacache::write_sync(
        cache_dir(),
        cache_key(path, settings, autofix),
--- a/src/check_ast.rs
+++ b/src/check_ast.rs
--- a/src/check_lines.rs
+++ b/src/check_lines.rs
@@ -1,11 +1,10 @@
 use std::collections::BTreeMap;

+use crate::ast::types::Range;
 use rustpython_parser::ast::Location;

-use crate::ast::types::Range;
 use crate::autofix::fixer;
-use crate::autofix::Fix;
-use crate::checks::{Check, CheckCode, CheckKind};
+use crate::checks::{Check, CheckCode, CheckKind, Fix};
 use crate::noqa;
 use crate::noqa::Directive;
 use crate::settings::Settings;
@@ -67,12 +66,12 @@ pub fn check_lines(

                match noqa {
                    (Directive::All(_, _), matches) => {
-                        matches.push(check.kind.code().as_ref());
+                        matches.push(check.kind.code().as_str());
                        ignored.push(index)
                    }
                    (Directive::Codes(_, _, codes), matches) => {
-                        if codes.contains(&check.kind.code().as_ref()) {
-                            matches.push(check.kind.code().as_ref());
+                        if codes.contains(&check.kind.code().as_str()) {
+                            matches.push(check.kind.code().as_str());
                            ignored.push(index);
                        }
                    }
@@ -99,11 +98,11 @@ pub fn check_lines(

                match noqa {
                    (Directive::All(_, _), matches) => {
-                        matches.push(check.kind.code().as_ref());
+                        matches.push(check.kind.code().as_str());
                    }
                    (Directive::Codes(_, _, codes), matches) => {
-                        if codes.contains(&check.kind.code().as_ref()) {
-                            matches.push(check.kind.code().as_ref());
+                        if codes.contains(&check.kind.code().as_str()) {
+                            matches.push(check.kind.code().as_str());
                        } else {
                            line_checks.push(check);
                        }
@@ -114,45 +113,6 @@ pub fn check_lines(
        }
    }

-    // Enforce newlines at end of files.
-    if settings.enabled.contains(&CheckCode::W292) && !contents.ends_with('\n') {
-        // Note: if `lines.last()` is `None`, then `contents` is empty (and so we don't want to
-        // raise W292 anyway).
-        if let Some(line) = lines.last() {
-            let lineno = lines.len() - 1;
-            let noqa_lineno = noqa_line_for
-                .get(lineno)
-                .map(|lineno| lineno - 1)
-                .unwrap_or(lineno);
-
-            let noqa = noqa_directives
-                .entry(noqa_lineno)
-                .or_insert_with(|| (noqa::extract_noqa_directive(lines[noqa_lineno]), vec![]));
-
-            let check = Check::new(
-                CheckKind::NoNewLineAtEndOfFile,
-                Range {
-                    location: Location::new(lines.len(), line.len() + 1),
-                    end_location: Location::new(lines.len(), line.len() + 1),
-                },
-            );
-
-            match noqa {
-                (Directive::All(_, _), matches) => {
-                    matches.push(check.kind.code().as_ref());
-                }
-                (Directive::Codes(_, _, codes), matches) => {
-                    if codes.contains(&check.kind.code().as_ref()) {
-                        matches.push(check.kind.code().as_ref());
-                    } else {
-                        line_checks.push(check);
-                    }
-                }
-                (Directive::None, _) => line_checks.push(check),
-            }
-        }
-    }
-
    // Enforce that the noqa directive was actually used.
    if enforce_noqa {
        for (row, (directive, matches)) in noqa_directives {
@@ -167,10 +127,15 @@ pub fn check_lines(
                            },
                        );
                        if matches!(autofix, fixer::Mode::Generate | fixer::Mode::Apply) {
-                            check.amend(Fix::deletion(
-                                Location::new(row + 1, start + 1),
-                                Location::new(row + 1, lines[row].chars().count() + 1),
-                            ));
+                            check.amend(Fix {
+                                content: "".to_string(),
+                                location: Location::new(row + 1, start + 1),
+                                end_location: Location::new(
+                                    row + 1,
+                                    lines[row].chars().count() + 1,
+                                ),
+                                applied: false,
+                            });
                        }
                        line_checks.push(check);
                    }
@@ -180,15 +145,15 @@ pub fn check_lines(
                    let mut valid_codes = vec![];
                    for code in codes {
                        if !matches.contains(&code) {
-                            invalid_codes.push(code.to_string());
+                            invalid_codes.push(code);
                        } else {
-                            valid_codes.push(code.to_string());
+                            valid_codes.push(code);
                        }
                    }

                    if !invalid_codes.is_empty() {
                        let mut check = Check::new(
-                            CheckKind::UnusedNOQA(Some(invalid_codes)),
+                            CheckKind::UnusedNOQA(Some(invalid_codes.join(", "))),
                            Range {
                                location: Location::new(row + 1, start + 1),
                                end_location: Location::new(row + 1, end + 1),
@@ -196,16 +161,25 @@ pub fn check_lines(
                        );
                        if matches!(autofix, fixer::Mode::Generate | fixer::Mode::Apply) {
                            if valid_codes.is_empty() {
-                                check.amend(Fix::deletion(
-                                    Location::new(row + 1, start + 1),
-                                    Location::new(row + 1, lines[row].chars().count() + 1),
-                                ));
+                                check.amend(Fix {
+                                    content: "".to_string(),
+                                    location: Location::new(row + 1, start + 1),
+                                    end_location: Location::new(
+                                        row + 1,
+                                        lines[row].chars().count() + 1,
+                                    ),
+                                    applied: false,
+                                });
                            } else {
-                                check.amend(Fix::replacement(
-                                    format!("  # noqa: {}", valid_codes.join(", ")),
-                                    Location::new(row + 1, start + 1),
-                                    Location::new(row + 1, lines[row].chars().count() + 1),
-                                ));
+                                check.amend(Fix {
+                                    content: format!("  # noqa: {}", valid_codes.join(", ")),
+                                    location: Location::new(row + 1, start + 1),
+                                    end_location: Location::new(
+                                        row + 1,
+                                        lines[row].chars().count() + 1,
+                                    ),
+                                    applied: false,
+                                });
                            }
                        }
                        line_checks.push(check);
--- a/src/checks.rs
+++ b/src/checks.rs
--- a/src/cli.rs
+++ b/src/cli.rs
@@ -1,142 +0,0 @@
-use std::fmt;
-use std::path::PathBuf;
-
-use clap::{command, Parser};
-use log::warn;
-use regex::Regex;
-
-use crate::checks::CheckCode;
-use crate::printer::SerializationFormat;
-use crate::pyproject::StrCheckCodePair;
-use crate::settings::PythonVersion;
-use crate::RawSettings;
-
-#[derive(Debug, Parser)]
-#[command(author, about = "ruff: An extremely fast Python linter.")]
-#[command(version)]
-pub struct Cli {
-    #[arg(required = true)]
-    pub files: Vec<PathBuf>,
-    /// Path to the `pyproject.toml` file to use for configuration.
-    #[arg(long)]
-    pub config: Option<PathBuf>,
-    /// Enable verbose logging.
-    #[arg(short, long)]
-    pub verbose: bool,
-    /// Disable all logging (but still exit with status code "1" upon detecting errors).
-    #[arg(short, long)]
-    pub quiet: bool,
-    /// Exit with status code "0", even upon detecting errors.
-    #[arg(short, long)]
-    pub exit_zero: bool,
-    /// Run in watch mode by re-running whenever files change.
-    #[arg(short, long)]
-    pub watch: bool,
-    /// Attempt to automatically fix lint errors.
-    #[arg(short, long)]
-    pub fix: bool,
-    /// Disable cache reads.
-    #[arg(short, long)]
-    pub no_cache: bool,
-    /// List of error codes to enable.
-    #[arg(long, value_delimiter = ',')]
-    pub select: Vec<CheckCode>,
-    /// Like --select, but adds additional error codes on top of the selected ones.
-    #[arg(long, value_delimiter = ',')]
-    pub extend_select: Vec<CheckCode>,
-    /// List of error codes to ignore.
-    #[arg(long, value_delimiter = ',')]
-    pub ignore: Vec<CheckCode>,
-    /// Like --ignore, but adds additional error codes on top of the ignored ones.
-    #[arg(long, value_delimiter = ',')]
-    pub extend_ignore: Vec<CheckCode>,
-    /// List of paths, used to exclude files and/or directories from checks.
-    #[arg(long, value_delimiter = ',')]
-    pub exclude: Vec<String>,
-    /// Like --exclude, but adds additional files and directories on top of the excluded ones.
-    #[arg(long, value_delimiter = ',')]
-    pub extend_exclude: Vec<String>,
-    /// List of mappings from file pattern to code to exclude
-    #[arg(long, value_delimiter = ',')]
-    pub per_file_ignores: Vec<StrCheckCodePair>,
-    /// Output serialization format for error messages.
-    #[arg(long, value_enum, default_value_t=SerializationFormat::Text)]
-    pub format: SerializationFormat,
-    /// See the files ruff will be run against with the current settings.
-    #[arg(long)]
-    pub show_files: bool,
-    /// See ruff's settings.
-    #[arg(long)]
-    pub show_settings: bool,
-    /// Enable automatic additions of noqa directives to failing lines.
-    #[arg(long)]
-    pub add_noqa: bool,
-    /// Regular expression matching the name of dummy variables.
-    #[arg(long)]
-    pub dummy_variable_rgx: Option<Regex>,
-    /// The minimum Python version that should be supported.
-    #[arg(long)]
-    pub target_version: Option<PythonVersion>,
-    /// Round-trip auto-formatting.
-    // TODO(charlie): This should be a sub-command.
-    #[arg(long, hide = true)]
-    pub autoformat: bool,
-    /// The name of the file when passing it through stdin.
-    #[arg(long)]
-    pub stdin_filename: Option<String>,
-}
-
-pub enum Warnable {
-    Select,
-    ExtendSelect,
-}
-
-impl fmt::Display for Warnable {
-    fn fmt(&self, fmt: &mut fmt::Formatter) -> fmt::Result {
-        match self {
-            Warnable::Select => fmt.write_str("--select"),
-            Warnable::ExtendSelect => fmt.write_str("--extend-select"),
-        }
-    }
-}
-
-/// Warn the user if they attempt to enable a code that won't be respected.
-pub fn warn_on(
-    flag: Warnable,
-    codes: &[CheckCode],
-    cli_ignore: &[CheckCode],
-    cli_extend_ignore: &[CheckCode],
-    pyproject_settings: &RawSettings,
-    pyproject_path: &Option<PathBuf>,
-) {
-    for code in codes {
-        if !cli_ignore.is_empty() {
-            if cli_ignore.contains(code) {
-                warn!("{code:?} was passed to {flag}, but ignored via --ignore")
-            }
-        } else if pyproject_settings.ignore.contains(code) {
-            if let Some(path) = pyproject_path {
-                warn!(
-                    "{code:?} was passed to {flag}, but ignored by the `ignore` field in {}",
-                    path.to_string_lossy()
-                )
-            } else {
-                warn!("{code:?} was passed to {flag}, but ignored by the default `ignore` field",)
-            }
-        }
-        if !cli_extend_ignore.is_empty() {
-            if cli_extend_ignore.contains(code) {
-                warn!("{code:?} was passed to {flag}, but ignored via --extend-ignore")
-            }
-        } else if pyproject_settings.extend_ignore.contains(code) {
-            if let Some(path) = pyproject_path {
-                warn!(
-                    "{code:?} was passed to {flag}, but ignored by the `extend_ignore` field in {}",
-                    path.to_string_lossy()
-                )
-            } else {
-                warn!("{code:?} was passed to {flag}, but ignored by the default `extend_ignore` field")
-            }
-        }
-    }
-}
--- a/src/code_gen.rs
+++ b/src/code_gen.rs
@@ -115,7 +115,7 @@ impl SourceGenerator {
        Ok(())
    }

-    pub fn unparse_stmt<U>(&mut self, ast: &Stmt<U>) -> fmt::Result {
+    fn unparse_stmt<U>(&mut self, ast: &Stmt<U>) -> fmt::Result {
        macro_rules! statement {
            ($body:block) => {{
                self.newline()?;
@@ -547,7 +547,7 @@ impl SourceGenerator {
        Ok(())
    }

-    pub fn unparse_expr<U>(&mut self, ast: &Expr<U>, level: u8) -> fmt::Result {
+    fn unparse_expr<U>(&mut self, ast: &Expr<U>, level: u8) -> fmt::Result {
        macro_rules! opprec {
            ($opty:ident, $x:expr, $enu:path, $($var:ident($op:literal, $prec:ident)),*$(,)?) => {
                match $x {
--- a/src/docstrings/definition.rs
+++ b/src/docstrings/definition.rs
@@ -1,23 +0,0 @@
-use rustpython_ast::{Expr, Stmt};
-
-#[derive(Debug)]
-pub enum DefinitionKind<'a> {
-    Module,
-    Package,
-    Class(&'a Stmt),
-    NestedClass(&'a Stmt),
-    Function(&'a Stmt),
-    NestedFunction(&'a Stmt),
-    Method(&'a Stmt),
-}
-
-#[derive(Debug)]
-pub struct Definition<'a> {
-    pub kind: DefinitionKind<'a>,
-    pub docstring: Option<&'a Expr>,
-}
-
-pub enum Documentable {
-    Class,
-    Function,
-}
--- a/src/docstrings/extraction.rs
+++ b/src/docstrings/extraction.rs
@@ -1,82 +0,0 @@
-//! Extract docstrings from an AST.
-
-use rustpython_ast::{Constant, Expr, ExprKind, Stmt, StmtKind};
-
-use crate::docstrings::definition::{Definition, DefinitionKind, Documentable};
-use crate::visibility::{Modifier, VisibleScope};
-
-/// Extract a docstring from a function or class body.
-pub fn docstring_from(suite: &[Stmt]) -> Option<&Expr> {
-    if let Some(stmt) = suite.first() {
-        if let StmtKind::Expr { value } = &stmt.node {
-            if matches!(
-                &value.node,
-                ExprKind::Constant {
-                    value: Constant::Str(_),
-                    ..
-                }
-            ) {
-                return Some(value);
-            }
-        }
-    }
-    None
-}
-
-/// Extract a `Definition` from the AST node defined by a `Stmt`.
-pub fn extract<'a>(
-    scope: &VisibleScope,
-    stmt: &'a Stmt,
-    body: &'a [Stmt],
-    kind: &Documentable,
-) -> Definition<'a> {
-    let expr = docstring_from(body);
-    match kind {
-        Documentable::Function => match scope {
-            VisibleScope {
-                modifier: Modifier::Module,
-                ..
-            } => Definition {
-                kind: DefinitionKind::Function(stmt),
-                docstring: expr,
-            },
-            VisibleScope {
-                modifier: Modifier::Class,
-                ..
-            } => Definition {
-                kind: DefinitionKind::Method(stmt),
-                docstring: expr,
-            },
-            VisibleScope {
-                modifier: Modifier::Function,
-                ..
-            } => Definition {
-                kind: DefinitionKind::NestedFunction(stmt),
-                docstring: expr,
-            },
-        },
-        Documentable::Class => match scope {
-            VisibleScope {
-                modifier: Modifier::Module,
-                ..
-            } => Definition {
-                kind: DefinitionKind::Class(stmt),
-                docstring: expr,
-            },
-            VisibleScope {
-                modifier: Modifier::Class,
-                ..
-            } => Definition {
-                kind: DefinitionKind::NestedClass(stmt),
-                docstring: expr,
-            },
-            VisibleScope {
-                modifier: Modifier::Function,
-                ..
-            } => Definition {
-                kind: DefinitionKind::NestedClass(stmt),
-                docstring: expr,
-            },
-        },
-    }
-}
--- a/src/docstrings/google.rs
+++ b/src/docstrings/google.rs
@@ -1,71 +0,0 @@
-//! Abstractions for Google-style docstrings.
-
-use std::collections::BTreeSet;
-
-use once_cell::sync::Lazy;
-
-pub(crate) static GOOGLE_SECTION_NAMES: Lazy<BTreeSet<&'static str>> = Lazy::new(|| {
-    BTreeSet::from([
-        "Args",
-        "Arguments",
-        "Attention",
-        "Attributes",
-        "Caution",
-        "Danger",
-        "Error",
-        "Example",
-        "Examples",
-        "Hint",
-        "Important",
-        "Keyword Args",
-        "Keyword Arguments",
-        "Methods",
-        "Note",
-        "Notes",
-        "Return",
-        "Returns",
-        "Raises",
-        "References",
-        "See Also",
-        "Tip",
-        "Todo",
-        "Warning",
-        "Warnings",
-        "Warns",
-        "Yield",
-        "Yields",
-    ])
-});
-
-pub(crate) static LOWERCASE_GOOGLE_SECTION_NAMES: Lazy<BTreeSet<&'static str>> = Lazy::new(|| {
-    BTreeSet::from([
-        "args",
-        "arguments",
-        "attention",
-        "attributes",
-        "caution",
-        "danger",
-        "error",
-        "example",
-        "examples",
-        "hint",
-        "important",
-        "keyword args",
-        "keyword arguments",
-        "methods",
-        "note",
-        "notes",
-        "return",
-        "returns",
-        "raises",
-        "references",
-        "see also",
-        "tip",
-        "todo",
-        "warning",
-        "warnings",
-        "warns",
-        "yield",
-        "yields",
-    ])
-});
--- a/src/docstrings/helpers.rs
+++ b/src/docstrings/helpers.rs
@@ -1,42 +0,0 @@
-use rustpython_ast::{Expr, Location};
-
-use crate::ast::types::Range;
-use crate::check_ast::Checker;
-
-pub const TRIPLE_QUOTE_PREFIXES: &[&str] = &[
-    "ur\"\"\"", "ur'''", "u\"\"\"", "u'''", "r\"\"\"", "r'''", "\"\"\"", "'''",
-];
-
-pub const SINGLE_QUOTE_PREFIXES: &[&str] = &["ur\"", "ur'", "u\"", "u'", "r\"", "r'", "\"", "'"];
-
-/// Extract the leading words from a line of text.
-pub fn leading_words(line: &str) -> String {
-    line.trim()
-        .chars()
-        .take_while(|char| char.is_alphanumeric() || char.is_whitespace())
-        .collect()
-}
-
-/// Extract the leading whitespace from a line of text.
-pub fn leading_space(line: &str) -> String {
-    line.chars()
-        .take_while(|char| char.is_whitespace())
-        .collect()
-}
-
-/// Extract the leading indentation from a docstring.
-pub fn indentation<'a>(checker: &'a mut Checker, docstring: &Expr) -> &'a str {
-    let range = Range::from_located(docstring);
-    checker.locator.slice_source_code_range(&Range {
-        location: Location::new(range.location.row(), 1),
-        end_location: Location::new(range.location.row(), range.location.column()),
-    })
-}
-
-/// Replace any non-whitespace characters from an indentation string.
-pub fn clean(indentation: &str) -> String {
-    indentation
-        .chars()
-        .map(|char| if char.is_whitespace() { char } else { ' ' })
-        .collect()
-}
--- a/src/docstrings/mod.rs
+++ b/src/docstrings/mod.rs
@@ -1,7 +0,0 @@
-pub mod definition;
-pub mod extraction;
-pub mod google;
-pub mod helpers;
-pub mod numpy;
-pub mod sections;
-pub mod styles;
--- a/src/docstrings/numpy.rs
+++ b/src/docstrings/numpy.rs
@@ -1,41 +0,0 @@
-//! Abstractions for NumPy-style docstrings.
-
-use std::collections::BTreeSet;
-
-use once_cell::sync::Lazy;
-
-pub(crate) static LOWERCASE_NUMPY_SECTION_NAMES: Lazy<BTreeSet<&'static str>> = Lazy::new(|| {
-    BTreeSet::from([
-        "short summary",
-        "extended summary",
-        "parameters",
-        "returns",
-        "yields",
-        "other parameters",
-        "raises",
-        "see also",
-        "notes",
-        "references",
-        "examples",
-        "attributes",
-        "methods",
-    ])
-});
-
-pub(crate) static NUMPY_SECTION_NAMES: Lazy<BTreeSet<&'static str>> = Lazy::new(|| {
-    BTreeSet::from([
-        "Short Summary",
-        "Extended Summary",
-        "Parameters",
-        "Returns",
-        "Yields",
-        "Other Parameters",
-        "Raises",
-        "See Also",
-        "Notes",
-        "References",
-        "Examples",
-        "Attributes",
-        "Methods",
-    ])
-});
--- a/src/docstrings/sections.rs
+++ b/src/docstrings/sections.rs
@@ -1,99 +0,0 @@
-use crate::docstrings::helpers;
-use crate::docstrings::styles::SectionStyle;
-
-#[derive(Debug)]
-pub(crate) struct SectionContext<'a> {
-    pub(crate) section_name: String,
-    pub(crate) previous_line: &'a str,
-    pub(crate) line: &'a str,
-    pub(crate) following_lines: &'a [&'a str],
-    pub(crate) is_last_section: bool,
-    pub(crate) original_index: usize,
-}
-
-fn suspected_as_section(line: &str, style: &SectionStyle) -> bool {
-    style
-        .lowercase_section_names()
-        .contains(&helpers::leading_words(line).to_lowercase().as_str())
-}
-
-/// Check if the suspected context is really a section header.
-fn is_docstring_section(context: &SectionContext) -> bool {
-    let section_name_suffix = context
-        .line
-        .trim()
-        .strip_prefix(&context.section_name)
-        .unwrap()
-        .trim();
-    let this_looks_like_a_section_name =
-        section_name_suffix == ":" || section_name_suffix.is_empty();
-    if !this_looks_like_a_section_name {
-        return false;
-    }
-
-    let prev_line = context.previous_line.trim();
-    let prev_line_ends_with_punctuation = [',', ';', '.', '-', '\\', '/', ']', '}', ')']
-        .into_iter()
-        .any(|char| prev_line.ends_with(char));
-    let prev_line_looks_like_end_of_paragraph =
-        prev_line_ends_with_punctuation || prev_line.is_empty();
-    if !prev_line_looks_like_end_of_paragraph {
-        return false;
-    }
-
-    true
-}
-
-/// Extract all `SectionContext` values from a docstring.
-pub(crate) fn section_contexts<'a>(
-    lines: &'a [&'a str],
-    style: &SectionStyle,
-) -> Vec<SectionContext<'a>> {
-    let suspected_section_indices: Vec<usize> = lines
-        .iter()
-        .enumerate()
-        .filter_map(|(lineno, line)| {
-            if lineno > 0 && suspected_as_section(line, style) {
-                Some(lineno)
-            } else {
-                None
-            }
-        })
-        .collect();
-
-    let mut contexts = vec![];
-    for lineno in suspected_section_indices {
-        let context = SectionContext {
-            section_name: helpers::leading_words(lines[lineno]),
-            previous_line: lines[lineno - 1],
-            line: lines[lineno],
-            following_lines: &lines[lineno + 1..],
-            original_index: lineno,
-            is_last_section: false,
-        };
-        if is_docstring_section(&context) {
-            contexts.push(context);
-        }
-    }
-
-    let mut truncated_contexts = vec![];
-    let mut end: Option<usize> = None;
-    for context in contexts.into_iter().rev() {
-        let next_end = context.original_index;
-        truncated_contexts.push(SectionContext {
-            section_name: context.section_name,
-            previous_line: context.previous_line,
-            line: context.line,
-            following_lines: if let Some(end) = end {
-                &lines[context.original_index + 1..end]
-            } else {
-                context.following_lines
-            },
-            original_index: context.original_index,
-            is_last_section: end.is_none(),
-        });
-        end = Some(next_end);
-    }
-    truncated_contexts.reverse();
-    truncated_contexts
-}
--- a/src/docstrings/styles.rs
+++ b/src/docstrings/styles.rs
@@ -1,27 +0,0 @@
-use std::collections::BTreeSet;
-
-use once_cell::sync::Lazy;
-
-use crate::docstrings::google::{GOOGLE_SECTION_NAMES, LOWERCASE_GOOGLE_SECTION_NAMES};
-use crate::docstrings::numpy::{LOWERCASE_NUMPY_SECTION_NAMES, NUMPY_SECTION_NAMES};
-
-pub(crate) enum SectionStyle {
-    NumPy,
-    Google,
-}
-
-impl SectionStyle {
-    pub(crate) fn section_names(&self) -> &Lazy<BTreeSet<&'static str>> {
-        match self {
-            SectionStyle::NumPy => &NUMPY_SECTION_NAMES,
-            SectionStyle::Google => &GOOGLE_SECTION_NAMES,
-        }
-    }
-
-    pub(crate) fn lowercase_section_names(&self) -> &Lazy<BTreeSet<&'static str>> {
-        match self {
-            SectionStyle::NumPy => &LOWERCASE_NUMPY_SECTION_NAMES,
-            SectionStyle::Google => &LOWERCASE_GOOGLE_SECTION_NAMES,
-        }
-    }
-}
--- a/src/flake8_bugbear/mod.rs
+++ b/src/flake8_bugbear/mod.rs
@@ -1 +0,0 @@
-pub mod plugins;
--- a/src/flake8_bugbear/plugins/assert_false.rs
+++ b/src/flake8_bugbear/plugins/assert_false.rs
@@ -1,61 +0,0 @@
-use rustpython_ast::{Constant, Expr, ExprContext, ExprKind, Stmt, StmtKind};
-
-use crate::ast::types::Range;
-use crate::autofix::fixer;
-use crate::autofix::Fix;
-use crate::check_ast::Checker;
-use crate::checks::{Check, CheckKind};
-use crate::code_gen::SourceGenerator;
-
-fn assertion_error(msg: &Option<Box<Expr>>) -> Stmt {
-    Stmt::new(
-        Default::default(),
-        Default::default(),
-        StmtKind::Raise {
-            exc: Some(Box::new(Expr::new(
-                Default::default(),
-                Default::default(),
-                ExprKind::Call {
-                    func: Box::new(Expr::new(
-                        Default::default(),
-                        Default::default(),
-                        ExprKind::Name {
-                            id: "AssertionError".to_string(),
-                            ctx: ExprContext::Load,
-                        },
-                    )),
-                    args: if let Some(msg) = msg {
-                        vec![*msg.clone()]
-                    } else {
-                        vec![]
-                    },
-                    keywords: vec![],
-                },
-            ))),
-            cause: None,
-        },
-    )
-}
-
-pub fn assert_false(checker: &mut Checker, stmt: &Stmt, test: &Expr, msg: &Option<Box<Expr>>) {
-    if let ExprKind::Constant {
-        value: Constant::Bool(false),
-        ..
-    } = &test.node
-    {
-        let mut check = Check::new(CheckKind::DoNotAssertFalse, Range::from_located(test));
-        if matches!(checker.autofix, fixer::Mode::Generate | fixer::Mode::Apply) {
-            let mut generator = SourceGenerator::new();
-            if let Ok(()) = generator.unparse_stmt(&assertion_error(msg)) {
-                if let Ok(content) = generator.generate() {
-                    check.amend(Fix::replacement(
-                        content,
-                        stmt.location,
-                        stmt.end_location.unwrap(),
-                    ));
-                }
-            }
-        }
-        checker.add_check(check);
-    }
-}
--- a/src/flake8_bugbear/plugins/duplicate_exceptions.rs
+++ b/src/flake8_bugbear/plugins/duplicate_exceptions.rs
@@ -1,114 +0,0 @@
-use std::collections::BTreeSet;
-
-use itertools::Itertools;
-use rustpython_ast::{Excepthandler, ExcepthandlerKind, Expr, ExprContext, ExprKind, Stmt};
-
-use crate::ast::helpers;
-use crate::ast::types::{CheckLocator, Range};
-use crate::autofix::fixer;
-use crate::autofix::Fix;
-use crate::check_ast::Checker;
-use crate::checks::{Check, CheckCode, CheckKind};
-use crate::code_gen::SourceGenerator;
-
-fn type_pattern(elts: Vec<&Expr>) -> Expr {
-    Expr::new(
-        Default::default(),
-        Default::default(),
-        ExprKind::Tuple {
-            elts: elts.into_iter().cloned().collect(),
-            ctx: ExprContext::Load,
-        },
-    )
-}
-
-pub fn duplicate_handler_exceptions(
-    checker: &mut Checker,
-    expr: &Expr,
-    elts: &[Expr],
-) -> BTreeSet<String> {
-    let mut seen: BTreeSet<String> = Default::default();
-    let mut duplicates: BTreeSet<String> = Default::default();
-    let mut unique_elts: Vec<&Expr> = Default::default();
-    for type_ in elts {
-        if let Some(name) = helpers::compose_call_path(type_) {
-            if seen.contains(&name) {
-                duplicates.insert(name);
-            } else {
-                seen.insert(name);
-                unique_elts.push(type_);
-            }
-        }
-    }
-
-    if checker.settings.enabled.contains(&CheckCode::B014) {
-        // TODO(charlie): Handle "BaseException" and redundant exception aliases.
-        if !duplicates.is_empty() {
-            let mut check = Check::new(
-                CheckKind::DuplicateHandlerException(
-                    duplicates.into_iter().sorted().collect::<Vec<String>>(),
-                ),
-                checker.locate_check(Range::from_located(expr)),
-            );
-            if matches!(checker.autofix, fixer::Mode::Generate | fixer::Mode::Apply) {
-                // TODO(charlie): If we have a single element, remove the tuple.
-                let mut generator = SourceGenerator::new();
-                if let Ok(()) = generator.unparse_expr(&type_pattern(unique_elts), 0) {
-                    if let Ok(content) = generator.generate() {
-                        check.amend(Fix::replacement(
-                            content,
-                            expr.location,
-                            expr.end_location.unwrap(),
-                        ))
-                    }
-                }
-            }
-            checker.add_check(check);
-        }
-    }
-
-    seen
-}
-
-pub fn duplicate_exceptions(checker: &mut Checker, stmt: &Stmt, handlers: &[Excepthandler]) {
-    let mut seen: BTreeSet<String> = Default::default();
-    let mut duplicates: BTreeSet<String> = Default::default();
-    for handler in handlers {
-        match &handler.node {
-            ExcepthandlerKind::ExceptHandler { type_, .. } => {
-                if let Some(type_) = type_ {
-                    match &type_.node {
-                        ExprKind::Attribute { .. } | ExprKind::Name { .. } => {
-                            if let Some(name) = helpers::compose_call_path(type_) {
-                                if seen.contains(&name) {
-                                    duplicates.insert(name);
-                                } else {
-                                    seen.insert(name);
-                                }
-                            }
-                        }
-                        ExprKind::Tuple { elts, .. } => {
-                            for name in duplicate_handler_exceptions(checker, type_, elts) {
-                                if seen.contains(&name) {
-                                    duplicates.insert(name);
-                                } else {
-                                    seen.insert(name);
-                                }
-                            }
-                        }
-                        _ => {}
-                    }
-                }
-            }
-        }
-    }
-
-    if checker.settings.enabled.contains(&CheckCode::B025) {
-        for duplicate in duplicates.into_iter().sorted() {
-            checker.add_check(Check::new(
-                CheckKind::DuplicateTryBlockException(duplicate),
-                checker.locate_check(Range::from_located(stmt)),
-            ));
-        }
-    }
-}
--- a/src/flake8_bugbear/plugins/mod.rs
+++ b/src/flake8_bugbear/plugins/mod.rs
@@ -1,6 +0,0 @@
-pub use assert_false::assert_false;
-pub use duplicate_exceptions::duplicate_exceptions;
-pub use duplicate_exceptions::duplicate_handler_exceptions;
-
-mod assert_false;
-mod duplicate_exceptions;
--- a/src/flake8_builtins/checks.rs
+++ b/src/flake8_builtins/checks.rs
@@ -1,20 +0,0 @@
-use crate::ast::types::Range;
-use crate::checks::{Check, CheckKind};
-use crate::flake8_builtins::types::ShadowingType;
-use crate::python::builtins::BUILTINS;
-
-/// Check builtin name shadowing.
-pub fn builtin_shadowing(name: &str, location: Range, node_type: ShadowingType) -> Option<Check> {
-    if BUILTINS.contains(&name) {
-        Some(Check::new(
-            match node_type {
-                ShadowingType::Variable => CheckKind::BuiltinVariableShadowing(name.to_string()),
-                ShadowingType::Argument => CheckKind::BuiltinArgumentShadowing(name.to_string()),
-                ShadowingType::Attribute => CheckKind::BuiltinAttributeShadowing(name.to_string()),
-            },
-            location,
-        ))
-    } else {
-        None
-    }
-}
--- a/src/flake8_builtins/mod.rs
+++ b/src/flake8_builtins/mod.rs
@@ -1,2 +0,0 @@
-pub mod checks;
-pub mod types;
--- a/src/flake8_builtins/types.rs
+++ b/src/flake8_builtins/types.rs
@@ -1,5 +0,0 @@
-pub enum ShadowingType {
-    Variable,
-    Argument,
-    Attribute,
-}
--- a/src/flake8_comprehensions/checks.rs
+++ b/src/flake8_comprehensions/checks.rs
@@ -1,505 +0,0 @@
-use num_bigint::BigInt;
-use rustpython_ast::{Comprehension, Constant, Expr, ExprKind, KeywordData, Located, Unaryop};
-
-use crate::ast::types::Range;
-use crate::checks::{Check, CheckKind};
-
-/// Check `list(generator)` compliance.
-pub fn unnecessary_generator_list(expr: &Expr, func: &Expr, args: &[Expr]) -> Option<Check> {
-    if args.len() == 1 {
-        if let ExprKind::Name { id, .. } = &func.node {
-            if id == "list" {
-                if let ExprKind::GeneratorExp { .. } = &args[0].node {
-                    return Some(Check::new(
-                        CheckKind::UnnecessaryGeneratorList,
-                        Range::from_located(expr),
-                    ));
-                }
-            }
-        }
-    }
-    None
-}
-
-/// Check `set(generator)` compliance.
-pub fn unnecessary_generator_set(expr: &Expr, func: &Expr, args: &[Expr]) -> Option<Check> {
-    if args.len() == 1 {
-        if let ExprKind::Name { id, .. } = &func.node {
-            if id == "set" {
-                if let ExprKind::GeneratorExp { .. } = &args[0].node {
-                    return Some(Check::new(
-                        CheckKind::UnnecessaryGeneratorSet,
-                        Range::from_located(expr),
-                    ));
-                }
-            }
-        }
-    }
-    None
-}
-
-/// Check `dict((x, y) for x, y in iterable)` compliance.
-pub fn unnecessary_generator_dict(expr: &Expr, func: &Expr, args: &[Expr]) -> Option<Check> {
-    if args.len() == 1 {
-        if let ExprKind::Name { id, .. } = &func.node {
-            if id == "dict" {
-                if let ExprKind::GeneratorExp { elt, .. } = &args[0].node {
-                    match &elt.node {
-                        ExprKind::Tuple { elts, .. } if elts.len() == 2 => {
-                            return Some(Check::new(
-                                CheckKind::UnnecessaryGeneratorDict,
-                                Range::from_located(expr),
-                            ));
-                        }
-                        _ => {}
-                    }
-                }
-            }
-        }
-    }
-    None
-}
-
-/// Check `set([...])` compliance.
-pub fn unnecessary_list_comprehension_set(
-    expr: &Expr,
-    func: &Expr,
-    args: &[Expr],
-) -> Option<Check> {
-    if args.len() == 1 {
-        if let ExprKind::Name { id, .. } = &func.node {
-            if id == "set" {
-                if let ExprKind::ListComp { .. } = &args[0].node {
-                    return Some(Check::new(
-                        CheckKind::UnnecessaryListComprehensionSet,
-                        Range::from_located(expr),
-                    ));
-                }
-            }
-        }
-    }
-    None
-}
-
-/// Check `dict([...])` compliance.
-pub fn unnecessary_list_comprehension_dict(
-    expr: &Expr,
-    func: &Expr,
-    args: &[Expr],
-) -> Option<Check> {
-    if args.len() == 1 {
-        if let ExprKind::Name { id, .. } = &func.node {
-            if id == "dict" {
-                if let ExprKind::ListComp { elt, .. } = &args[0].node {
-                    match &elt.node {
-                        ExprKind::Tuple { elts, .. } if elts.len() == 2 => {
-                            return Some(Check::new(
-                                CheckKind::UnnecessaryListComprehensionDict,
-                                Range::from_located(expr),
-                            ));
-                        }
-                        _ => {}
-                    }
-                }
-            }
-        }
-    }
-    None
-}
-
-/// Check `set([1, 2])` compliance.
-pub fn unnecessary_literal_set(expr: &Expr, func: &Expr, args: &[Expr]) -> Option<Check> {
-    if args.len() == 1 {
-        if let ExprKind::Name { id, .. } = &func.node {
-            if id == "set" {
-                match &args[0].node {
-                    ExprKind::List { .. } => {
-                        return Some(Check::new(
-                            CheckKind::UnnecessaryLiteralSet("list".to_string()),
-                            Range::from_located(expr),
-                        ));
-                    }
-                    ExprKind::Tuple { .. } => {
-                        return Some(Check::new(
-                            CheckKind::UnnecessaryLiteralSet("tuple".to_string()),
-                            Range::from_located(expr),
-                        ));
-                    }
-                    _ => {}
-                }
-            }
-        }
-    }
-    None
-}
-
-/// Check `dict([(1, 2)])` compliance.
-pub fn unnecessary_literal_dict(expr: &Expr, func: &Expr, args: &[Expr]) -> Option<Check> {
-    if args.len() == 1 {
-        if let ExprKind::Name { id, .. } = &func.node {
-            if id == "dict" {
-                match &args[0].node {
-                    ExprKind::Tuple { elts, .. } => {
-                        if let Some(elt) = elts.first() {
-                            match &elt.node {
-                                // dict((1, 2), ...))
-                                ExprKind::Tuple { elts, .. } if elts.len() == 2 => {
-                                    return Some(Check::new(
-                                        CheckKind::UnnecessaryLiteralDict("tuple".to_string()),
-                                        Range::from_located(expr),
-                                    ));
-                                }
-                                _ => {}
-                            }
-                        } else {
-                            // dict(())
-                            return Some(Check::new(
-                                CheckKind::UnnecessaryLiteralDict("tuple".to_string()),
-                                Range::from_located(expr),
-                            ));
-                        }
-                    }
-                    ExprKind::List { elts, .. } => {
-                        if let Some(elt) = elts.first() {
-                            match &elt.node {
-                                // dict([(1, 2), ...])
-                                ExprKind::Tuple { elts, .. } if elts.len() == 2 => {
-                                    return Some(Check::new(
-                                        CheckKind::UnnecessaryLiteralDict("list".to_string()),
-                                        Range::from_located(expr),
-                                    ));
-                                }
-                                _ => {}
-                            }
-                        } else {
-                            // dict([])
-                            return Some(Check::new(
-                                CheckKind::UnnecessaryLiteralDict("list".to_string()),
-                                Range::from_located(expr),
-                            ));
-                        }
-                    }
-                    _ => {}
-                }
-            }
-        }
-    }
-    None
-}
-
-pub fn unnecessary_collection_call(
-    expr: &Expr,
-    func: &Expr,
-    args: &[Expr],
-    keywords: &[Located<KeywordData>],
-) -> Option<Check> {
-    if args.is_empty() {
-        if let ExprKind::Name { id, .. } = &func.node {
-            if id == "list" || id == "tuple" {
-                // list() or tuple()
-                return Some(Check::new(
-                    CheckKind::UnnecessaryCollectionCall(id.to_string()),
-                    Range::from_located(expr),
-                ));
-            } else if id == "dict" {
-                // dict() or dict(a=1)
-                if keywords.is_empty() || keywords.iter().all(|kw| kw.node.arg.is_some()) {
-                    return Some(Check::new(
-                        CheckKind::UnnecessaryCollectionCall(id.to_string()),
-                        Range::from_located(expr),
-                    ));
-                }
-            }
-        }
-    }
-    None
-}
-
-pub fn unnecessary_literal_within_tuple_call(
-    expr: &Expr,
-    func: &Expr,
-    args: &[Expr],
-) -> Option<Check> {
-    if let ExprKind::Name { id, .. } = &func.node {
-        if id == "tuple" {
-            if let Some(arg) = args.first() {
-                match &arg.node {
-                    ExprKind::Tuple { .. } => {
-                        return Some(Check::new(
-                            CheckKind::UnnecessaryLiteralWithinTupleCall("tuple".to_string()),
-                            Range::from_located(expr),
-                        ));
-                    }
-                    ExprKind::List { .. } => {
-                        return Some(Check::new(
-                            CheckKind::UnnecessaryLiteralWithinTupleCall("list".to_string()),
-                            Range::from_located(expr),
-                        ));
-                    }
-                    _ => {}
-                }
-            }
-        }
-    }
-
-    None
-}
-
-pub fn unnecessary_literal_within_list_call(
-    expr: &Expr,
-    func: &Expr,
-    args: &[Expr],
-) -> Option<Check> {
-    if let ExprKind::Name { id, .. } = &func.node {
-        if id == "list" {
-            if let Some(arg) = args.first() {
-                match &arg.node {
-                    ExprKind::Tuple { .. } => {
-                        return Some(Check::new(
-                            CheckKind::UnnecessaryLiteralWithinListCall("tuple".to_string()),
-                            Range::from_located(expr),
-                        ));
-                    }
-                    ExprKind::List { .. } => {
-                        return Some(Check::new(
-                            CheckKind::UnnecessaryLiteralWithinListCall("list".to_string()),
-                            Range::from_located(expr),
-                        ));
-                    }
-                    _ => {}
-                }
-            }
-        }
-    }
-
-    None
-}
-
-pub fn unnecessary_list_call(expr: &Expr, func: &Expr, args: &[Expr]) -> Option<Check> {
-    if let ExprKind::Name { id, .. } = &func.node {
-        if id == "list" {
-            if let Some(arg) = args.first() {
-                if let ExprKind::ListComp { .. } = &arg.node {
-                    return Some(Check::new(
-                        CheckKind::UnnecessaryListCall,
-                        Range::from_located(expr),
-                    ));
-                }
-            }
-        }
-    }
-    None
-}
-
-pub fn unnecessary_call_around_sorted(expr: &Expr, func: &Expr, args: &[Expr]) -> Option<Check> {
-    if let ExprKind::Name { id: outer, .. } = &func.node {
-        if outer == "list" || outer == "reversed" {
-            if let Some(arg) = args.first() {
-                if let ExprKind::Call { func, .. } = &arg.node {
-                    if let ExprKind::Name { id: inner, .. } = &func.node {
-                        if inner == "sorted" {
-                            return Some(Check::new(
-                                CheckKind::UnnecessaryCallAroundSorted(outer.to_string()),
-                                Range::from_located(expr),
-                            ));
-                        }
-                    }
-                }
-            }
-        }
-    }
-    None
-}
-
-pub fn unnecessary_double_cast_or_process(
-    expr: &Expr,
-    func: &Expr,
-    args: &[Expr],
-) -> Option<Check> {
-    if let ExprKind::Name { id: outer, .. } = &func.node {
-        if outer == "list"
-            || outer == "tuple"
-            || outer == "set"
-            || outer == "reversed"
-            || outer == "sorted"
-        {
-            if let Some(arg) = args.first() {
-                if let ExprKind::Call { func, .. } = &arg.node {
-                    if let ExprKind::Name { id: inner, .. } = &func.node {
-                        // Ex) set(tuple(...))
-                        if (outer == "set" || outer == "sorted")
-                            && (inner == "list"
-                                || inner == "tuple"
-                                || inner == "reversed"
-                                || inner == "sorted")
-                        {
-                            return Some(Check::new(
-                                CheckKind::UnnecessaryDoubleCastOrProcess(
-                                    inner.to_string(),
-                                    outer.to_string(),
-                                ),
-                                Range::from_located(expr),
-                            ));
-                        }
-
-                        // Ex) list(tuple(...))
-                        if (outer == "list" || outer == "tuple")
-                            && (inner == "list" || inner == "tuple")
-                        {
-                            return Some(Check::new(
-                                CheckKind::UnnecessaryDoubleCastOrProcess(
-                                    inner.to_string(),
-                                    outer.to_string(),
-                                ),
-                                Range::from_located(expr),
-                            ));
-                        }
-
-                        // Ex) set(set(...))
-                        if outer == "set" && inner == "set" {
-                            return Some(Check::new(
-                                CheckKind::UnnecessaryDoubleCastOrProcess(
-                                    inner.to_string(),
-                                    outer.to_string(),
-                                ),
-                                Range::from_located(expr),
-                            ));
-                        }
-                    }
-                }
-            }
-        }
-    }
-    None
-}
-
-pub fn unnecessary_subscript_reversal(expr: &Expr, func: &Expr, args: &[Expr]) -> Option<Check> {
-    if let Some(first_arg) = args.first() {
-        if let ExprKind::Name { id, .. } = &func.node {
-            if id == "set" || id == "sorted" || id == "reversed" {
-                if let ExprKind::Subscript { slice, .. } = &first_arg.node {
-                    if let ExprKind::Slice { lower, upper, step } = &slice.node {
-                        if lower.is_none() && upper.is_none() {
-                            if let Some(step) = step {
-                                if let ExprKind::UnaryOp {
-                                    op: Unaryop::USub,
-                                    operand,
-                                } = &step.node
-                                {
-                                    if let ExprKind::Constant {
-                                        value: Constant::Int(val),
-                                        ..
-                                    } = &operand.node
-                                    {
-                                        if *val == BigInt::from(1) {
-                                            return Some(Check::new(
-                                                CheckKind::UnnecessarySubscriptReversal(
-                                                    id.to_string(),
-                                                ),
-                                                Range::from_located(expr),
-                                            ));
-                                        }
-                                    }
-                                }
-                            }
-                        }
-                    }
-                }
-            }
-        }
-    }
-    None
-}
-
-pub fn unnecessary_comprehension(
-    expr: &Expr,
-    elt: &Expr,
-    generators: &[Comprehension],
-) -> Option<Check> {
-    if generators.len() == 1 {
-        let generator = &generators[0];
-        if generator.ifs.is_empty() && generator.is_async == 0 {
-            if let ExprKind::Name { id: elt_id, .. } = &elt.node {
-                if let ExprKind::Name { id: target_id, .. } = &generator.target.node {
-                    if elt_id == target_id {
-                        match &expr.node {
-                            ExprKind::ListComp { .. } => {
-                                return Some(Check::new(
-                                    CheckKind::UnnecessaryComprehension("list".to_string()),
-                                    Range::from_located(expr),
-                                ))
-                            }
-                            ExprKind::SetComp { .. } => {
-                                return Some(Check::new(
-                                    CheckKind::UnnecessaryComprehension("set".to_string()),
-                                    Range::from_located(expr),
-                                ))
-                            }
-                            _ => {}
-                        };
-                    }
-                }
-            }
-        }
-    }
-
-    None
-}
-
-pub fn unnecessary_map(expr: &Expr, func: &Expr, args: &[Expr]) -> Option<Check> {
-    if let ExprKind::Name { id, .. } = &func.node {
-        if id == "map" {
-            if args.len() == 2 {
-                if let ExprKind::Lambda { .. } = &args[0].node {
-                    return Some(Check::new(
-                        CheckKind::UnnecessaryMap("generator".to_string()),
-                        Range::from_located(expr),
-                    ));
-                }
-            }
-        } else if id == "list" || id == "set" {
-            if let Some(arg) = args.first() {
-                if let ExprKind::Call { func, args, .. } = &arg.node {
-                    if let ExprKind::Name { id: f, .. } = &func.node {
-                        if f == "map" {
-                            if let Some(arg) = args.first() {
-                                if let ExprKind::Lambda { .. } = &arg.node {
-                                    return Some(Check::new(
-                                        CheckKind::UnnecessaryMap(id.to_string()),
-                                        Range::from_located(expr),
-                                    ));
-                                }
-                            }
-                        }
-                    }
-                }
-            }
-        } else if id == "dict" {
-            if args.len() == 1 {
-                if let ExprKind::Call { func, args, .. } = &args[0].node {
-                    if let ExprKind::Name { id: f, .. } = &func.node {
-                        if f == "map" {
-                            if let Some(arg) = args.first() {
-                                if let ExprKind::Lambda { body, .. } = &arg.node {
-                                    match &body.node {
-                                        ExprKind::Tuple { elts, .. }
-                                        | ExprKind::List { elts, .. }
-                                            if elts.len() == 2 =>
-                                        {
-                                            return Some(Check::new(
-                                                CheckKind::UnnecessaryMap(id.to_string()),
-                                                Range::from_located(expr),
-                                            ))
-                                        }
-                                        _ => {}
-                                    }
-                                }
-                            }
-                        }
-                    }
-                }
-            }
-        }
-    }
-    None
-}
--- a/src/flake8_comprehensions/mod.rs
+++ b/src/flake8_comprehensions/mod.rs
@@ -1 +0,0 @@
-pub mod checks;
--- a/src/flake8_print/checks.rs
+++ b/src/flake8_print/checks.rs
@@ -1,36 +0,0 @@
-use rustpython_ast::{Expr, ExprKind};
-
-use crate::ast::types::Range;
-use crate::checks::{Check, CheckKind};
-
-/// Check whether a function call is a `print` or `pprint` invocation
-pub fn print_call(
-    expr: &Expr,
-    func: &Expr,
-    check_print: bool,
-    check_pprint: bool,
-) -> Option<Check> {
-    if let ExprKind::Name { id, .. } = &func.node {
-        if check_print && id == "print" {
-            return Some(Check::new(CheckKind::PrintFound, Range::from_located(expr)));
-        } else if check_pprint && id == "pprint" {
-            return Some(Check::new(
-                CheckKind::PPrintFound,
-                Range::from_located(expr),
-            ));
-        }
-    }
-
-    if let ExprKind::Attribute { value, attr, .. } = &func.node {
-        if let ExprKind::Name { id, .. } = &value.node {
-            if check_pprint && id == "pprint" && attr == "pprint" {
-                return Some(Check::new(
-                    CheckKind::PPrintFound,
-                    Range::from_located(expr),
-                ));
-            }
-        }
-    }
-
-    None
-}
--- a/Show More
+++ b/Show More