Updated output format

.cargo/config.toml

@@ -26,11 +26,4 @@ rustflags = [
   "-Wclippy::print_stdout",
   "-Wclippy::print_stderr",
   "-Wclippy::dbg_macro",
-  "-Wclippy::empty_drop",
-  "-Wclippy::empty_structs_with_brackets",
-  "-Wclippy::exit",
-  "-Wclippy::get_unwrap",
-  "-Wclippy::rc_buffer",
-  "-Wclippy::rc_mutex",
-  "-Wclippy::rest_pat_in_fully_bound_structs",
 ]

.github/workflows/ci.yaml

@@ -85,6 +85,7 @@ jobs:
           name: ruff
           path: target/debug/ruff
 
+
   cargo-test-wasm:
     runs-on: ubuntu-latest
     name: "cargo test (wasm)"
@@ -110,16 +111,14 @@ jobs:
     steps:
       - uses: actions/checkout@v3
       - name: "Install Rust toolchain"
-        run: rustup component add rustfmt
+        run: rustup show
       - uses: Swatinem/rust-cache@v2
-      - run: ./scripts/add_rule.py --name DoTheThing --prefix PL --code C0999 --linter pylint
+      - run: ./scripts/add_rule.py --name DoTheThing --code PLC999 --linter pylint
       - run: cargo check
-      - run: cargo fmt --all --check
       - run: |
           ./scripts/add_plugin.py test --url https://pypi.org/project/-test/0.1.0/ --prefix TST
-          ./scripts/add_rule.py --name FirstRule --prefix TST --code 001 --linter test
+          ./scripts/add_rule.py --name FirstRule --code TST001 --linter test
       - run: cargo check
-      - run: cargo fmt --all --check
 
   typos:
     name: "spell check"
@@ -177,56 +176,3 @@ jobs:
         with:
           name: ecosystem-result
           path: ecosystem-result
-
-  cargo-udeps:
-    name: "cargo udeps"
-    runs-on: ubuntu-latest
-    steps:
-      - uses: actions/checkout@v3
-      - name: "Install nightly Rust toolchain"
-        # Only pinned to make caching work, update freely
-        run: rustup toolchain install nightly-2023-03-30
-      - uses: Swatinem/rust-cache@v2
-      - name: "Install cargo-udeps"
-        uses: taiki-e/install-action@cargo-udeps
-      - name: "Run cargo-udeps"
-        run: |
-          unused_dependencies=$(cargo +nightly-2023-03-30 udeps > unused.txt && cat unused.txt | cut -d $'\n' -f 2-)
-          if [ -z "$unused_dependencies" ]; then
-            echo "No unused dependencies found" > $GITHUB_STEP_SUMMARY
-            exit 0
-          else
-            echo "Found unused dependencies" > $GITHUB_STEP_SUMMARY
-            echo '```console' >> $GITHUB_STEP_SUMMARY
-            echo "$unused_dependencies" >> $GITHUB_STEP_SUMMARY
-            echo '```' >> $GITHUB_STEP_SUMMARY
-            exit 1
-          fi
-
-  pre-commit:
-    name: "pre-commit"
-    runs-on: ubuntu-latest
-    steps:
-      - uses: actions/checkout@v3
-      - uses: actions/setup-python@v4
-        with:
-          python-version: "3.11"
-      - name: "Install Rust toolchain"
-        run: rustup show
-      - uses: Swatinem/rust-cache@v2
-      - name: "Install pre-commit"
-        run: pip install pre-commit
-      - name: "Cache pre-commit"
-        uses: actions/cache@v3
-        with:
-            path: ~/.cache/pre-commit
-            key: pre-commit-${{ hashFiles('.pre-commit-config.yaml') }}
-      - name: "Run pre-commit"
-        run: |
-          echo '```console' > $GITHUB_STEP_SUMMARY
-          # Enable color output for pre-commit and remove it for the summary
-          SKIP=cargo-fmt,clippy,dev-generate-all pre-commit run --all-files --show-diff-on-failure --color=always | \
-            tee >(sed -E 's/\x1B\[([0-9]{1,2}(;[0-9]{1,2})*)?[mGK]//g' >> $GITHUB_STEP_SUMMARY) >&1
-          exit_code=${PIPESTATUS[0]}
-          echo '```' >> $GITHUB_STEP_SUMMARY
-          exit $exit_code

Cargo.lock

@@ -132,6 +132,12 @@ dependencies = [
  "serde",
 ]
 
+[[package]]
+name = "bisection"
+version = "0.1.0"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "021e079a1bab0ecce6cf4b4b74c0c37afa4a697136eb3b127875c84a8f04a8c3"
+
 [[package]]
 name = "bit-set"
 version = "0.5.3"
@@ -774,7 +780,7 @@ checksum = "0ce7134b9999ecaf8bcd65542e436736ef32ddca1b3e06094cb6ec5755203b80"
 
 [[package]]
 name = "flake8-to-ruff"
-version = "0.0.260"
+version = "0.0.257"
 dependencies = [
  "anyhow",
  "clap 4.1.8",
@@ -1543,9 +1549,8 @@ checksum = "9fa00462b37ead6d11a82c9d568b26682d78e0477dc02d1966c013af80969739"
 
 [[package]]
 name = "pep440_rs"
-version = "0.3.1"
-source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "d5daf676dd9ff1a39faf9c9da9c46f0dbb6211b21a1839a749f5510c24ceca3f"
+version = "0.2.0"
+source = "git+https://github.com/konstin/pep440-rs.git?rev=a8fef4ec47f4c25b070b39cdbe6a0b9847e49941#a8fef4ec47f4c25b070b39cdbe6a0b9847e49941"
 dependencies = [
  "lazy_static",
  "regex",
@@ -1977,9 +1982,10 @@ dependencies = [
 
 [[package]]
 name = "ruff"
-version = "0.0.260"
+version = "0.0.257"
 dependencies = [
  "anyhow",
+ "bisection",
  "bitflags",
  "chrono",
  "clap 4.1.8",
@@ -2027,8 +2033,6 @@ dependencies = [
  "textwrap",
  "thiserror",
  "toml",
- "typed-arena",
- "unicode-width",
 ]
 
 [[package]]
@@ -2059,7 +2063,7 @@ dependencies = [
 
 [[package]]
 name = "ruff_cli"
-version = "0.0.260"
+version = "0.0.257"
 dependencies = [
  "annotate-snippets 0.9.1",
  "anyhow",
@@ -2098,7 +2102,6 @@ dependencies = [
  "tikv-jemallocator",
  "ureq",
  "walkdir",
- "wild",
 ]
 
 [[package]]
@@ -2128,8 +2131,6 @@ dependencies = [
 name = "ruff_diagnostics"
 version = "0.0.0"
 dependencies = [
- "anyhow",
- "log",
  "ruff_python_ast",
  "rustpython-parser",
  "serde",
@@ -2253,7 +2254,6 @@ dependencies = [
  "js-sys",
  "log",
  "ruff",
- "ruff_diagnostics",
  "ruff_python_ast",
  "ruff_rustpython",
  "rustpython-parser",
@@ -2916,12 +2916,6 @@ dependencies = [
  "static_assertions",
 ]
 
-[[package]]
-name = "typed-arena"
-version = "2.0.2"
-source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "6af6ae20167a9ece4bcb41af5b80f8a1f1df981f6391189ce00fd257af04126a"
-
 [[package]]
 name = "typenum"
 version = "1.16.0"
@@ -3268,15 +3262,6 @@ version = "0.5.1"
 source = "registry+https://github.com/rust-lang/crates.io-index"
 checksum = "17882f045410753661207383517a6f62ec3dbeb6a4ed2acce01f0728238d1983"
 
-[[package]]
-name = "wild"
-version = "2.1.0"
-source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "05b116685a6be0c52f5a103334cbff26db643826c7b3735fc0a3ba9871310a74"
-dependencies = [
- "glob",
-]
-
 [[package]]
 name = "winapi"
 version = "0.3.9"

Cargo.toml

@@ -49,6 +49,8 @@ toml = { version = "0.7.2" }
 
 [profile.release]
 lto = "fat"
+codegen-units = 1
+opt-level = 3
 
 [profile.dev.package.insta]
 opt-level = 3

LICENSE

@@ -195,15 +195,6 @@ are:
     CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
   """
 
-- flake8-gettext, licensed as follows:
-  """
-    BSD Zero Clause License
-
-    Permission to use, copy, modify, and/or distribute this software for any purpose with or without fee is hereby granted.
-
-    THE SOFTWARE IS PROVIDED "AS IS" AND THE AUTHOR DISCLAIMS ALL WARRANTIES WITH REGARD TO THIS SOFTWARE INCLUDING ALL IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS. IN NO EVENT SHALL THE AUTHOR BE LIABLE FOR ANY SPECIAL, DIRECT, INDIRECT, OR CONSEQUENTIAL DAMAGES OR ANY DAMAGES WHATSOEVER RESULTING FROM LOSS OF USE, DATA OR PROFITS, WHETHER IN AN ACTION OF CONTRACT, NEGLIGENCE OR OTHER TORTIOUS ACTION, ARISING OUT OF OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
-  """
-
 - flake8-implicit-str-concat, licensed as follows:
   """
     The MIT License (MIT)

README.md

@@ -137,7 +137,7 @@ Ruff can also be used as a [pre-commit](https://pre-commit.com) hook:
 ```yaml
 - repo: https://github.com/charliermarsh/ruff-pre-commit
   # Ruff version.
-  rev: 'v0.0.260'
+  rev: 'v0.0.257'
   hooks:
     - id: ruff
 ```
@@ -160,7 +160,7 @@ select = ["E", "F"]
 ignore = []
 
 # Allow autofix for all enabled rules (when `--fix`) is provided.
-fixable = ["A", "B", "C", "D", "E", "F", "G", "I", "N", "Q", "S", "T", "W", "ANN", "ARG", "BLE", "COM", "DJ", "DTZ", "EM", "ERA", "EXE", "FBT", "ICN", "INP", "ISC", "NPY", "PD", "PGH", "PIE", "PL", "PT", "PTH", "PYI", "RET", "RSE", "RUF", "SIM", "SLF", "TCH", "TID", "TRY", "UP", "YTT"]
+fixable = ["A", "B", "C", "D", "E", "F", "..."]
 unfixable = []
 
 # Exclude a variety of commonly ignored directories.
@@ -243,12 +243,11 @@ quality tools, including:
 - [flake8-comprehensions](https://pypi.org/project/flake8-comprehensions/)
 - [flake8-datetimez](https://pypi.org/project/flake8-datetimez/)
 - [flake8-debugger](https://pypi.org/project/flake8-debugger/)
-- [flake8-django](https://pypi.org/project/flake8-django/)
+- [flake8-django](https://pypi.org/project/flake8-django/) ([#2817](https://github.com/charliermarsh/ruff/issues/2817))
 - [flake8-docstrings](https://pypi.org/project/flake8-docstrings/)
 - [flake8-eradicate](https://pypi.org/project/flake8-eradicate/)
 - [flake8-errmsg](https://pypi.org/project/flake8-errmsg/)
 - [flake8-executable](https://pypi.org/project/flake8-executable/)
-- [flake8-gettext](https://pypi.org/project/flake8-gettext/)
 - [flake8-implicit-str-concat](https://pypi.org/project/flake8-implicit-str-concat/)
 - [flake8-import-conventions](https://github.com/joaopalmeiro/flake8-import-conventions)
 - [flake8-logging-format](https://pypi.org/project/flake8-logging-format/)
@@ -363,7 +362,6 @@ Ruff is used in a number of major open-source projects, including:
 - [nox](https://github.com/wntrblm/nox)
 - [Neon](https://github.com/neondatabase/neon)
 - [The Algorithms](https://github.com/TheAlgorithms/Python)
-- [Openverse](https://github.com/WordPress/openverse)
 
 ## License
 

crates/flake8_to_ruff/Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "flake8-to-ruff"
-version = "0.0.260"
+version = "0.0.257"
 edition = { workspace = true }
 rust-version = { workspace = true }
 

crates/ruff/Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "ruff"
-version = "0.0.260"
+version = "0.0.257"
 authors.workspace = true
 edition.workspace = true
 rust-version.workspace = true
@@ -22,6 +22,7 @@ ruff_python_stdlib = { path = "../ruff_python_stdlib" }
 ruff_rustpython = { path = "../ruff_rustpython" }
 
 anyhow = { workspace = true }
+bisection = { version = "0.1.0" }
 bitflags = { workspace = true }
 chrono = { workspace = true }
 clap = { workspace = true, features = ["derive", "string"], optional = true }
@@ -46,7 +47,9 @@ path-absolutize = { workspace = true, features = [
   "use_unix_paths_on_wasm",
 ] }
 pathdiff = { version = "0.2.1" }
-pep440_rs = { version = "0.3.1", features = ["serde"] }
+pep440_rs = { git = "https://github.com/konstin/pep440-rs.git", features = [
+  "serde",
+], rev = "a8fef4ec47f4c25b070b39cdbe6a0b9847e49941" }
 regex = { workspace = true }
 result-like = { version = "0.4.6" }
 rustc-hash = { workspace = true }
@@ -55,7 +58,6 @@ rustpython-parser = { workspace = true }
 schemars = { workspace = true }
 semver = { version = "1.0.16" }
 serde = { workspace = true }
-serde_json = { workspace = true }
 shellexpand = { workspace = true }
 smallvec = { version = "1.10.0" }
 strum = { workspace = true }
@@ -63,8 +65,7 @@ strum_macros = { workspace = true }
 textwrap = { workspace = true }
 thiserror = { version = "1.0.38" }
 toml = { workspace = true }
-typed-arena = { version = "2.0.2" }
-unicode-width = { version = "0.1.10" }
+serde_json = { workspace = true }
 
 [dev-dependencies]
 insta = { workspace = true, features = ["yaml", "redactions"] }
@@ -73,5 +74,4 @@ test-case = { workspace = true }
 
 [features]
 default = []
-logical_lines = []
 jupyter_notebook = []

crates/ruff/resources/test/disallowed_rule_names.txt

@@ -5,4 +5,3 @@ uses-*
 rewrite-*
 prefer-*
 consider-*
-use-*

crates/ruff/resources/test/fixtures/flake8_bandit/S001.py

@@ -1,3 +1,6 @@
+import pickle
 from telnetlib import Telnet
 
+pickle.loads()
+
 Telnet("localhost", 23)

crates/ruff/resources/test/fixtures/flake8_bandit/S301.py

@@ -1,3 +0,0 @@
-import pickle
-
-pickle.loads()

crates/ruff/resources/test/fixtures/flake8_bugbear/B006_B008.py

@@ -1,9 +1,7 @@
 import collections
 import datetime as dt
-from decimal import Decimal
 import logging
 import operator
-from pathlib import Path
 import random
 import re
 import time
@@ -167,26 +165,6 @@ def float_str_not_inf_or_nan_is_wrong(value=float("3.14")):
     pass
 
 
-# Allow decimals
-def decimal_okay(value=Decimal("0.1")):
-    pass
-
-# Allow dates
-def date_okay(value=dt.date(2023, 3, 27)):
-    pass
-
-# Allow datetimes
-def datetime_okay(value=dt.datetime(2023, 3, 27, 13, 51, 59)):
-    pass
-
-# Allow timedeltas
-def timedelta_okay(value=dt.timedelta(hours=1)):
-    pass
-
-# Allow paths
-def path_okay(value=Path(".")):
-    pass
-
 # B006 and B008
 # We should handle arbitrary nesting of these B008.
 def nested_combo(a=[float(3), dt.datetime.now()]):

crates/ruff/resources/test/fixtures/flake8_bugbear/B018.py

@@ -57,9 +57,3 @@ def foo3():
 
 def foo4():
     ...
-
-
-def foo5():
-    foo.bar  # Attribute (raise)
-    object().__class__  # Attribute (raise)
-    "foo" + "bar"  # BinOp (raise)

crates/ruff/resources/test/fixtures/flake8_bugbear/B031.py

@@ -1,112 +0,0 @@
-import itertools
-from itertools import groupby
-
-shoppers = ["Jane", "Joe", "Sarah"]
-items = [
-    ("lettuce", "greens"),
-    ("tomatoes", "greens"),
-    ("cucumber", "greens"),
-    ("chicken breast", "meats & fish"),
-    ("salmon", "meats & fish"),
-    ("ice cream", "frozen items"),
-]
-
-carts = {shopper: [] for shopper in shoppers}
-
-
-def collect_shop_items(shopper, items):
-    # Imagine this an expensive database query or calculation that is
-    # advantageous to batch.
-    carts[shopper] += items
-
-
-# Invoking the `groupby` function directly
-for _section, section_items in groupby(items, key=lambda p: p[1]):
-    for shopper in shoppers:
-        shopper = shopper.title()
-        collect_shop_items(shopper, section_items)  # B031
-    # We're outside the nested loop and used the group again.
-    collect_shop_items(shopper, section_items)  # B031
-
-for _section, section_items in groupby(items, key=lambda p: p[1]):
-    collect_shop_items("Jane", section_items)
-    collect_shop_items("Joe", section_items)  # B031
-
-
-# Make sure to detect in other loop constructs as well - `while` loop
-for _section, section_items in groupby(items, key=lambda p: p[1]):
-    countdown = 3
-    while countdown > 0:
-        collect_shop_items(shopper, section_items)  # B031
-        countdown -= 1
-
-# Make sure to detect in other loop constructs as well - `list` comprehension
-collection = []
-for _section, section_items in groupby(items, key=lambda p: p[1]):
-    collection.append([list(section_items) for _ in range(3)])  # B031
-
-unique_items = set()
-another_set = set()
-for _section, section_items in groupby(items, key=lambda p: p[1]):
-    # For nested loops, it should not flag the usage of the name
-    for item in section_items:
-        unique_items.add(item)
-
-    # But it should be detected when used again
-    for item in section_items:  # B031
-        another_set.add(item)
-
-for _section, section_items in groupby(items, key=lambda p: p[1]):
-    # Variable has been overridden, skip checking
-    section_items = list(unique_items)
-    collect_shop_items("Jane", section_items)
-    collect_shop_items("Jane", section_items)
-
-for _section, section_items in groupby(items, key=lambda p: p[1]):
-    # Variable has been overridden, skip checking
-    # Not a realistic situation, just for testing purpose
-    (section_items := list(unique_items))
-    collect_shop_items("Jane", section_items)
-    collect_shop_items("Jane", section_items)
-
-for _section, section_items in groupby(items, key=lambda p: p[1]):
-    # This is ok
-    collect_shop_items("Jane", section_items)
-
-# Invocation via the `itertools` module
-for _section, section_items in itertools.groupby(items, key=lambda p: p[1]):
-    for shopper in shoppers:
-        collect_shop_items(shopper, section_items)  # B031
-
-for group in groupby(items, key=lambda p: p[1]):
-    # This is bad, but not detected currently
-    collect_shop_items("Jane", group[1])
-    collect_shop_items("Joe", group[1])
-
-
-#  Make sure we ignore - but don't fail on more complicated invocations
-for _key, (_value1, _value2) in groupby(
-    [("a", (1, 2)), ("b", (3, 4)), ("a", (5, 6))], key=lambda p: p[1]
-):
-    collect_shop_items("Jane", group[1])
-    collect_shop_items("Joe", group[1])
-
-#  Make sure we ignore - but don't fail on more complicated invocations
-for (_key1, _key2), (_value1, _value2) in groupby(
-    [(("a", "a"), (1, 2)), (("b", "b"), (3, 4)), (("a", "a"), (5, 6))],
-    key=lambda p: p[1],
-):
-    collect_shop_items("Jane", group[1])
-    collect_shop_items("Joe", group[1])
-
-
-# Let's redefine the `groupby` function to make sure we pick up the correct one.
-# NOTE: This should always be at the end of the file.
-def groupby(data, key=None):
-    pass
-
-
-for name, group in groupby(items):
-    collect_shop_items("Jane", items)
-    # This shouldn't be flagged as the `groupby` function is different
-    collect_shop_items("Joe", items)

crates/ruff/resources/test/fixtures/flake8_django/DJ012.py

@@ -1,113 +0,0 @@
-from django.db import models
-from django.db.models import Model
-
-
-class StrBeforeRandomField(models.Model):
-    """Model with `__str__` before a random property."""
-
-    class Meta:
-        verbose_name = "test"
-        verbose_name_plural = "tests"
-
-    def __str__(self):
-        return ""
-
-    random_property = "foo"
-
-
-class StrBeforeFieldModel(models.Model):
-    """Model with `__str__` before fields."""
-
-    class Meta:
-        verbose_name = "test"
-        verbose_name_plural = "tests"
-
-    def __str__(self):
-        return "foobar"
-
-    first_name = models.CharField(max_length=32)
-
-
-class ManagerBeforeField(models.Model):
-    """Model with manager before fields."""
-
-    objects = "manager"
-
-    class Meta:
-        verbose_name = "test"
-        verbose_name_plural = "tests"
-
-    def __str__(self):
-        return "foobar"
-
-    first_name = models.CharField(max_length=32)
-
-
-class CustomMethodBeforeStr(models.Model):
-    """Model with a custom method before `__str__`."""
-
-    class Meta:
-        verbose_name = "test"
-        verbose_name_plural = "tests"
-
-    def my_method(self):
-        pass
-
-    def __str__(self):
-        return "foobar"
-
-
-class GetAbsoluteUrlBeforeSave(Model):
-    """Model with `get_absolute_url` method before `save` method.
-
-    Subclass this directly using the `Model` class.
-    """
-
-    def get_absolute_url(self):
-        pass
-
-    def save(self):
-        pass
-
-
-class ConstantsAreNotFields(models.Model):
-    """Model with an assignment to a constant after `__str__`."""
-
-    first_name = models.CharField(max_length=32)
-
-    class Meta:
-        verbose_name = "test"
-        verbose_name_plural = "tests"
-
-    def __str__(self):
-        pass
-
-    MY_CONSTANT = id(1)
-
-
-class PerfectlyFine(models.Model):
-    """Model which has everything in perfect order."""
-
-    first_name = models.CharField(max_length=32)
-    last_name = models.CharField(max_length=32)
-    objects = "manager"
-
-    class Meta:
-        verbose_name = "test"
-        verbose_name_plural = "tests"
-
-    def __str__(self):
-        return "Perfectly fine!"
-
-    def save(self, **kwargs):
-        super(PerfectlyFine, self).save(**kwargs)
-
-    def get_absolute_url(self):
-        return "http://%s" % self
-
-    def my_method(self):
-        pass
-
-    @property
-    def random_property(self):
-        return "%s" % self

crates/ruff/resources/test/fixtures/flake8_gettext/INT001.py

@@ -1 +0,0 @@
-_(f"{'value'}")

crates/ruff/resources/test/fixtures/flake8_gettext/INT002.py

@@ -1 +0,0 @@
-_("{}".format("line"))

crates/ruff/resources/test/fixtures/flake8_gettext/INT003.py

@@ -1 +0,0 @@
-_("%s" % "line")

crates/ruff/resources/test/fixtures/flake8_logging_format/G001.py

@@ -1,9 +1,3 @@
 import logging
-import logging as foo
 
 logging.info("Hello {}".format("World!"))
-logging.log(logging.INFO, "Hello {}".format("World!"))
-foo.info("Hello {}".format("World!"))
-logging.log(logging.INFO, msg="Hello {}".format("World!"))
-logging.log(level=logging.INFO, msg="Hello {}".format("World!"))
-logging.log(msg="Hello {}".format("World!"), level=logging.INFO)

crates/ruff/resources/test/fixtures/flake8_logging_format/G002.py

@@ -1,4 +1,3 @@
 import logging
 
 logging.info("Hello %s" % "World!")
-logging.log(logging.INFO, "Hello %s" % "World!")

crates/ruff/resources/test/fixtures/flake8_logging_format/G003.py

@@ -1,4 +1,3 @@
 import logging
 
 logging.info("Hello" + " " + "World!")
-logging.log(logging.INFO, "Hello" + " " + "World!")

crates/ruff/resources/test/fixtures/flake8_logging_format/G004.py

@@ -2,4 +2,3 @@ import logging
 
 name = "world"
 logging.info(f"Hello {name}")
-logging.log(logging.INFO, f"Hello {name}")

crates/ruff/resources/test/fixtures/flake8_pie/PIE802.py

@@ -1,4 +1,11 @@
-# PIE802
+# no error
+all((x.id for x in bar))
+all(x.id for x in bar)
+all(x.id for x in bar)
+any(x.id for x in bar)
+any({x.id for x in bar})
+
+# PIE 802
 any([x.id for x in bar])
 all([x.id for x in bar])
 any(  # first comment
@@ -7,13 +14,3 @@ any(  # first comment
 all(  # first comment
     [x.id for x in bar],  # second comment
 )  # third comment
-any({x.id for x in bar})
-
-# OK
-all(x.id for x in bar)
-all(x.id for x in bar)
-any(x.id for x in bar)
-all((x.id for x in bar))
-
-async def f() -> bool:
-    return all([await use_greeting(greeting) for greeting in await greetings()])

crates/ruff/resources/test/fixtures/flake8_pyi/PYI011.py

@@ -1,161 +1,79 @@
-import math
-import os
-import sys
-from math import inf
-
-import numpy as np
-
 def f12(
     x,
     y: str = os.pathsep,  # OK
-) -> None: ...
-def f11(*, x: str = "x") -> None: ...  # OK
+) -> None:
+    ...
+
+
+def f11(*, x: str = "x") -> None:  # OK
+    ...
+
+
 def f13(
-    x: list[str] = [  # OK
+    x: list[str] = [
         "foo",
         "bar",
         "baz",
-    ]
-) -> None: ...
+    ]  # OK
+) -> None:
+    ...
+
+
 def f14(
-    x: tuple[str, ...] = (  # OK
+    x: tuple[str, ...] = (
         "foo",
         "bar",
         "baz",
-    )
-) -> None: ...
+    )  # OK
+) -> None:
+    ...
+
+
 def f15(
-    x: set[str] = {  # OK
+    x: set[str] = {
         "foo",
         "bar",
         "baz",
-    }
-) -> None: ...
-def f151(x: dict[int, int] = {1: 2}) -> None: ...  # Ok
-def f152(
-    x: dict[
-        int, int
-    ] = {  # OK
-        1: 2,
-        **{3: 4},
-    }
-) -> None: ...
-def f153(
-    x: list[
-        int
-    ] = [  # OK
-        1,
-        2,
-        3,
-        4,
-        5,
-        6,
-        7,
-        8,
-        9,
-        10,
-        11,
-    ]
-) -> None: ...
-def f154(
-    x: tuple[
-        str, tuple[str, ...]
-    ] = (  # OK
-        "foo",
-        ("bar", "baz"),
-    )
-) -> None: ...
-def f141(
-    x: list[
-        int
-    ] = [  # OK
-        *range(10)
-    ],
-) -> None: ...
-def f142(
-    x: list[
-        int
-    ] = list(  # OK
-        range(10)
-    ),
-) -> None: ...
-def f16(
-    x: frozenset[
-        bytes
-    ] = frozenset(  # OK
-        {b"foo", b"bar", b"baz"}
-    )
-) -> None: ...
+    }  # OK
+) -> None:
+    ...
+
+
+def f16(x: frozenset[bytes] = frozenset({b"foo", b"bar", b"baz"})) -> None:  # OK
+    ...
+
+
 def f17(
-    x: str = "foo"  # OK
-    + "bar",
-) -> None: ...
+    x: str = "foo" + "bar",  # OK
+) -> None:
+    ...
+
+
 def f18(
-    x: str = b"foo"  # OK
-    + b"bar",
-) -> None: ...
+    x: str = b"foo" + b"bar",  # OK
+) -> None:
+    ...
+
+
 def f19(
-    x: object = "foo"  # OK
-    + 4,
-) -> None: ...
+    x: object = "foo" + 4,  # OK
+) -> None:
+    ...
+
+
 def f20(
-    x: int = 5
-    + 5,  # OK
-) -> None: ...
+    x: int = 5 + 5,  # OK
+) -> None:
+    ...
+
+
 def f21(
-    x: complex = 3j
-    - 3j,  # OK
-) -> None: ...
+    x: complex = 3j - 3j,  # OK
+) -> None:
+    ...
+
+
 def f22(
-    x: complex = -42.5j  # OK
-    + 4.3j,
-) -> None: ...
-def f23(
-    x: bool = True,  # OK
-) -> None: ...
-def f24(
-    x: float = 3.14,  # OK
-) -> None: ...
-def f25(
-    x: float = -3.14,  # OK
-) -> None: ...
-def f26(
-    x: complex = -3.14j,  # OK
-) -> None: ...
-def f27(
-    x: complex = -3 - 3.14j,  # OK
-) -> None: ...
-def f28(
-    x: float = math.tau,  # OK
-) -> None: ...
-def f29(
-    x: float = math.inf,  # OK
-) -> None: ...
-def f30(
-    x: float = -math.inf,  # OK
-) -> None: ...
-def f31(
-    x: float = inf,  # OK
-) -> None: ...
-def f32(
-    x: float = np.inf,  # OK
-) -> None: ...
-def f33(
-    x: float = math.nan,  # OK
-) -> None: ...
-def f34(
-    x: float = -math.nan,  # OK
-) -> None: ...
-def f35(
-    x: complex = math.inf  # OK
-    + 1j,
-) -> None: ...
-def f36(
-    *,
-    x: str = sys.version,  # OK
-) -> None: ...
-def f37(
-    *,
-    x: str = ""  # OK
-    + "",
-) -> None: ...
+    x: complex = -42.5j + 4.3j,  # OK
+) -> None:
+    ...

crates/ruff/resources/test/fixtures/flake8_pyi/PYI011.pyi

@@ -11,74 +11,32 @@ def f12(
 ) -> None: ...
 def f11(*, x: str = "x") -> None: ...  # OK
 def f13(
-    x: list[str] = [  # OK
+    x: list[
+        str
+    ] = [  # Error PYI011 Only simple default values allowed for typed arguments
         "foo",
         "bar",
         "baz",
     ]
 ) -> None: ...
 def f14(
-    x: tuple[str, ...] = (  # OK
+    x: tuple[
+        str, ...
+    ] = (  # Error PYI011 Only simple default values allowed for typed arguments
         "foo",
         "bar",
         "baz",
     )
 ) -> None: ...
 def f15(
-    x: set[str] = {  # OK
+    x: set[
+        str
+    ] = {  # Error PYI011 Only simple default values allowed for typed arguments
         "foo",
         "bar",
         "baz",
     }
 ) -> None: ...
-def f151(x: dict[int, int] = {1: 2}) -> None: ...  # Ok
-def f152(
-    x: dict[
-        int, int
-    ] = {  # Error PYI011 Only simple default values allowed for typed arguments
-        1: 2,
-        **{3: 4},
-    }
-) -> None: ...
-def f153(
-    x: list[
-        int
-    ] = [  # Error PYI011 Only simple default values allowed for typed arguments
-        1,
-        2,
-        3,
-        4,
-        5,
-        6,
-        7,
-        8,
-        9,
-        10,
-        11,
-    ]
-) -> None: ...
-def f154(
-    x: tuple[
-        str, tuple[str, ...]
-    ] = (  # Error PYI011 Only simple default values allowed for typed arguments
-        "foo",
-        ("bar", "baz"),
-    )
-) -> None: ...
-def f141(
-    x: list[
-        int
-    ] = [  # Error PYI011 Only simple default values allowed for typed arguments
-        *range(10)
-    ],
-) -> None: ...
-def f142(
-    x: list[
-        int
-    ] = list(  # Error PYI011 Only simple default values allowed for typed arguments
-        range(10)
-    ),
-) -> None: ...
 def f16(
     x: frozenset[
         bytes
@@ -151,11 +109,8 @@ def f35(
     + 1j,
 ) -> None: ...
 def f36(
-    *,
-    x: str = sys.version,  # OK
+    *, x: str = sys.version,  # OK
 ) -> None: ...
 def f37(
-    *,
-    x: str = ""  # Error PYI011 Only simple default values allowed for typed arguments
-    + "",
+    *, x: str = "" + "",  # Error PYI011 Only simple default values allowed for typed arguments
 ) -> None: ...

crates/ruff/resources/test/fixtures/flake8_pyi/PYI012.py

@@ -1,75 +0,0 @@
-# Violations of PYI012
-
-
-class OneAttributeClass:
-    value: int
-    pass  # PYI012 Class body must not contain `pass`
-
-
-class OneAttributeClassRev:
-    pass  # PYI012 Class body must not contain `pass`
-    value: int
-
-
-class DocstringClass:
-    """
-    My body only contains pass.
-    """
-
-    pass  # PYI012 Class body must not contain `pass`
-
-
-class NonEmptyChild(Exception):
-    value: int
-    pass  # PYI012 Class body must not contain `pass`
-
-
-class NonEmptyChild2(Exception):
-    pass  # PYI012 Class body must not contain `pass`
-    value: int
-
-
-class NonEmptyWithInit:
-    value: int
-    pass  # PYI012 Class body must not contain `pass`
-
-    def __init__():
-        pass
-
-
-# Not violations (of PYI012)
-
-
-class EmptyClass:
-    pass  # Y009 Empty body should contain `...`, not `pass`
-
-
-class EmptyOneLine:
-    pass  # Y009 Empty body should contain `...`, not `pass`
-
-
-class Dog:
-    eyes: int = 2
-
-
-class EmptyEllipsis:
-    ...
-
-
-class NonEmptyEllipsis:
-    value: int
-    ...  # Y013 Non-empty class body must not contain `...`
-
-
-class WithInit:
-    value: int = 0
-
-    def __init__():
-        pass
-
-
-def function():
-    pass
-
-
-pass

crates/ruff/resources/test/fixtures/flake8_pyi/PYI012.pyi

@@ -1,59 +0,0 @@
-# Violations of PYI012
-
-class OneAttributeClass:
-    value: int
-    pass  # PYI012 Class body must not contain `pass`
-
-class OneAttributeClassRev:
-    pass  # PYI012 Class body must not contain `pass`
-    value: int
-
-class DocstringClass:
-    """
-    My body only contains pass.
-    """
-
-    pass  # PYI012 Class body must not contain `pass`
-
-class NonEmptyChild(Exception):
-    value: int
-    pass  # PYI012 Class body must not contain `pass`
-
-class NonEmptyChild2(Exception):
-    pass  # PYI012 Class body must not contain `pass`
-    value: int
-
-class NonEmptyWithInit:
-    value: int
-    pass  # PYI012 Class body must not contain `pass`
-
-    def __init__():
-        pass
-
-# Not violations (of PYI012)
-
-class EmptyClass:
-    pass  # Y009 Empty body should contain `...`, not `pass`
-
-class EmptyOneLine:
-    pass  # Y009 Empty body should contain `...`, not `pass`
-
-class Dog:
-    eyes: int = 2
-
-class EmptyEllipsis: ...
-
-class NonEmptyEllipsis:
-    value: int
-    ...  # Y013 Non-empty class body must not contain `...`
-
-class WithInit:
-    value: int = 0
-
-    def __init__():
-        pass
-
-def function():
-    pass
-
-pass

crates/ruff/resources/test/fixtures/flake8_pyi/PYI014.py

@@ -39,58 +39,6 @@ def f15(
     ...
 
 
-def f151(x={1: 2}) -> None:  # Ok
-    ...
-
-
-def f152(
-    x={  # OK
-        1: 2,
-        **{3: 4},
-    }
-) -> None:
-    ...
-
-
-def f153(
-    x=[  # OK
-        1,
-        2,
-        3,
-        4,
-        5,
-        6,
-        7,
-        8,
-        9,
-        10,
-        11,
-    ]
-) -> None:
-    ...
-
-
-def f154(
-    x=(  # OK
-        "foo",
-        ("bar", "baz"),
-    )
-) -> None:
-    ...
-
-
-def f141(
-    x=[*range(10)],  # OK
-) -> None:
-    ...
-
-
-def f142(
-    x=list(range(10)),  # OK
-) -> None:
-    ...
-
-
 def f16(x=frozenset({b"foo", b"bar", b"baz"})) -> None:
     ...  # OK
 

crates/ruff/resources/test/fixtures/flake8_pyi/PYI014.pyi

@@ -4,60 +4,26 @@ def f12(
 ) -> None: ...
 def f11(*, x="x") -> None: ...  # OK
 def f13(
-    x=[  # OK
+    x=[  # Error PYI014
         "foo",
         "bar",
         "baz",
     ]
 ) -> None: ...
 def f14(
-    x=(  # OK
+    x=(  # Error PYI014
         "foo",
         "bar",
         "baz",
     )
 ) -> None: ...
 def f15(
-    x={  # OK
+    x={  # Error PYI014
         "foo",
         "bar",
         "baz",
     }
 ) -> None: ...
-def f151(x={1: 2}) -> None: ...
-def f152(
-    x={  # Error PYI014
-        1: 2,
-        **{3: 4},
-    }
-) -> None: ...
-def f153(
-    x=[  # Error PYI014
-        1,
-        2,
-        3,
-        4,
-        5,
-        6,
-        7,
-        8,
-        9,
-        10,
-        11,
-    ]
-) -> None: ...
-def f154(
-    x=(  # Error PYI014
-        "foo",
-        ("bar", "baz"),
-    )
-) -> None: ...
-def f141(
-    x=[*range(10)],  # Error PYI014
-) -> None: ...
-def f142(
-    x=list(range(10)),  # Error PYI014
-) -> None: ...
 def f16(x=frozenset({b"foo", b"bar", b"baz"})) -> None: ...  # Error PYI014
 def f17(
     x="foo" + "bar",  # Error PYI014
@@ -78,5 +44,5 @@ def f22(
     x=-42.5j + 4.3j,  # Error PYI014
 ) -> None: ...
 def f23(
-    x=True,  # OK
+  x=True,  # OK
 ) -> None: ...

crates/ruff/resources/test/fixtures/flake8_pyi/PYI015.py

@@ -1,48 +0,0 @@
-import builtins
-import typing
-from typing import TypeAlias, Final
-
-field1: int
-field2: int = ...
-field3 = ...  # type: int  # Y033 Do not use type comments in stubs (e.g. use "x: int" instead of "x = ... # type: int")
-field4: int = 0
-field41: int = 0xFFFFFFFF
-field42: int = 1234567890
-field43: int = -0xFFFFFFFF
-field44: int = -1234567890
-field5 = 0  # type: int  # Y033 Do not use type comments in stubs (e.g. use "x: int" instead of "x = ... # type: int")  # Y052 Need type annotation for "field5"
-field6 = 0  # Y052 Need type annotation for "field6"
-field7 = b""  # Y052 Need type annotation for "field7"
-field71 = "foo"  # Y052 Need type annotation for "field71"
-field72: str = "foo"
-field8 = False  # Y052 Need type annotation for "field8"
-field81 = -1  # Y052 Need type annotation for "field81"
-field82: float = -98.43
-field83 = -42j  # Y052 Need type annotation for "field83"
-field84 = 5 + 42j  # Y052 Need type annotation for "field84"
-field85 = -5 - 42j  # Y052 Need type annotation for "field85"
-field9 = None  # Y026 Use typing_extensions.TypeAlias for type aliases, e.g. "field9: TypeAlias = None"
-Field95: TypeAlias = None
-Field96: TypeAlias = int | None
-Field97: TypeAlias = None | typing.SupportsInt | builtins.str | float | bool
-field19 = [1, 2, 3]  # Y052 Need type annotation for "field19"
-field191: list[int] = [1, 2, 3]
-field20 = (1, 2, 3)  # Y052 Need type annotation for "field20"
-field201: tuple[int, ...] = (1, 2, 3)
-field21 = {1, 2, 3}  # Y052 Need type annotation for "field21"
-field211: set[int] = {1, 2, 3}
-field212 = {"foo": "bar"}  # Y052 Need type annotation for "field212"
-field213: dict[str, str] = {"foo": "bar"}
-field22: Final = {"foo": 5}
-field221: list[int] = [1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11]  # Y015 Only simple default values are allowed for assignments
-field223: list[int] = [*range(10)]  # Y015 Only simple default values are allowed for assignments
-field224: list[int] = list(range(10))  # Y015 Only simple default values are allowed for assignments
-field225: list[object] = [{}, 1, 2]  # Y015 Only simple default values are allowed for assignments
-field226: tuple[str | tuple[str, ...], ...] = ("foo", ("foo", "bar"))  # Y015 Only simple default values are allowed for assignments
-field227: dict[str, object] = {"foo": {"foo": "bar"}}  # Y015 Only simple default values are allowed for assignments
-field228: dict[str, list[object]] = {"foo": []}  # Y015 Only simple default values are allowed for assignments
-# When parsed, this case results in `None` being placed in the `.keys` list for the `ast.Dict` node
-field229: dict[int, int] = {1: 2, **{3: 4}}  # Y015 Only simple default values are allowed for assignments
-field23 = "foo" + "bar"  # Y015 Only simple default values are allowed for assignments
-field24 = b"foo" + b"bar"  # Y015 Only simple default values are allowed for assignments
-field25 = 5 * 5  # Y015 Only simple default values are allowed for assignments

crates/ruff/resources/test/fixtures/flake8_pyi/PYI015.pyi

@@ -1,51 +0,0 @@
-import builtins
-import typing
-from typing import TypeAlias, Final
-
-# We shouldn't emit Y015 for simple default values
-field1: int
-field2: int = ...
-field3 = ...  # type: int  # Y033 Do not use type comments in stubs (e.g. use "x: int" instead of "x = ... # type: int")
-field4: int = 0
-field41: int = 0xFFFFFFFF
-field42: int = 1234567890
-field43: int = -0xFFFFFFFF
-field44: int = -1234567890
-field5 = 0  # type: int  # Y033 Do not use type comments in stubs (e.g. use "x: int" instead of "x = ... # type: int")  # Y052 Need type annotation for "field5"
-field6 = 0  # Y052 Need type annotation for "field6"
-field7 = b""  # Y052 Need type annotation for "field7"
-field71 = "foo"  # Y052 Need type annotation for "field71"
-field72: str = "foo"
-field8 = False  # Y052 Need type annotation for "field8"
-field81 = -1  # Y052 Need type annotation for "field81"
-field82: float = -98.43
-field83 = -42j  # Y052 Need type annotation for "field83"
-field84 = 5 + 42j  # Y052 Need type annotation for "field84"
-field85 = -5 - 42j  # Y052 Need type annotation for "field85"
-field9 = None  # Y026 Use typing_extensions.TypeAlias for type aliases, e.g. "field9: TypeAlias = None"
-Field95: TypeAlias = None
-Field96: TypeAlias = int | None
-Field97: TypeAlias = None | typing.SupportsInt | builtins.str | float | bool
-field19 = [1, 2, 3]  # Y052 Need type annotation for "field19"
-field191: list[int] = [1, 2, 3]
-field20 = (1, 2, 3)  # Y052 Need type annotation for "field20"
-field201: tuple[int, ...] = (1, 2, 3)
-field21 = {1, 2, 3}  # Y052 Need type annotation for "field21"
-field211: set[int] = {1, 2, 3}
-field212 = {"foo": "bar"}  # Y052 Need type annotation for "field212"
-field213: dict[str, str] = {"foo": "bar"}
-field22: Final = {"foo": 5}
-
-# We *should* emit Y015 for more complex default values
-field221: list[int] = [1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11]  # Y015 Only simple default values are allowed for assignments
-field223: list[int] = [*range(10)]  # Y015 Only simple default values are allowed for assignments
-field224: list[int] = list(range(10))  # Y015 Only simple default values are allowed for assignments
-field225: list[object] = [{}, 1, 2]  # Y015 Only simple default values are allowed for assignments
-field226: tuple[str | tuple[str, ...], ...] = ("foo", ("foo", "bar"))  # Y015 Only simple default values are allowed for assignments
-field227: dict[str, object] = {"foo": {"foo": "bar"}}  # Y015 Only simple default values are allowed for assignments
-field228: dict[str, list[object]] = {"foo": []}  # Y015 Only simple default values are allowed for assignments
-# When parsed, this case results in `None` being placed in the `.keys` list for the `ast.Dict` node
-field229: dict[int, int] = {1: 2, **{3: 4}}  # Y015 Only simple default values are allowed for assignments
-field23 = "foo" + "bar"  # Y015 Only simple default values are allowed for assignments
-field24 = b"foo" + b"bar"  # Y015 Only simple default values are allowed for assignments
-field25 = 5 * 5  # Y015 Only simple default values are allowed for assignments

crates/ruff/resources/test/fixtures/flake8_return/RET501.py

@@ -2,13 +2,3 @@ def x(y):
     if not y:
         return
     return None  # error
-
-
-class BaseCache:
-    def get(self, key: str) -> str | None:
-        print(f"{key} not found")
-        return None
-
-    def get(self, key: str) -> None:
-        print(f"{key} not found")
-        return None

crates/ruff/resources/test/fixtures/flake8_simplify/SIM101.py

@@ -16,26 +16,15 @@ if isinstance(a, int) or isinstance(b, bool) or isinstance(a, float):  # SIM101
 if (isinstance(a, int) or isinstance(a, float)) and isinstance(b, bool):  # SIM101
     pass
 
-if isinstance(a.b, int) or isinstance(a.b, float):  # SIM101
-    pass
-
-if isinstance(a(), int) or isinstance(a(), float):  # SIM101
-    pass
-
 if isinstance(a, int) and isinstance(b, bool) or isinstance(a, float):
     pass
 
 if isinstance(a, bool) or isinstance(b, str):
     pass
 
-if isinstance(a, int) or isinstance(a.b, float):
-    pass
-
-
 def f():
     # OK
     def isinstance(a, b):
         return False
-
     if isinstance(a, int) or isinstance(a, float):
         pass

crates/ruff/resources/test/fixtures/flake8_simplify/SIM102.py

@@ -46,10 +46,10 @@ if a:
     if b:
         c
 
-while x > 0:
+while True:
     # SIM102
-    if y > 0:
-        if z > 0:
+    if True:
+        if True:
             """this
 is valid"""
 
@@ -64,8 +64,8 @@ is valid"""
 
 
 # SIM102
-if x > 0:
-    if y > 0:
+if True:
+    if True:
         """this
 is valid"""
 
@@ -78,7 +78,7 @@ is valid"""
         ("so is"
 "this for some reason")
 
-while x > 0:
+while True:
     # SIM102
     if node.module:
         if node.module == "multiprocessing" or node.module.startswith(
@@ -129,15 +129,3 @@ if a:
         print("baz")
 else:
     print("bar")
-
-
-# OK
-if False:
-    if a:
-        pass
-
-
-# OK
-if True:
-    if a:
-        pass

crates/ruff/resources/test/fixtures/flake8_simplify/SIM118.py

@@ -20,5 +20,3 @@ for key in list(obj.keys()):
 {k: k for k in obj.keys()}  # SIM118
 
 (k for k in obj.keys())  # SIM118
-
-key in (obj or {}).keys()  # SIM118

crates/ruff/resources/test/fixtures/flake8_simplify/SIM222.py

@@ -1,10 +1,10 @@
-if a or True:  # SIM222
+if a or True:  # SIM223
     pass
 
-if (a or b) or True:  # SIM222
+if (a or b) or True:  # SIM223
     pass
 
-if a or (b or True):  # SIM222
+if a or (b or True):  # SIM223
     pass
 
 if a and True:  # OK
@@ -16,29 +16,3 @@ if True:  # OK
 
 def validate(self, value):
     return json.loads(value) or True  # OK
-
-
-if a or f() or b or g() or True:  # OK
-    pass
-
-if a or f() or True or g() or b:  # SIM222
-    pass
-
-if True or f() or a or g() or b:  # SIM222
-    pass
-
-if a or True or f() or b or g():  # SIM222
-    pass
-
-
-if a and f() and b and g() and False:  # OK
-    pass
-
-if a and f() and False and g() and b:  # OK
-    pass
-
-if False and f() and a and g() and b:  # OK
-    pass
-
-if a and False and f() and b and g():  # OK
-    pass

crates/ruff/resources/test/fixtures/flake8_simplify/SIM223.py

@@ -12,28 +12,3 @@ if a or False:
 
 if False:
     pass
-
-if a and f() and b and g() and False:  # OK
-    pass
-
-if a and f() and False and g() and b:  # SIM223
-    pass
-
-if False and f() and a and g() and b:  # SIM223
-    pass
-
-if a and False and f() and b and g():  # SIM223
-    pass
-
-
-if a or f() or b or g() or True:  # OK
-    pass
-
-if a or f() or True or g() or b:  # OK
-    pass
-
-if True or f() or a or g() or b:  # OK
-    pass
-
-if a or True or f() or b or g():  # OK
-    pass

crates/ruff/resources/test/fixtures/flake8_tidy_imports/TID252/my_package/sublib/api/application.py

@@ -7,4 +7,3 @@ from ..protocol import commands, definitions, responses
 from ..server import example
 from .. import server
 from . import logger, models
-from ..protocol.UpperCaseModule import some_function

crates/ruff/resources/test/fixtures/flake8_tidy_imports/TID252/my_package/sublib/protocol/UpperCaseModule.py

@@ -1,2 +0,0 @@
-def some_function():
-    pass

crates/ruff/resources/test/fixtures/isort/force_sort_within_sections.py

@@ -2,9 +2,7 @@ from a import a1  # import_from
 from c import * # import_from_star
 import a  # import
 import c.d
-from z import z1
 import b as b1  # import_as
-import z
 
 from ..parent import *
 from .my import fn

crates/ruff/resources/test/fixtures/isort/separate_subpackage_first_and_third_party_imports.py

@@ -1,8 +0,0 @@
-import sys
-import baz
-from foo import bar, baz
-from foo.bar import blah, blub
-from foo.bar.baz import something
-import foo
-import foo.bar
-import foo.bar.baz

crates/ruff/resources/test/fixtures/pandas_vet/PD002.py

@@ -18,7 +18,3 @@ if True:
         columns=["a"],
         axis=1,
     )
-
-x.drop(["a"], axis=1, **kwargs, inplace=True)
-x.drop(["a"], axis=1, inplace=True, **kwargs)
-f(x.drop(["a"], axis=1, inplace=True))

crates/ruff/resources/test/fixtures/pycodestyle/E23.py

@@ -13,8 +13,3 @@ result = {
     'key1': 'value',
     'key2': 'value',
 }
-
-def foo() -> None:
-    #: E231
-    if (1,2):
-        pass

crates/ruff/resources/test/fixtures/pycodestyle/E501.py

@@ -56,29 +56,7 @@ sit amet  consectetur adipiscing elit, sed do eiusmod tempor incididunt ut labor
 # OK
 # https://loooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooong.url.com
 
-# OK
+# Not OK
 _ = """
 Source: https://github.com/PyCQA/pycodestyle/pull/258/files#diff-841c622497a8033d10152bfdfb15b20b92437ecdea21a260944ea86b77b51533
 """
-
-# OK
-_ = """
-[this-is-ok](https://github.com/PyCQA/pycodestyle/pull/258/files#diff-841c622497a8033d10152bfdfb15b20b92437ecdea21a260944ea86b77b51533)
-[this is ok](https://github.com/PyCQA/pycodestyle/pull/258/files#diff-841c622497a8033d10152bfdfb15b20b92437ecdea21a260944ea86b77b51533)
-"""
-
-
-# OK
-class Foo:
-    """
-    @see https://looooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooong.url.com
-
-    :param dynamodb_scan_kwargs: kwargs pass to <https://boto3.amazonaws.com/v1/documentation/api/latest/reference/services/dynamodb.html#DynamoDB.Table.scan>
-    """
-
-
-# Error
-class Bar:
-    """
-    This is a long sentence that ends with a shortened URL and, therefore, could easily be broken across multiple lines ([source](https://ruff.rs))
-    """

crates/ruff/resources/test/fixtures/pydocstyle/D403.py

@@ -1,15 +0,0 @@
-def bad_function():
-    """this docstring is not capitalized"""
-
-def good_function():
-    """This docstring is capitalized."""
-
-def other_function():
-    """
-    This docstring is capitalized."""
-
-def another_function():
-    """ This docstring is capitalized."""
-
-def utf8_function():
-    """éste docstring is capitalized."""

crates/ruff/resources/test/fixtures/pyflakes/F401_10.py

@@ -1,19 +1,10 @@
-"""Test: imports within `ModuleNotFoundError` and `ImportError` handlers."""
+"""Test: imports within `ModuleNotFoundError` handlers."""
 
 
-def module_not_found_error():
+def check_orjson():
     try:
         import orjson
 
         return True
     except ModuleNotFoundError:
         return False
-
-
-def import_error():
-    try:
-        import orjson
-
-        return True
-    except ImportError:
-        return False

crates/ruff/resources/test/fixtures/pyflakes/F401_11.py

@@ -1,7 +0,0 @@
-"""Test: parsing of nested string annotations."""
-
-from typing import List
-from pathlib import Path, PurePath
-
-
-x: """List['Path']""" = []

crates/ruff/resources/test/fixtures/pyflakes/F722.py

@@ -8,6 +8,3 @@ def f() -> "A":
 
 def g() -> "///":
     pass
-
-
-X: """List[int]"""'☃' = []

crates/ruff/resources/test/fixtures/pyflakes/F821_13.py

@@ -1,8 +0,0 @@
-"""Test case: ForwardRef."""
-
-from typing import ForwardRef, TypeVar
-
-X = ForwardRef("List[int]")
-Y: ForwardRef("List[int]")
-
-Z = TypeVar("X", "List[int]", "int")

crates/ruff/resources/test/fixtures/pyflakes/F821_14.py

@@ -1,7 +0,0 @@
-"""Test case: f-strings in type annotations."""
-
-from typing import List
-
-x = 1
-
-x: List[f"i{x}nt"] = []

crates/ruff/resources/test/fixtures/pyflakes/F821_15.py

@@ -1,9 +0,0 @@
-"""Test case: f-strings in future type annotations."""
-
-from __future__ import annotations
-
-from typing import List
-
-x = 1
-
-x: List[f"i{x}nt"] = []

crates/ruff/resources/test/fixtures/pylint/sys_exit_alias_1.py

@@ -9,8 +9,4 @@ def main():
     quit(1)
 
 
-def main():
-    sys = 1
-
-    exit(1)
-    quit(1)
+sys.exit(2)

crates/ruff/resources/test/fixtures/pylint/sys_exit_alias_3.py

@@ -7,10 +7,3 @@ quit(0)
 def main():
     exit(1)
     quit(1)
-
-
-def main():
-    exit = 1
-
-    exit(1)
-    quit(1)

crates/ruff/resources/test/fixtures/pylint/useless_return.py

@@ -48,13 +48,3 @@ def print_python_version():
     """This function returns None."""
     print(sys.version)
     return None  # [useless-return]
-
-
-class BaseCache:
-    def get(self, key: str) -> str | None:
-        print(f"{key} not found")
-        return None
-
-    def get(self, key: str) -> None:
-        print(f"{key} not found")
-        return None

crates/ruff/resources/test/fixtures/pyupgrade/UP006.py

@@ -30,29 +30,8 @@ def f(x: "List[str]") -> None:
     ...
 
 
-def f(x: r"List[str]") -> None:
-    ...
-
-
-def f(x: "List[str]") -> None:
-    ...
-
-
-def f(x: """List[str]""") -> None:
-    ...
-
-
-def f(x: "Li" "st[str]") -> None:
-    ...
-
-
-def f(x: "List['List[str]']") -> None:
-    ...
-
-
-def f(x: "List['Li' 'st[str]']") -> None:
-    ...
-
-
-def f(x: "Li" "st['List[str]']") -> None:
+list = "abc"
+
+
+def f(x: List[str]) -> None:
     ...

crates/ruff/resources/test/fixtures/pyupgrade/UP012.py

@@ -29,13 +29,10 @@ string = "hello there"
 string.encode("utf-8")
 
 bar = "bar"
-f"foo{bar}".encode("utf-8")
+f"foo{bar}".encode("utf-8")  # f"foo{bar}".encode()
 encoding = "latin"
 "foo".encode(encoding)
 f"foo{bar}".encode(encoding)
-f"{a=} {b=}".encode(
-    "utf-8",
-)
 
 # `encode` with custom args and kwargs should not be processed.
 "foo".encode("utf-8", errors="replace")

crates/ruff/resources/test/fixtures/pyupgrade/UP033.py

@@ -1,4 +1,5 @@
 import functools
+from functools import lru_cache
 
 
 @functools.lru_cache(maxsize=None)
@@ -6,6 +7,11 @@ def fixme():
     pass
 
 
+@lru_cache(maxsize=None)
+def fixme():
+    pass
+
+
 @other_decorator
 @functools.lru_cache(maxsize=None)
 def fixme():
@@ -23,16 +29,31 @@ def ok():
     pass
 
 
+@lru_cache()
+def ok():
+    pass
+
+
 @functools.lru_cache(maxsize=64)
 def ok():
     pass
 
 
+@lru_cache(maxsize=64)
+def ok():
+    pass
+
+
 def user_func():
     pass
 
 
-@functools.lru_cache(user_func)
+@lru_cache(user_func)
+def ok():
+    pass
+
+
+@lru_cache(user_func, maxsize=None)
 def ok():
     pass
 

crates/ruff/resources/test/fixtures/pyupgrade/UP033_1.py

@@ -1,51 +0,0 @@
-from functools import lru_cache
-
-
-@lru_cache(maxsize=None)
-def fixme():
-    pass
-
-
-@other_decorator
-@lru_cache(maxsize=None)
-def fixme():
-    pass
-
-
-@lru_cache(maxsize=None)
-@other_decorator
-def fixme():
-    pass
-
-
-@lru_cache()
-def ok():
-    pass
-
-
-@lru_cache(maxsize=64)
-def ok():
-    pass
-
-
-def user_func():
-    pass
-
-
-@lru_cache(user_func)
-def ok():
-    pass
-
-
-@lru_cache(user_func, maxsize=None)
-def ok():
-    pass
-
-
-def lru_cache(maxsize=None):
-    pass
-
-
-@lru_cache(maxsize=None)
-def ok():
-    pass

crates/ruff/resources/test/fixtures/ruff/RUF007.py

@@ -1,6 +1,5 @@
 input = [1, 2, 3]
 otherInput = [2, 3, 4]
-foo = [1, 2, 3, 4]
 
 # OK
 zip(input, otherInput)  # different inputs
@@ -9,8 +8,6 @@ zip(input, input[2:])  # not successive
 zip(input[:-1], input[2:])  # not successive
 list(zip(input, otherInput))  # nested call
 zip(input, input[1::2])  # not successive
-zip(foo[:-1], foo[1:], foo, strict=False)  # more than 2 inputs
-zip(foo[:-1], foo[1:], foo, strict=True)  # more than 2 inputs
 
 # Errors
 zip(input, input[1:])
@@ -20,6 +17,3 @@ zip(input[1:], input[2:])
 zip(input[1:-1], input[2:])
 list(zip(input, input[1:]))
 list(zip(input[:-1], input[1:]))
-zip(foo[:-1], foo[1:], strict=True)
-zip(foo[:-1], foo[1:], strict=False)
-zip(foo[:-1], foo[1:], strict=bool(foo))

crates/ruff/resources/test/fixtures/tryceratops/TRY300.py

@@ -40,9 +40,8 @@ def noreturn():
         logger.exception("process failed")
 
 
-def good_return_with_side_effects():
+def still_good():
     try:
-        pass
         return process()
     except MyException:
         logger.exception("process failed")

crates/ruff/src/autofix/helpers.rs

@@ -6,17 +6,14 @@ use libcst_native::{
 use rustpython_parser::ast::{ExcepthandlerKind, Expr, Keyword, Location, Stmt, StmtKind};
 use rustpython_parser::{lexer, Mode, Tok};
 
-use ruff_diagnostics::Edit;
-use ruff_python_ast::context::Context;
+use ruff_diagnostics::Fix;
 use ruff_python_ast::helpers;
 use ruff_python_ast::helpers::to_absolute;
-use ruff_python_ast::imports::{AnyImport, Import};
 use ruff_python_ast::newlines::NewlineWithTrailingNewline;
 use ruff_python_ast::source_code::{Indexer, Locator, Stylist};
 
 use crate::cst::helpers::compose_module_path;
 use crate::cst::matchers::match_module;
-use crate::importer::Importer;
 
 /// Determine if a body contains only a single statement, taking into account
 /// deleted.
@@ -181,7 +178,7 @@ pub fn delete_stmt(
     locator: &Locator,
     indexer: &Indexer,
     stylist: &Stylist,
-) -> Result<Edit> {
+) -> Result<Fix> {
     if parent
         .map(|parent| is_lone_child(stmt, parent, deleted))
         .map_or(Ok(None), |v| v.map(Some))?
@@ -189,7 +186,7 @@ pub fn delete_stmt(
     {
         // If removing this node would lead to an invalid syntax tree, replace
         // it with a `pass`.
-        Ok(Edit::replacement(
+        Ok(Fix::replacement(
             "pass".to_string(),
             stmt.location,
             stmt.end_location.unwrap(),
@@ -197,22 +194,22 @@ pub fn delete_stmt(
     } else {
         Ok(if let Some(semicolon) = trailing_semicolon(stmt, locator) {
             let next = next_stmt_break(semicolon, locator);
-            Edit::deletion(stmt.location, next)
+            Fix::deletion(stmt.location, next)
         } else if helpers::match_leading_content(stmt, locator) {
-            Edit::deletion(stmt.location, stmt.end_location.unwrap())
+            Fix::deletion(stmt.location, stmt.end_location.unwrap())
         } else if helpers::preceded_by_continuation(stmt, indexer) {
             if is_end_of_file(stmt, locator) && stmt.location.column() == 0 {
                 // Special-case: a file can't end in a continuation.
-                Edit::replacement(
+                Fix::replacement(
                     stylist.line_ending().to_string(),
                     stmt.location,
                     stmt.end_location.unwrap(),
                 )
             } else {
-                Edit::deletion(stmt.location, stmt.end_location.unwrap())
+                Fix::deletion(stmt.location, stmt.end_location.unwrap())
             }
         } else {
-            Edit::deletion(
+            Fix::deletion(
                 Location::new(stmt.location.row(), 0),
                 Location::new(stmt.end_location.unwrap().row() + 1, 0),
             )
@@ -229,7 +226,7 @@ pub fn remove_unused_imports<'a>(
     locator: &Locator,
     indexer: &Indexer,
     stylist: &Stylist,
-) -> Result<Edit> {
+) -> Result<Fix> {
     let module_text = locator.slice(stmt);
     let mut tree = match_module(module_text)?;
 
@@ -330,13 +327,13 @@ pub fn remove_unused_imports<'a>(
         delete_stmt(stmt, parent, deleted, locator, indexer, stylist)
     } else {
         let mut state = CodegenState {
-            default_newline: &stylist.line_ending(),
+            default_newline: stylist.line_ending(),
             default_indent: stylist.indentation(),
             ..CodegenState::default()
         };
         tree.codegen(&mut state);
 
-        Ok(Edit::replacement(
+        Ok(Fix::replacement(
             state.to_string(),
             stmt.location,
             stmt.end_location.unwrap(),
@@ -352,15 +349,15 @@ pub fn remove_unused_imports<'a>(
 /// For this behavior, set `remove_parentheses` to `true`.
 pub fn remove_argument(
     locator: &Locator,
-    call_at: Location,
+    stmt_at: Location,
     expr_at: Location,
     expr_end: Location,
     args: &[Expr],
     keywords: &[Keyword],
     remove_parentheses: bool,
-) -> Result<Edit> {
+) -> Result<Fix> {
     // TODO(sbrugman): Preserve trailing comments.
-    let contents = locator.skip(call_at);
+    let contents = locator.skip(stmt_at);
 
     let mut fix_start = None;
     let mut fix_end = None;
@@ -373,7 +370,7 @@ pub fn remove_argument(
     if n_arguments == 1 {
         // Case 1: there is only one argument.
         let mut count: usize = 0;
-        for (start, tok, end) in lexer::lex_located(contents, Mode::Module, call_at).flatten() {
+        for (start, tok, end) in lexer::lex_located(contents, Mode::Module, stmt_at).flatten() {
             if matches!(tok, Tok::Lpar) {
                 if count == 0 {
                     fix_start = Some(if remove_parentheses {
@@ -405,7 +402,7 @@ pub fn remove_argument(
     {
         // Case 2: argument or keyword is _not_ the last node.
         let mut seen_comma = false;
-        for (start, tok, end) in lexer::lex_located(contents, Mode::Module, call_at).flatten() {
+        for (start, tok, end) in lexer::lex_located(contents, Mode::Module, stmt_at).flatten() {
             if seen_comma {
                 if matches!(tok, Tok::NonLogicalNewline) {
                     // Also delete any non-logical newlines after the comma.
@@ -428,7 +425,7 @@ pub fn remove_argument(
     } else {
         // Case 3: argument or keyword is the last node, so we have to find the last
         // comma in the stmt.
-        for (start, tok, _) in lexer::lex_located(contents, Mode::Module, call_at).flatten() {
+        for (start, tok, _) in lexer::lex_located(contents, Mode::Module, stmt_at).flatten() {
             if start == expr_at {
                 fix_end = Some(expr_end);
                 break;
@@ -440,89 +437,13 @@ pub fn remove_argument(
     }
 
     match (fix_start, fix_end) {
-        (Some(start), Some(end)) => Ok(Edit::deletion(start, end)),
+        (Some(start), Some(end)) => Ok(Fix::deletion(start, end)),
         _ => {
             bail!("No fix could be constructed")
         }
     }
 }
 
-/// Generate an [`Edit`] to reference the given symbol. Returns the [`Edit`] necessary to make the
-/// symbol available in the current scope along with the bound name of the symbol.
-///
-/// For example, assuming `module` is `"functools"` and `member` is `"lru_cache"`, this function
-/// could return an [`Edit`] to add `import functools` to the top of the file, alongside with the
-/// name on which the `lru_cache` symbol would be made available (`"functools.lru_cache"`).
-///
-/// Attempts to reuse existing imports when possible.
-pub fn get_or_import_symbol(
-    module: &str,
-    member: &str,
-    context: &Context,
-    importer: &Importer,
-    locator: &Locator,
-) -> Result<(Edit, String)> {
-    if let Some((source, binding)) = context.resolve_qualified_import_name(module, member) {
-        // If the symbol is already available in the current scope, use it.
-        //
-        // We also add a no-nop edit to force conflicts with any other fixes that might try to
-        // remove the import. Consider:
-        //
-        // ```py
-        // import sys
-        //
-        // quit()
-        // ```
-        //
-        // Assume you omit this no-op edit. If you run Ruff with `unused-imports` and
-        // `sys-exit-alias` over this snippet, it will generate two fixes: (1) remove the unused
-        // `sys` import; and (2) replace `quit()` with `sys.exit()`, under the assumption that `sys`
-        // is already imported and available.
-        //
-        // By adding this no-op edit, we force the `unused-imports` fix to conflict with the
-        // `sys-exit-alias` fix, and thus will avoid applying both fixes in the same pass.
-        let import_edit = Edit::replacement(
-            locator.slice(source).to_string(),
-            source.location,
-            source.end_location.unwrap(),
-        );
-        Ok((import_edit, binding))
-    } else {
-        if let Some(stmt) = importer.get_import_from(module) {
-            // Case 1: `from functools import lru_cache` is in scope, and we're trying to reference
-            // `functools.cache`; thus, we add `cache` to the import, and return `"cache"` as the
-            // bound name.
-            if context
-                .find_binding(member)
-                .map_or(true, |binding| binding.kind.is_builtin())
-            {
-                let import_edit = importer.add_member(stmt, member)?;
-                Ok((import_edit, member.to_string()))
-            } else {
-                bail!(
-                    "Unable to insert `{}` into scope due to name conflict",
-                    member
-                )
-            }
-        } else {
-            // Case 2: No `functools` import is in scope; thus, we add `import functools`, and
-            // return `"functools.cache"` as the bound name.
-            if context
-                .find_binding(module)
-                .map_or(true, |binding| binding.kind.is_builtin())
-            {
-                let import_edit = importer.add_import(&AnyImport::Import(Import::module(module)));
-                Ok((import_edit, format!("{module}.{member}")))
-            } else {
-                bail!(
-                    "Unable to insert `{}` into scope due to name conflict",
-                    module
-                )
-            }
-        }
-    }
-}
-
 #[cfg(test)]
 mod tests {
     use anyhow::Result;

crates/ruff/src/autofix/mod.rs

@@ -4,7 +4,7 @@ use itertools::Itertools;
 use rustc_hash::FxHashMap;
 use rustpython_parser::ast::Location;
 
-use ruff_diagnostics::{Diagnostic, Edit, Fix};
+use ruff_diagnostics::{Diagnostic, Fix};
 use ruff_python_ast::source_code::Locator;
 use ruff_python_ast::types::Range;
 
@@ -15,7 +15,7 @@ pub mod helpers;
 
 /// Auto-fix errors in a file, and write the fixed source code to disk.
 pub fn fix_file(diagnostics: &[Diagnostic], locator: &Locator) -> Option<(String, FixTable)> {
-    if diagnostics.iter().all(|check| check.fix.is_empty()) {
+    if diagnostics.iter().all(|check| check.fix.is_none()) {
         None
     } else {
         Some(apply_fixes(diagnostics.iter(), locator))
@@ -29,48 +29,41 @@ fn apply_fixes<'a>(
 ) -> (String, FixTable) {
     let mut output = String::with_capacity(locator.len());
     let mut last_pos: Option<Location> = None;
-    let mut applied: BTreeSet<&Edit> = BTreeSet::default();
+    let mut applied: BTreeSet<&Fix> = BTreeSet::default();
     let mut fixed = FxHashMap::default();
 
     for (rule, fix) in diagnostics
         .filter_map(|diagnostic| {
-            if diagnostic.fix.is_empty() {
-                None
-            } else {
-                Some((diagnostic.kind.rule(), &diagnostic.fix))
-            }
+            diagnostic
+                .fix
+                .as_ref()
+                .map(|fix| (diagnostic.kind.rule(), fix))
         })
         .sorted_by(|(rule1, fix1), (rule2, fix2)| cmp_fix(*rule1, *rule2, fix1, fix2))
     {
         // If we already applied an identical fix as part of another correction, skip
         // any re-application.
-        if fix.edits().iter().all(|edit| applied.contains(edit)) {
+        if applied.contains(&fix) {
             *fixed.entry(rule).or_default() += 1;
             continue;
         }
 
         // Best-effort approach: if this fix overlaps with a fix we've already applied,
         // skip it.
-        if last_pos.map_or(false, |last_pos| {
-            fix.location()
-                .map_or(false, |fix_location| last_pos >= fix_location)
-        }) {
+        if last_pos.map_or(false, |last_pos| last_pos >= fix.location) {
             continue;
         }
 
-        for edit in fix.edits() {
-            // Add all contents from `last_pos` to `fix.location`.
-            let slice = locator.slice(Range::new(last_pos.unwrap_or_default(), edit.location));
-            output.push_str(slice);
+        // Add all contents from `last_pos` to `fix.location`.
+        let slice = locator.slice(Range::new(last_pos.unwrap_or_default(), fix.location));
+        output.push_str(slice);
 
-            // Add the patch itself.
-            output.push_str(&edit.content);
-
-            // Track that the edit was applied.
-            last_pos = Some(edit.end_location);
-            applied.insert(edit);
-        }
+        // Add the patch itself.
+        output.push_str(&fix.content);
 
+        // Track that the fix was applied.
+        last_pos = Some(fix.end_location);
+        applied.insert(fix);
         *fixed.entry(rule).or_default() += 1;
     }
 
@@ -81,10 +74,28 @@ fn apply_fixes<'a>(
     (output, fixed)
 }
 
+/// Apply a single fix.
+pub(crate) fn apply_fix(fix: &Fix, locator: &Locator) -> String {
+    let mut output = String::with_capacity(locator.len());
+
+    // Add all contents from `last_pos` to `fix.location`.
+    let slice = locator.slice(Range::new(Location::new(1, 0), fix.location));
+    output.push_str(slice);
+
+    // Add the patch itself.
+    output.push_str(&fix.content);
+
+    // Add the remaining content.
+    let slice = locator.skip(fix.end_location);
+    output.push_str(slice);
+
+    output
+}
+
 /// Compare two fixes.
 fn cmp_fix(rule1: Rule, rule2: Rule, fix1: &Fix, fix2: &Fix) -> std::cmp::Ordering {
-    fix1.location()
-        .cmp(&fix2.location())
+    fix1.location
+        .cmp(&fix2.location)
         .then_with(|| match (&rule1, &rule2) {
             // Apply `EndsInPeriod` fixes before `NewLineAfterLastParagraph` fixes.
             (Rule::EndsInPeriod, Rule::NewLineAfterLastParagraph) => std::cmp::Ordering::Less,
@@ -98,20 +109,21 @@ mod tests {
     use rustpython_parser::ast::Location;
 
     use ruff_diagnostics::Diagnostic;
-    use ruff_diagnostics::Edit;
+    use ruff_diagnostics::Fix;
     use ruff_python_ast::source_code::Locator;
 
-    use crate::autofix::apply_fixes;
+    use crate::autofix::{apply_fix, apply_fixes};
     use crate::rules::pycodestyle::rules::MissingNewlineAtEndOfFile;
 
-    fn create_diagnostics(edit: impl IntoIterator<Item = Edit>) -> Vec<Diagnostic> {
-        edit.into_iter()
-            .map(|edit| Diagnostic {
+    fn create_diagnostics(fixes: impl IntoIterator<Item = Fix>) -> Vec<Diagnostic> {
+        fixes
+            .into_iter()
+            .map(|fix| Diagnostic {
                 // The choice of rule here is arbitrary.
                 kind: MissingNewlineAtEndOfFile.into(),
-                location: edit.location,
-                end_location: edit.end_location,
-                fix: edit.into(),
+                location: fix.location,
+                end_location: fix.end_location,
+                fix: Some(fix),
                 parent: None,
             })
             .collect()
@@ -135,7 +147,7 @@ class A(object):
 "#
             .trim(),
         );
-        let diagnostics = create_diagnostics([Edit {
+        let diagnostics = create_diagnostics([Fix {
             content: "Bar".to_string(),
             location: Location::new(1, 8),
             end_location: Location::new(1, 14),
@@ -161,7 +173,7 @@ class A(object):
 "#
             .trim(),
         );
-        let diagnostics = create_diagnostics([Edit {
+        let diagnostics = create_diagnostics([Fix {
             content: String::new(),
             location: Location::new(1, 7),
             end_location: Location::new(1, 15),
@@ -188,12 +200,12 @@ class A(object, object, object):
             .trim(),
         );
         let diagnostics = create_diagnostics([
-            Edit {
+            Fix {
                 content: String::new(),
                 location: Location::new(1, 8),
                 end_location: Location::new(1, 16),
             },
-            Edit {
+            Fix {
                 content: String::new(),
                 location: Location::new(1, 22),
                 end_location: Location::new(1, 30),
@@ -222,12 +234,12 @@ class A(object):
             .trim(),
         );
         let diagnostics = create_diagnostics([
-            Edit {
+            Fix {
                 content: String::new(),
                 location: Location::new(1, 7),
                 end_location: Location::new(1, 15),
             },
-            Edit {
+            Fix {
                 content: "ignored".to_string(),
                 location: Location::new(1, 9),
                 end_location: Location::new(1, 11),
@@ -244,4 +256,31 @@ class A:
         );
         assert_eq!(fixed.values().sum::<usize>(), 1);
     }
+
+    #[test]
+    fn apply_single_fix() {
+        let locator = Locator::new(
+            r#"
+class A(object):
+    ...
+"#
+            .trim(),
+        );
+        let contents = apply_fix(
+            &Fix {
+                content: String::new(),
+                location: Location::new(1, 7),
+                end_location: Location::new(1, 15),
+            },
+            &locator,
+        );
+        assert_eq!(
+            contents,
+            r#"
+class A:
+    ...
+"#
+            .trim()
+        );
+    }
 }

crates/ruff/src/checkers/imports.rs

@@ -29,7 +29,9 @@ pub fn check_imports(
     // Extract all imports from the AST.
     let tracker = {
         let mut tracker = ImportTracker::new(locator, directives, path);
-        tracker.visit_body(python_ast);
+        for stmt in python_ast {
+            tracker.visit_stmt(stmt);
+        }
         tracker
     };
     let blocks: Vec<&Block> = tracker.iter().collect();

crates/ruff/src/checkers/logical_lines.rs

@@ -1,32 +1,42 @@
+#![allow(dead_code, unused_imports, unused_variables)]
+
+use bisection::bisect_left;
+use itertools::Itertools;
 use rustpython_parser::ast::Location;
 use rustpython_parser::lexer::LexResult;
 
-use ruff_diagnostics::{Diagnostic, Fix};
+use ruff_diagnostics::Diagnostic;
 use ruff_python_ast::source_code::{Locator, Stylist};
 use ruff_python_ast::types::Range;
 
 use crate::registry::{AsRule, Rule};
-use crate::rules::pycodestyle::rules::logical_lines::{
+use crate::rules::pycodestyle::logical_lines::{iter_logical_lines, TokenFlags};
+use crate::rules::pycodestyle::rules::{
     extraneous_whitespace, indentation, missing_whitespace, missing_whitespace_after_keyword,
     missing_whitespace_around_operator, space_around_operator, whitespace_around_keywords,
     whitespace_around_named_parameter_equals, whitespace_before_comment,
-    whitespace_before_parameters, LogicalLines, TokenFlags,
+    whitespace_before_parameters,
 };
 use crate::settings::{flags, Settings};
 
 /// Return the amount of indentation, expanding tabs to the next multiple of 8.
-fn expand_indent(line: &str) -> usize {
-    let line = line.trim_end_matches(['\n', '\r']);
-
+fn expand_indent(mut line: &str) -> usize {
+    while line.ends_with("\n\r") {
+        line = &line[..line.len() - 2];
+    }
+    if !line.contains('\t') {
+        return line.len() - line.trim_start().len();
+    }
     let mut indent = 0;
-    for c in line.bytes() {
-        match c {
-            b'\t' => indent = (indent / 8) * 8 + 8,
-            b' ' => indent += 1,
-            _ => break,
+    for c in line.chars() {
+        if c == '\t' {
+            indent = (indent / 8) * 8 + 8;
+        } else if c == ' ' {
+            indent += 1;
+        } else {
+            break;
         }
     }
-
     indent
 }
 
@@ -39,140 +49,151 @@ pub fn check_logical_lines(
 ) -> Vec<Diagnostic> {
     let mut diagnostics = vec![];
 
-    #[cfg(feature = "logical_lines")]
-    let should_fix_missing_whitespace =
-        autofix.into() && settings.rules.should_fix(Rule::MissingWhitespace);
-
-    #[cfg(not(feature = "logical_lines"))]
-    let should_fix_missing_whitespace = false;
-
-    #[cfg(feature = "logical_lines")]
-    let should_fix_whitespace_before_parameters =
-        autofix.into() && settings.rules.should_fix(Rule::WhitespaceBeforeParameters);
-
-    #[cfg(not(feature = "logical_lines"))]
-    let should_fix_whitespace_before_parameters = false;
-
+    let indent_char = stylist.indentation().as_char();
     let mut prev_line = None;
     let mut prev_indent_level = None;
-    let indent_char = stylist.indentation().as_char();
+    for line in iter_logical_lines(tokens, locator) {
+        if line.mapping.is_empty() {
+            continue;
+        }
 
-    for line in &LogicalLines::from_tokens(tokens, locator) {
-        if line.flags().contains(TokenFlags::OPERATOR) {
-            for (location, kind) in space_around_operator(&line) {
+        // Extract the indentation level.
+        let start_loc = line.mapping[0].1;
+        let start_line = locator.slice(Range::new(Location::new(start_loc.row(), 0), start_loc));
+        let indent_level = expand_indent(start_line);
+        let indent_size = 4;
+
+        // Generate mapping from logical to physical offsets.
+        let mapping_offsets = line.mapping.iter().map(|(offset, _)| *offset).collect_vec();
+
+        if line.flags.contains(TokenFlags::OPERATOR) {
+            for (index, kind) in space_around_operator(&line.text) {
+                let (token_offset, pos) = line.mapping[bisect_left(&mapping_offsets, &index)];
+                let location = Location::new(pos.row(), pos.column() + index - token_offset);
                 if settings.rules.enabled(kind.rule()) {
                     diagnostics.push(Diagnostic {
                         kind,
                         location,
                         end_location: location,
-                        fix: Fix::empty(),
+                        fix: None,
                         parent: None,
                     });
                 }
             }
-
-            for (location, kind) in whitespace_around_named_parameter_equals(&line.tokens()) {
-                if settings.rules.enabled(kind.rule()) {
-                    diagnostics.push(Diagnostic {
-                        kind,
-                        location,
-                        end_location: location,
-                        fix: Fix::empty(),
-                        parent: None,
-                    });
-                }
-            }
-            for (location, kind) in missing_whitespace_around_operator(&line.tokens()) {
-                if settings.rules.enabled(kind.rule()) {
-                    diagnostics.push(Diagnostic {
-                        kind,
-                        location,
-                        end_location: location,
-                        fix: Fix::empty(),
-                        parent: None,
-                    });
-                }
-            }
-
-            for diagnostic in missing_whitespace(&line, should_fix_missing_whitespace) {
-                if settings.rules.enabled(diagnostic.kind.rule()) {
-                    diagnostics.push(diagnostic);
-                }
-            }
         }
         if line
-            .flags()
+            .flags
             .contains(TokenFlags::OPERATOR | TokenFlags::PUNCTUATION)
         {
-            for (location, kind) in extraneous_whitespace(&line) {
+            for (index, kind) in extraneous_whitespace(&line.text) {
+                let (token_offset, pos) = line.mapping[bisect_left(&mapping_offsets, &index)];
+                let location = Location::new(pos.row(), pos.column() + index - token_offset);
                 if settings.rules.enabled(kind.rule()) {
                     diagnostics.push(Diagnostic {
                         kind,
                         location,
                         end_location: location,
-                        fix: Fix::empty(),
+                        fix: None,
                         parent: None,
                     });
                 }
             }
         }
-        if line.flags().contains(TokenFlags::KEYWORD) {
-            for (location, kind) in whitespace_around_keywords(&line) {
+        if line.flags.contains(TokenFlags::KEYWORD) {
+            for (index, kind) in whitespace_around_keywords(&line.text) {
+                let (token_offset, pos) = line.mapping[bisect_left(&mapping_offsets, &index)];
+                let location = Location::new(pos.row(), pos.column() + index - token_offset);
                 if settings.rules.enabled(kind.rule()) {
                     diagnostics.push(Diagnostic {
                         kind,
                         location,
                         end_location: location,
-                        fix: Fix::empty(),
+                        fix: None,
                         parent: None,
                     });
                 }
             }
 
-            for (location, kind) in missing_whitespace_after_keyword(&line.tokens()) {
+            for (location, kind) in missing_whitespace_after_keyword(&line.tokens) {
                 if settings.rules.enabled(kind.rule()) {
                     diagnostics.push(Diagnostic {
                         kind,
                         location,
                         end_location: location,
-                        fix: Fix::empty(),
+                        fix: None,
                         parent: None,
                     });
                 }
             }
         }
-        if line.flags().contains(TokenFlags::COMMENT) {
-            for (range, kind) in whitespace_before_comment(&line.tokens(), locator) {
+        if line.flags.contains(TokenFlags::COMMENT) {
+            for (range, kind) in whitespace_before_comment(&line.tokens, locator) {
                 if settings.rules.enabled(kind.rule()) {
                     diagnostics.push(Diagnostic {
                         kind,
                         location: range.location,
                         end_location: range.end_location,
-                        fix: Fix::empty(),
+                        fix: None,
                         parent: None,
                     });
                 }
             }
         }
+        if line.flags.contains(TokenFlags::OPERATOR) {
+            for (location, kind) in
+                whitespace_around_named_parameter_equals(&line.tokens, &line.text)
+            {
+                if settings.rules.enabled(kind.rule()) {
+                    diagnostics.push(Diagnostic {
+                        kind,
+                        location,
+                        end_location: location,
+                        fix: None,
+                        parent: None,
+                    });
+                }
+            }
+            for (location, kind) in missing_whitespace_around_operator(&line.tokens) {
+                if settings.rules.enabled(kind.rule()) {
+                    diagnostics.push(Diagnostic {
+                        kind,
+                        location,
+                        end_location: location,
+                        fix: None,
+                        parent: None,
+                    });
+                }
+            }
 
-        if line.flags().contains(TokenFlags::BRACKET) {
-            for diagnostic in whitespace_before_parameters(
-                &line.tokens(),
-                should_fix_whitespace_before_parameters,
-            ) {
+            #[cfg(debug_assertions)]
+            let should_fix = autofix.into() && settings.rules.should_fix(Rule::MissingWhitespace);
+
+            #[cfg(not(debug_assertions))]
+            let should_fix = false;
+
+            for diagnostic in missing_whitespace(&line.text, start_loc.row(), should_fix) {
                 if settings.rules.enabled(diagnostic.kind.rule()) {
                     diagnostics.push(diagnostic);
                 }
             }
         }
 
-        // Extract the indentation level.
-        let Some(start_loc) = line.first_token_location() else { continue; };
-        let start_line = locator.slice(Range::new(Location::new(start_loc.row(), 0), start_loc));
-        let indent_level = expand_indent(start_line);
-        let indent_size = 4;
+        if line.flags.contains(TokenFlags::BRACKET) {
+            #[cfg(debug_assertions)]
+            let should_fix =
+                autofix.into() && settings.rules.should_fix(Rule::WhitespaceBeforeParameters);
 
-        for (location, kind) in indentation(
+            #[cfg(not(debug_assertions))]
+            let should_fix = false;
+
+            for diagnostic in whitespace_before_parameters(&line.tokens, should_fix) {
+                if settings.rules.enabled(diagnostic.kind.rule()) {
+                    diagnostics.push(diagnostic);
+                }
+            }
+        }
+
+        for (index, kind) in indentation(
             &line,
             prev_line.as_ref(),
             indent_char,
@@ -180,18 +201,20 @@ pub fn check_logical_lines(
             prev_indent_level,
             indent_size,
         ) {
+            let (token_offset, pos) = line.mapping[bisect_left(&mapping_offsets, &index)];
+            let location = Location::new(pos.row(), pos.column() + index - token_offset);
             if settings.rules.enabled(kind.rule()) {
                 diagnostics.push(Diagnostic {
                     kind,
                     location,
                     end_location: location,
-                    fix: Fix::empty(),
+                    fix: None,
                     parent: None,
                 });
             }
         }
 
-        if !line.is_comment_only() {
+        if !line.is_comment() {
             prev_line = Some(line);
             prev_indent_level = Some(indent_level);
         }
@@ -204,9 +227,10 @@ mod tests {
     use rustpython_parser::lexer::LexResult;
     use rustpython_parser::{lexer, Mode};
 
-    use crate::rules::pycodestyle::rules::logical_lines::LogicalLines;
     use ruff_python_ast::source_code::Locator;
 
+    use crate::checkers::logical_lines::iter_logical_lines;
+
     #[test]
     fn split_logical_lines() {
         let contents = r#"
@@ -215,9 +239,9 @@ y = 2
 z = x + 1"#;
         let lxr: Vec<LexResult> = lexer::lex(contents, Mode::Module).collect();
         let locator = Locator::new(contents);
-        let actual: Vec<String> = LogicalLines::from_tokens(&lxr, &locator)
+        let actual: Vec<String> = iter_logical_lines(&lxr, &locator)
             .into_iter()
-            .map(|line| line.text_trimmed().to_string())
+            .map(|line| line.text)
             .collect();
         let expected = vec![
             "x = 1".to_string(),
@@ -236,12 +260,12 @@ y = 2
 z = x + 1"#;
         let lxr: Vec<LexResult> = lexer::lex(contents, Mode::Module).collect();
         let locator = Locator::new(contents);
-        let actual: Vec<String> = LogicalLines::from_tokens(&lxr, &locator)
+        let actual: Vec<String> = iter_logical_lines(&lxr, &locator)
             .into_iter()
-            .map(|line| line.text_trimmed().to_string())
+            .map(|line| line.text)
             .collect();
         let expected = vec![
-            "x = [\n  1,\n  2,\n  3,\n]".to_string(),
+            "x = [1, 2, 3, ]".to_string(),
             "y = 2".to_string(),
             "z = x + 1".to_string(),
         ];
@@ -250,11 +274,11 @@ z = x + 1"#;
         let contents = "x = 'abc'";
         let lxr: Vec<LexResult> = lexer::lex(contents, Mode::Module).collect();
         let locator = Locator::new(contents);
-        let actual: Vec<String> = LogicalLines::from_tokens(&lxr, &locator)
+        let actual: Vec<String> = iter_logical_lines(&lxr, &locator)
             .into_iter()
-            .map(|line| line.text_trimmed().to_string())
+            .map(|line| line.text)
             .collect();
-        let expected = vec!["x = 'abc'".to_string()];
+        let expected = vec!["x = \"xxx\"".to_string()];
         assert_eq!(actual, expected);
 
         let contents = r#"
@@ -263,9 +287,9 @@ def f():
 f()"#;
         let lxr: Vec<LexResult> = lexer::lex(contents, Mode::Module).collect();
         let locator = Locator::new(contents);
-        let actual: Vec<String> = LogicalLines::from_tokens(&lxr, &locator)
+        let actual: Vec<String> = iter_logical_lines(&lxr, &locator)
             .into_iter()
-            .map(|line| line.text_trimmed().to_string())
+            .map(|line| line.text)
             .collect();
         let expected = vec!["def f():", "x = 1", "f()"];
         assert_eq!(actual, expected);
@@ -278,17 +302,11 @@ def f():
 f()"#;
         let lxr: Vec<LexResult> = lexer::lex(contents, Mode::Module).collect();
         let locator = Locator::new(contents);
-        let actual: Vec<String> = LogicalLines::from_tokens(&lxr, &locator)
+        let actual: Vec<String> = iter_logical_lines(&lxr, &locator)
             .into_iter()
-            .map(|line| line.text_trimmed().to_string())
+            .map(|line| line.text)
             .collect();
-        let expected = vec![
-            "def f():",
-            "\"\"\"Docstring goes here.\"\"\"",
-            "",
-            "x = 1",
-            "f()",
-        ];
+        let expected = vec!["def f():", "\"xxxxxxxxxxxxxxxxxxxx\"", "", "x = 1", "f()"];
         assert_eq!(actual, expected);
     }
 }

crates/ruff/src/checkers/mod.rs

@@ -1,8 +1,7 @@
 pub mod ast;
 pub mod filesystem;
 pub mod imports;
-#[cfg(feature = "logical_lines")]
-pub(crate) mod logical_lines;
+pub mod logical_lines;
 pub mod noqa;
 pub mod physical_lines;
 pub mod tokens;

crates/ruff/src/checkers/noqa.rs

@@ -1,15 +1,16 @@
 //! `NoQA` enforcement and validation.
 
+use log::warn;
 use nohash_hasher::IntMap;
 use rustpython_parser::ast::Location;
 
-use ruff_diagnostics::{Diagnostic, Edit};
+use ruff_diagnostics::{Diagnostic, Fix};
 use ruff_python_ast::newlines::StrExt;
 use ruff_python_ast::types::Range;
 
 use crate::codes::NoqaCode;
 use crate::noqa;
-use crate::noqa::{Directive, FileExemption};
+use crate::noqa::{extract_file_exemption, Directive, Exemption};
 use crate::registry::{AsRule, Rule};
 use crate::rule_redirects::get_redirect_target;
 use crate::rules::ruff::rules::{UnusedCodes, UnusedNOQA};
@@ -25,79 +26,92 @@ pub fn check_noqa(
 ) -> Vec<usize> {
     let enforce_noqa = settings.rules.enabled(Rule::UnusedNOQA);
 
-    let lines: Vec<&str> = contents.universal_newlines().collect();
+    // Whether the file is exempted from all checks.
+    let mut file_exempted = false;
 
-    // Identify any codes that are globally exempted (within the current file).
-    let exemption = noqa::file_exemption(&lines, commented_lines);
+    // Codes that are globally exempted (within the current file).
+    let mut file_exemptions: Vec<NoqaCode> = vec![];
 
     // Map from line number to `noqa` directive on that line, along with any codes
     // that were matched by the directive.
     let mut noqa_directives: IntMap<usize, (Directive, Vec<NoqaCode>)> = IntMap::default();
 
-    // Extract all `noqa` directives.
-    if enforce_noqa {
-        for lineno in commented_lines {
+    // Indices of diagnostics that were ignored by a `noqa` directive.
+    let mut ignored_diagnostics = vec![];
+
+    let lines: Vec<&str> = contents.universal_newlines().collect();
+    for lineno in commented_lines {
+        match extract_file_exemption(lines[lineno - 1]) {
+            Exemption::All => {
+                file_exempted = true;
+            }
+            Exemption::Codes(codes) => {
+                file_exemptions.extend(codes.into_iter().filter_map(|code| {
+                    if let Ok(rule) = Rule::from_code(get_redirect_target(code).unwrap_or(code)) {
+                        Some(rule.noqa_code())
+                    } else {
+                        warn!("Invalid code provided to `# ruff: noqa`: {}", code);
+                        None
+                    }
+                }));
+            }
+            Exemption::None => {}
+        }
+
+        if enforce_noqa {
             noqa_directives
                 .entry(lineno - 1)
                 .or_insert_with(|| (noqa::extract_noqa_directive(lines[lineno - 1]), vec![]));
         }
     }
 
-    // Indices of diagnostics that were ignored by a `noqa` directive.
-    let mut ignored_diagnostics = vec![];
-
     // Remove any ignored diagnostics.
     for (index, diagnostic) in diagnostics.iter().enumerate() {
         if matches!(diagnostic.kind.rule(), Rule::BlanketNOQA) {
             continue;
         }
 
-        match &exemption {
-            FileExemption::All => {
-                // If the file is exempted, ignore all diagnostics.
+        // If the file is exempted, ignore all diagnostics.
+        if file_exempted {
+            ignored_diagnostics.push(index);
+            continue;
+        }
+
+        // If the diagnostic is ignored by a global exemption, ignore it.
+        if !file_exemptions.is_empty() {
+            if file_exemptions.contains(&diagnostic.kind.rule().noqa_code()) {
                 ignored_diagnostics.push(index);
                 continue;
             }
-            FileExemption::Codes(codes) => {
-                // If the diagnostic is ignored by a global exemption, ignore it.
-                if codes.contains(&diagnostic.kind.rule().noqa_code()) {
-                    ignored_diagnostics.push(index);
-                    continue;
-                }
-            }
-            FileExemption::None => {}
         }
 
-        let diagnostic_lineno = diagnostic.location.row();
-
         // Is the violation ignored by a `noqa` directive on the parent line?
         if let Some(parent_lineno) = diagnostic.parent.map(|location| location.row()) {
-            if parent_lineno != diagnostic_lineno {
-                let noqa_lineno = noqa_line_for.get(&parent_lineno).unwrap_or(&parent_lineno);
-                if commented_lines.contains(noqa_lineno) {
-                    let noqa = noqa_directives.entry(noqa_lineno - 1).or_insert_with(|| {
-                        (noqa::extract_noqa_directive(lines[noqa_lineno - 1]), vec![])
-                    });
-                    match noqa {
-                        (Directive::All(..), matches) => {
+            let noqa_lineno = noqa_line_for.get(&parent_lineno).unwrap_or(&parent_lineno);
+            if commented_lines.contains(noqa_lineno) {
+                let noqa = noqa_directives.entry(noqa_lineno - 1).or_insert_with(|| {
+                    (noqa::extract_noqa_directive(lines[noqa_lineno - 1]), vec![])
+                });
+                match noqa {
+                    (Directive::All(..), matches) => {
+                        matches.push(diagnostic.kind.rule().noqa_code());
+                        ignored_diagnostics.push(index);
+                        continue;
+                    }
+                    (Directive::Codes(.., codes, _), matches) => {
+                        if noqa::includes(diagnostic.kind.rule(), codes) {
                             matches.push(diagnostic.kind.rule().noqa_code());
                             ignored_diagnostics.push(index);
                             continue;
                         }
-                        (Directive::Codes(.., codes, _), matches) => {
-                            if noqa::includes(diagnostic.kind.rule(), codes) {
-                                matches.push(diagnostic.kind.rule().noqa_code());
-                                ignored_diagnostics.push(index);
-                                continue;
-                            }
-                        }
-                        (Directive::None, ..) => {}
                     }
+                    (Directive::None, ..) => {}
                 }
             }
         }
 
         // Is the diagnostic ignored by a `noqa` directive on the same line?
+        let diagnostic_lineno = diagnostic.location.row();
         let noqa_lineno = noqa_line_for
             .get(&diagnostic_lineno)
             .unwrap_or(&diagnostic_lineno);
@@ -141,7 +155,7 @@ pub fn check_noqa(
                             ),
                         );
                         if autofix.into() && settings.rules.should_fix(diagnostic.kind.rule()) {
-                            diagnostic.set_fix(delete_noqa(
+                            diagnostic.amend(delete_noqa(
                                 row,
                                 lines[row],
                                 leading_spaces,
@@ -217,7 +231,7 @@ pub fn check_noqa(
                         );
                         if autofix.into() && settings.rules.should_fix(diagnostic.kind.rule()) {
                             if valid_codes.is_empty() {
-                                diagnostic.set_fix(delete_noqa(
+                                diagnostic.amend(delete_noqa(
                                     row,
                                     lines[row],
                                     leading_spaces,
@@ -226,7 +240,7 @@ pub fn check_noqa(
                                     trailing_spaces,
                                 ));
                             } else {
-                                diagnostic.set_fix(Edit::replacement(
+                                diagnostic.amend(Fix::replacement(
                                     format!("# noqa: {}", valid_codes.join(", ")),
                                     Location::new(row + 1, start_char),
                                     Location::new(row + 1, end_char),
@@ -245,7 +259,7 @@ pub fn check_noqa(
     ignored_diagnostics
 }
 
-/// Generate a [`Edit`] to delete a `noqa` directive.
+/// Generate a [`Fix`] to delete a `noqa` directive.
 fn delete_noqa(
     row: usize,
     line: &str,
@@ -253,15 +267,15 @@ fn delete_noqa(
     start_byte: usize,
     end_byte: usize,
     trailing_spaces: usize,
-) -> Edit {
+) -> Fix {
     if start_byte - leading_spaces == 0 && end_byte == line.len() {
         // Ex) `# noqa`
-        Edit::deletion(Location::new(row + 1, 0), Location::new(row + 2, 0))
+        Fix::deletion(Location::new(row + 1, 0), Location::new(row + 2, 0))
     } else if end_byte == line.len() {
         // Ex) `x = 1  # noqa`
         let start_char = line[..start_byte].chars().count();
         let end_char = start_char + line[start_byte..end_byte].chars().count();
-        Edit::deletion(
+        Fix::deletion(
             Location::new(row + 1, start_char - leading_spaces),
             Location::new(row + 1, end_char + trailing_spaces),
         )
@@ -269,7 +283,7 @@ fn delete_noqa(
         // Ex) `x = 1  # noqa  # type: ignore`
         let start_char = line[..start_byte].chars().count();
         let end_char = start_char + line[start_byte..end_byte].chars().count();
-        Edit::deletion(
+        Fix::deletion(
             Location::new(row + 1, start_char),
             Location::new(row + 1, end_char + trailing_spaces),
         )
@@ -277,7 +291,7 @@ fn delete_noqa(
         // Ex) `x = 1  # noqa here`
         let start_char = line[..start_byte].chars().count();
         let end_char = start_char + line[start_byte..end_byte].chars().count();
-        Edit::deletion(
+        Fix::deletion(
             Location::new(row + 1, start_char + 1 + 1),
             Location::new(row + 1, end_char + trailing_spaces),
         )

crates/ruff/src/checkers/physical_lines.rs

@@ -182,8 +182,6 @@ pub fn check_physical_lines(
 
 #[cfg(test)]
 mod tests {
-    use rustpython_parser::lexer::lex;
-    use rustpython_parser::Mode;
     use std::path::Path;
 
     use ruff_python_ast::source_code::{Locator, Stylist};
@@ -197,8 +195,7 @@ mod tests {
     fn e501_non_ascii_char() {
         let line = "'\u{4e9c}' * 2"; // 7 in UTF-32, 9 in UTF-8.
         let locator = Locator::new(line);
-        let tokens: Vec<_> = lex(line, Mode::Module).collect();
-        let stylist = Stylist::from_tokens(&tokens, &locator);
+        let stylist = Stylist::from_contents(line, &locator);
 
         let check_with_max_line_length = |line_length: usize| {
             check_physical_lines(
@@ -214,7 +211,7 @@ mod tests {
                 flags::Autofix::Enabled,
             )
         };
-        assert_eq!(check_with_max_line_length(8), vec![]);
-        assert_eq!(check_with_max_line_length(8), vec![]);
+        assert!(!check_with_max_line_length(6).is_empty());
+        assert!(check_with_max_line_length(7).is_empty());
     }
 }

crates/ruff/src/codes.rs

@@ -26,67 +26,67 @@ pub fn code_to_rule(linter: Linter, code: &str) -> Option<Rule> {
     Some(match (linter, code) {
         // pycodestyle errors
         (Pycodestyle, "E101") => Rule::MixedSpacesAndTabs,
-        #[cfg(feature = "logical_lines")]
+        #[cfg(debug_assertions)]
         (Pycodestyle, "E111") => Rule::IndentationWithInvalidMultiple,
-        #[cfg(feature = "logical_lines")]
+        #[cfg(debug_assertions)]
         (Pycodestyle, "E112") => Rule::NoIndentedBlock,
-        #[cfg(feature = "logical_lines")]
+        #[cfg(debug_assertions)]
         (Pycodestyle, "E113") => Rule::UnexpectedIndentation,
-        #[cfg(feature = "logical_lines")]
+        #[cfg(debug_assertions)]
         (Pycodestyle, "E114") => Rule::IndentationWithInvalidMultipleComment,
-        #[cfg(feature = "logical_lines")]
+        #[cfg(debug_assertions)]
         (Pycodestyle, "E115") => Rule::NoIndentedBlockComment,
-        #[cfg(feature = "logical_lines")]
+        #[cfg(debug_assertions)]
         (Pycodestyle, "E116") => Rule::UnexpectedIndentationComment,
-        #[cfg(feature = "logical_lines")]
+        #[cfg(debug_assertions)]
         (Pycodestyle, "E117") => Rule::OverIndented,
-        #[cfg(feature = "logical_lines")]
+        #[cfg(debug_assertions)]
         (Pycodestyle, "E201") => Rule::WhitespaceAfterOpenBracket,
-        #[cfg(feature = "logical_lines")]
+        #[cfg(debug_assertions)]
         (Pycodestyle, "E202") => Rule::WhitespaceBeforeCloseBracket,
-        #[cfg(feature = "logical_lines")]
+        #[cfg(debug_assertions)]
         (Pycodestyle, "E203") => Rule::WhitespaceBeforePunctuation,
-        #[cfg(feature = "logical_lines")]
+        #[cfg(debug_assertions)]
         (Pycodestyle, "E211") => Rule::WhitespaceBeforeParameters,
-        #[cfg(feature = "logical_lines")]
+        #[cfg(debug_assertions)]
         (Pycodestyle, "E221") => Rule::MultipleSpacesBeforeOperator,
-        #[cfg(feature = "logical_lines")]
+        #[cfg(debug_assertions)]
         (Pycodestyle, "E222") => Rule::MultipleSpacesAfterOperator,
-        #[cfg(feature = "logical_lines")]
+        #[cfg(debug_assertions)]
         (Pycodestyle, "E223") => Rule::TabBeforeOperator,
-        #[cfg(feature = "logical_lines")]
+        #[cfg(debug_assertions)]
         (Pycodestyle, "E224") => Rule::TabAfterOperator,
-        #[cfg(feature = "logical_lines")]
+        #[cfg(debug_assertions)]
         (Pycodestyle, "E225") => Rule::MissingWhitespaceAroundOperator,
-        #[cfg(feature = "logical_lines")]
+        #[cfg(debug_assertions)]
         (Pycodestyle, "E226") => Rule::MissingWhitespaceAroundArithmeticOperator,
-        #[cfg(feature = "logical_lines")]
+        #[cfg(debug_assertions)]
         (Pycodestyle, "E227") => Rule::MissingWhitespaceAroundBitwiseOrShiftOperator,
-        #[cfg(feature = "logical_lines")]
+        #[cfg(debug_assertions)]
         (Pycodestyle, "E228") => Rule::MissingWhitespaceAroundModuloOperator,
-        #[cfg(feature = "logical_lines")]
+        #[cfg(debug_assertions)]
         (Pycodestyle, "E231") => Rule::MissingWhitespace,
-        #[cfg(feature = "logical_lines")]
+        #[cfg(debug_assertions)]
         (Pycodestyle, "E251") => Rule::UnexpectedSpacesAroundKeywordParameterEquals,
-        #[cfg(feature = "logical_lines")]
+        #[cfg(debug_assertions)]
         (Pycodestyle, "E252") => Rule::MissingWhitespaceAroundParameterEquals,
-        #[cfg(feature = "logical_lines")]
+        #[cfg(debug_assertions)]
         (Pycodestyle, "E261") => Rule::TooFewSpacesBeforeInlineComment,
-        #[cfg(feature = "logical_lines")]
+        #[cfg(debug_assertions)]
         (Pycodestyle, "E262") => Rule::NoSpaceAfterInlineComment,
-        #[cfg(feature = "logical_lines")]
+        #[cfg(debug_assertions)]
         (Pycodestyle, "E265") => Rule::NoSpaceAfterBlockComment,
-        #[cfg(feature = "logical_lines")]
+        #[cfg(debug_assertions)]
         (Pycodestyle, "E266") => Rule::MultipleLeadingHashesForBlockComment,
-        #[cfg(feature = "logical_lines")]
+        #[cfg(debug_assertions)]
         (Pycodestyle, "E271") => Rule::MultipleSpacesAfterKeyword,
-        #[cfg(feature = "logical_lines")]
+        #[cfg(debug_assertions)]
         (Pycodestyle, "E272") => Rule::MultipleSpacesBeforeKeyword,
-        #[cfg(feature = "logical_lines")]
+        #[cfg(debug_assertions)]
         (Pycodestyle, "E273") => Rule::TabAfterKeyword,
-        #[cfg(feature = "logical_lines")]
+        #[cfg(debug_assertions)]
         (Pycodestyle, "E274") => Rule::TabBeforeKeyword,
-        #[cfg(feature = "logical_lines")]
+        #[cfg(debug_assertions)]
         (Pycodestyle, "E275") => Rule::MissingWhitespaceAfterKeyword,
         (Pycodestyle, "E401") => Rule::MultipleImportsOnOneLine,
         (Pycodestyle, "E402") => Rule::ModuleImportNotAtTopOfFile,
@@ -168,7 +168,7 @@ pub fn code_to_rule(linter: Linter, code: &str) -> Option<Rule> {
         (Pylint, "E0101") => Rule::ReturnInInit,
         (Pylint, "E0116") => Rule::ContinueInFinally,
         (Pylint, "E0117") => Rule::NonlocalWithoutBinding,
-        (Pylint, "E0118") => Rule::LoadBeforeGlobalDeclaration,
+        (Pylint, "E0118") => Rule::UsePriorToGlobalDeclaration,
         (Pylint, "E0604") => Rule::InvalidAllObject,
         (Pylint, "E0605") => Rule::InvalidAllFormat,
         (Pylint, "E1142") => Rule::AwaitOutsideAsync,
@@ -238,7 +238,6 @@ pub fn code_to_rule(linter: Linter, code: &str) -> Option<Rule> {
         (Flake8Bugbear, "028") => Rule::NoExplicitStacklevel,
         (Flake8Bugbear, "029") => Rule::ExceptWithEmptyTuple,
         (Flake8Bugbear, "030") => Rule::ExceptWithNonExceptionClasses,
-        (Flake8Bugbear, "031") => Rule::ReuseOfGroupbyGenerator,
         (Flake8Bugbear, "032") => Rule::UnintentionalTypeAnnotation,
         (Flake8Bugbear, "904") => Rule::RaiseWithoutFromInsideExcept,
         (Flake8Bugbear, "905") => Rule::ZipWithoutExplicitStrict,
@@ -284,11 +283,6 @@ pub fn code_to_rule(linter: Linter, code: &str) -> Option<Rule> {
         (Flake8Return, "507") => Rule::SuperfluousElseContinue,
         (Flake8Return, "508") => Rule::SuperfluousElseBreak,
 
-        // flake8-gettext
-        (Flake8GetText, "001") => Rule::FStringInGetTextFuncCall,
-        (Flake8GetText, "002") => Rule::FormatInGetTextFuncCall,
-        (Flake8GetText, "003") => Rule::PrintfInGetTextFuncCall,
-
         // flake8-implicit-str-concat
         (Flake8ImplicitStrConcat, "001") => Rule::SingleLineImplicitStringConcatenation,
         (Flake8ImplicitStrConcat, "002") => Rule::MultiLineImplicitStringConcatenation,
@@ -333,7 +327,7 @@ pub fn code_to_rule(linter: Linter, code: &str) -> Option<Rule> {
         (Flake8Simplify, "101") => Rule::DuplicateIsinstanceCall,
         (Flake8Simplify, "102") => Rule::CollapsibleIf,
         (Flake8Simplify, "103") => Rule::NeedlessBool,
-        (Flake8Simplify, "105") => Rule::SuppressibleException,
+        (Flake8Simplify, "105") => Rule::UseContextlibSuppress,
         (Flake8Simplify, "107") => Rule::ReturnInTryExceptFinally,
         (Flake8Simplify, "108") => Rule::IfElseBlockInsteadOfIfExp,
         (Flake8Simplify, "109") => Rule::CompareWithTuple,
@@ -469,6 +463,7 @@ pub fn code_to_rule(linter: Linter, code: &str) -> Option<Rule> {
         (Eradicate, "001") => Rule::CommentedOutCode,
 
         // flake8-bandit
+        (Flake8Bandit, "001") => Rule::DeniedFunctionCall,
         (Flake8Bandit, "101") => Rule::Assert,
         (Flake8Bandit, "102") => Rule::ExecBuiltin,
         (Flake8Bandit, "103") => Rule::BadFilePermissions,
@@ -476,37 +471,16 @@ pub fn code_to_rule(linter: Linter, code: &str) -> Option<Rule> {
         (Flake8Bandit, "105") => Rule::HardcodedPasswordString,
         (Flake8Bandit, "106") => Rule::HardcodedPasswordFuncArg,
         (Flake8Bandit, "107") => Rule::HardcodedPasswordDefault,
+        (Flake8Bandit, "608") => Rule::HardcodedSQLExpression,
         (Flake8Bandit, "108") => Rule::HardcodedTempFile,
         (Flake8Bandit, "110") => Rule::TryExceptPass,
         (Flake8Bandit, "112") => Rule::TryExceptContinue,
         (Flake8Bandit, "113") => Rule::RequestWithoutTimeout,
-        (Flake8Bandit, "301") => Rule::SuspiciousPickleUsage,
-        (Flake8Bandit, "302") => Rule::SuspiciousMarshalUsage,
-        (Flake8Bandit, "303") => Rule::SuspiciousInsecureHashUsage,
-        (Flake8Bandit, "304") => Rule::SuspiciousInsecureCipherUsage,
-        (Flake8Bandit, "305") => Rule::SuspiciousInsecureCipherModeUsage,
-        (Flake8Bandit, "306") => Rule::SuspiciousMktempUsage,
-        (Flake8Bandit, "307") => Rule::SuspiciousEvalUsage,
-        (Flake8Bandit, "308") => Rule::SuspiciousMarkSafeUsage,
-        (Flake8Bandit, "310") => Rule::SuspiciousURLOpenUsage,
-        (Flake8Bandit, "311") => Rule::SuspiciousNonCryptographicRandomUsage,
-        (Flake8Bandit, "312") => Rule::SuspiciousTelnetUsage,
-        (Flake8Bandit, "313") => Rule::SuspiciousXMLCElementTreeUsage,
-        (Flake8Bandit, "314") => Rule::SuspiciousXMLElementTreeUsage,
-        (Flake8Bandit, "315") => Rule::SuspiciousXMLExpatReaderUsage,
-        (Flake8Bandit, "316") => Rule::SuspiciousXMLExpatBuilderUsage,
-        (Flake8Bandit, "317") => Rule::SuspiciousXMLSaxUsage,
-        (Flake8Bandit, "318") => Rule::SuspiciousXMLMiniDOMUsage,
-        (Flake8Bandit, "319") => Rule::SuspiciousXMLPullDOMUsage,
-        (Flake8Bandit, "320") => Rule::SuspiciousXMLETreeUsage,
-        (Flake8Bandit, "321") => Rule::SuspiciousFTPLibUsage,
-        (Flake8Bandit, "323") => Rule::SuspiciousUnverifiedContextUsage,
         (Flake8Bandit, "324") => Rule::HashlibInsecureHashFunction,
         (Flake8Bandit, "501") => Rule::RequestWithNoCertValidation,
         (Flake8Bandit, "506") => Rule::UnsafeYAMLLoad,
         (Flake8Bandit, "508") => Rule::SnmpInsecureVersion,
         (Flake8Bandit, "509") => Rule::SnmpWeakCryptography,
-        (Flake8Bandit, "608") => Rule::HardcodedSQLExpression,
         (Flake8Bandit, "612") => Rule::LoggingConfigInsecureListen,
         (Flake8Bandit, "701") => Rule::Jinja2AutoescapeFalse,
 
@@ -569,9 +543,7 @@ pub fn code_to_rule(linter: Linter, code: &str) -> Option<Rule> {
         (Flake8Pyi, "009") => Rule::PassStatementStubBody,
         (Flake8Pyi, "010") => Rule::NonEmptyStubBody,
         (Flake8Pyi, "011") => Rule::TypedArgumentDefaultInStub,
-        (Flake8Pyi, "012") => Rule::PassInClassBody,
         (Flake8Pyi, "014") => Rule::ArgumentDefaultInStub,
-        (Flake8Pyi, "015") => Rule::AssignmentDefaultInStub,
         (Flake8Pyi, "021") => Rule::DocstringInStub,
         (Flake8Pyi, "033") => Rule::TypeCommentInStub,
 
@@ -646,31 +618,31 @@ pub fn code_to_rule(linter: Linter, code: &str) -> Option<Rule> {
         (Tryceratops, "401") => Rule::VerboseLogMessage,
 
         // flake8-use-pathlib
-        (Flake8UsePathlib, "100") => Rule::OsPathAbspath,
-        (Flake8UsePathlib, "101") => Rule::OsChmod,
-        (Flake8UsePathlib, "102") => Rule::OsMkdir,
-        (Flake8UsePathlib, "103") => Rule::OsMakedirs,
-        (Flake8UsePathlib, "104") => Rule::OsRename,
+        (Flake8UsePathlib, "100") => Rule::PathlibAbspath,
+        (Flake8UsePathlib, "101") => Rule::PathlibChmod,
+        (Flake8UsePathlib, "102") => Rule::PathlibMkdir,
+        (Flake8UsePathlib, "103") => Rule::PathlibMakedirs,
+        (Flake8UsePathlib, "104") => Rule::PathlibRename,
         (Flake8UsePathlib, "105") => Rule::PathlibReplace,
-        (Flake8UsePathlib, "106") => Rule::OsRmdir,
-        (Flake8UsePathlib, "107") => Rule::OsRemove,
-        (Flake8UsePathlib, "108") => Rule::OsUnlink,
-        (Flake8UsePathlib, "109") => Rule::OsGetcwd,
-        (Flake8UsePathlib, "110") => Rule::OsPathExists,
-        (Flake8UsePathlib, "111") => Rule::OsPathExpanduser,
-        (Flake8UsePathlib, "112") => Rule::OsPathIsdir,
-        (Flake8UsePathlib, "113") => Rule::OsPathIsfile,
-        (Flake8UsePathlib, "114") => Rule::OsPathIslink,
-        (Flake8UsePathlib, "115") => Rule::OsReadlink,
-        (Flake8UsePathlib, "116") => Rule::OsStat,
-        (Flake8UsePathlib, "117") => Rule::OsPathIsabs,
-        (Flake8UsePathlib, "118") => Rule::OsPathJoin,
-        (Flake8UsePathlib, "119") => Rule::OsPathBasename,
-        (Flake8UsePathlib, "120") => Rule::OsPathDirname,
-        (Flake8UsePathlib, "121") => Rule::OsPathSamefile,
-        (Flake8UsePathlib, "122") => Rule::OsPathSplitext,
-        (Flake8UsePathlib, "123") => Rule::BuiltinOpen,
-        (Flake8UsePathlib, "124") => Rule::PyPath,
+        (Flake8UsePathlib, "106") => Rule::PathlibRmdir,
+        (Flake8UsePathlib, "107") => Rule::PathlibRemove,
+        (Flake8UsePathlib, "108") => Rule::PathlibUnlink,
+        (Flake8UsePathlib, "109") => Rule::PathlibGetcwd,
+        (Flake8UsePathlib, "110") => Rule::PathlibExists,
+        (Flake8UsePathlib, "111") => Rule::PathlibExpanduser,
+        (Flake8UsePathlib, "112") => Rule::PathlibIsDir,
+        (Flake8UsePathlib, "113") => Rule::PathlibIsFile,
+        (Flake8UsePathlib, "114") => Rule::PathlibIsLink,
+        (Flake8UsePathlib, "115") => Rule::PathlibReadlink,
+        (Flake8UsePathlib, "116") => Rule::PathlibStat,
+        (Flake8UsePathlib, "117") => Rule::PathlibIsAbs,
+        (Flake8UsePathlib, "118") => Rule::PathlibJoin,
+        (Flake8UsePathlib, "119") => Rule::PathlibBasename,
+        (Flake8UsePathlib, "120") => Rule::PathlibDirname,
+        (Flake8UsePathlib, "121") => Rule::PathlibSamefile,
+        (Flake8UsePathlib, "122") => Rule::PathlibSplitext,
+        (Flake8UsePathlib, "123") => Rule::PathlibOpen,
+        (Flake8UsePathlib, "124") => Rule::PathlibPyPath,
 
         // flake8-logging-format
         (Flake8LoggingFormat, "001") => Rule::LoggingStringFormat,
@@ -707,7 +679,6 @@ pub fn code_to_rule(linter: Linter, code: &str) -> Option<Rule> {
         (Flake8Django, "006") => Rule::DjangoExcludeWithModelForm,
         (Flake8Django, "007") => Rule::DjangoAllWithModelForm,
         (Flake8Django, "008") => Rule::DjangoModelWithoutDunderStr,
-        (Flake8Django, "012") => Rule::DjangoUnorderedBodyContentInModel,
         (Flake8Django, "013") => Rule::DjangoNonLeadingReceiverDecorator,
 
         _ => return None,

crates/ruff/src/cst/matchers.rs

@@ -1,7 +1,7 @@
 use anyhow::{bail, Result};
 use libcst_native::{
-    Attribute, Call, Comparison, Dict, Expr, Expression, Import, ImportAlias, ImportFrom,
-    ImportNames, Module, SimpleString, SmallStatement, Statement,
+    Attribute, Call, Comparison, Dict, Expr, Expression, Import, ImportFrom, Module, SimpleString,
+    SmallStatement, Statement,
 };
 
 pub fn match_module(module_text: &str) -> Result<Module> {
@@ -54,16 +54,6 @@ pub fn match_import_from<'a, 'b>(module: &'a mut Module<'b>) -> Result<&'a mut I
     }
 }
 
-pub fn match_aliases<'a, 'b>(
-    import_from: &'a mut ImportFrom<'b>,
-) -> Result<&'a mut Vec<ImportAlias<'b>>> {
-    if let ImportNames::Aliases(aliases) = &mut import_from.names {
-        Ok(aliases)
-    } else {
-        bail!("Expected ImportNames::Aliases")
-    }
-}
-
 pub fn match_call<'a, 'b>(expression: &'a mut Expression<'b>) -> Result<&'a mut Call<'b>> {
     if let Expression::Call(call) = expression {
         Ok(call)

crates/ruff/src/docstrings/definition.rs

@@ -29,13 +29,12 @@ pub struct Docstring<'a> {
     pub indentation: &'a str,
 }
 
-#[derive(Copy, Clone)]
 pub enum Documentable {
     Class,
     Function,
 }
 
-pub fn transition_scope(scope: VisibleScope, stmt: &Stmt, kind: Documentable) -> VisibleScope {
+pub fn transition_scope(scope: &VisibleScope, stmt: &Stmt, kind: &Documentable) -> VisibleScope {
     match kind {
         Documentable::Function => VisibleScope {
             modifier: Modifier::Function,

crates/ruff/src/docstrings/extraction.rs

@@ -28,10 +28,10 @@ pub fn docstring_from(suite: &[Stmt]) -> Option<&Expr> {
 
 /// Extract a `Definition` from the AST node defined by a `Stmt`.
 pub fn extract<'a>(
-    scope: VisibleScope,
+    scope: &VisibleScope,
     stmt: &'a Stmt,
     body: &'a [Stmt],
-    kind: Documentable,
+    kind: &Documentable,
 ) -> Definition<'a> {
     let expr = docstring_from(body);
     match kind {

crates/ruff/src/docstrings/sections.rs

@@ -129,7 +129,7 @@ pub(crate) struct SectionContext<'a> {
     pub(crate) original_index: usize,
 }
 
-fn suspected_as_section(line: &str, style: SectionStyle) -> Option<SectionKind> {
+fn suspected_as_section(line: &str, style: &SectionStyle) -> Option<SectionKind> {
     if let Some(kind) = SectionKind::from_str(whitespace::leading_words(line)) {
         if style.sections().contains(&kind) {
             return Some(kind);
@@ -168,7 +168,7 @@ fn is_docstring_section(context: &SectionContext) -> bool {
 /// Extract all `SectionContext` values from a docstring.
 pub(crate) fn section_contexts<'a>(
     lines: &'a [&'a str],
-    style: SectionStyle,
+    style: &SectionStyle,
 ) -> Vec<SectionContext<'a>> {
     let mut contexts = vec![];
     for (kind, lineno) in lines

crates/ruff/src/docstrings/styles.rs

@@ -2,7 +2,6 @@ use crate::docstrings::google::GOOGLE_SECTIONS;
 use crate::docstrings::numpy::NUMPY_SECTIONS;
 use crate::docstrings::sections::SectionKind;
 
-#[derive(Copy, Clone)]
 pub(crate) enum SectionStyle {
     Numpy,
     Google,

crates/ruff/src/fix.rs

@@ -0,0 +1,17 @@
+#[derive(Debug, Copy, Clone, Hash)]
+pub enum FixMode {
+    Generate,
+    Apply,
+    Diff,
+    None,
+}
+
+impl From<bool> for FixMode {
+    fn from(value: bool) -> Self {
+        if value {
+            Self::Apply
+        } else {
+            Self::None
+        }
+    }
+}

crates/ruff/src/flake8_to_ruff/converter.rs

@@ -416,7 +416,7 @@ pub fn convert(
         if let Some(src_paths) = &isort.src_paths {
             match options.src.as_mut() {
                 Some(src) => {
-                    src.extend_from_slice(src_paths);
+                    src.extend(src_paths.clone());
                 }
                 None => {
                     options.src = Some(src_paths.clone());

crates/ruff/src/flake8_to_ruff/parser.rs

@@ -54,7 +54,7 @@ struct Token {
     src: String,
 }
 
-#[derive(Debug, Copy, Clone)]
+#[derive(Debug)]
 enum TokenType {
     Code,
     File,

crates/ruff/src/flake8_to_ruff/plugin.rs

@@ -7,7 +7,7 @@ use anyhow::anyhow;
 use crate::registry::Linter;
 use crate::rule_selector::RuleSelector;
 
-#[derive(Copy, Clone, Ord, PartialOrd, Eq, PartialEq)]
+#[derive(Clone, Ord, PartialOrd, Eq, PartialEq)]
 pub enum Plugin {
     Flake82020,
     Flake8Annotations,

crates/ruff/src/importer.rs

@@ -1,359 +0,0 @@
-//! Add and modify import statements to make module members available during fix execution.
-
-use anyhow::Result;
-use libcst_native::{Codegen, CodegenState, ImportAlias, Name, NameOrAttribute};
-use rustc_hash::FxHashMap;
-use rustpython_parser::ast::{Location, Stmt, StmtKind, Suite};
-use rustpython_parser::{lexer, Mode, Tok};
-
-use ruff_diagnostics::Edit;
-use ruff_python_ast::helpers::is_docstring_stmt;
-use ruff_python_ast::imports::AnyImport;
-use ruff_python_ast::source_code::{Locator, Stylist};
-
-use crate::cst::matchers::{match_aliases, match_import_from, match_module};
-
-pub struct Importer<'a> {
-    python_ast: &'a Suite,
-    locator: &'a Locator<'a>,
-    stylist: &'a Stylist<'a>,
-    /// A map from module name to top-level `StmtKind::ImportFrom` statements.
-    import_from_map: FxHashMap<&'a str, &'a Stmt>,
-    /// The last top-level import statement.
-    trailing_import: Option<&'a Stmt>,
-}
-
-impl<'a> Importer<'a> {
-    pub fn new(python_ast: &'a Suite, locator: &'a Locator<'a>, stylist: &'a Stylist<'a>) -> Self {
-        Self {
-            python_ast,
-            locator,
-            stylist,
-            import_from_map: FxHashMap::default(),
-            trailing_import: None,
-        }
-    }
-
-    /// Visit a top-level import statement.
-    pub fn visit_import(&mut self, import: &'a Stmt) {
-        // Store a reference to the import statement in the appropriate map.
-        match &import.node {
-            StmtKind::Import { .. } => {
-                // Nothing to do here, we don't extend top-level `import` statements at all, so
-                // no need to track them.
-            }
-            StmtKind::ImportFrom { module, level, .. } => {
-                // Store a reverse-map from module name to `import ... from` statement.
-                if level.map_or(true, |level| level == 0) {
-                    if let Some(module) = module {
-                        self.import_from_map.insert(module.as_str(), import);
-                    }
-                }
-            }
-            _ => {
-                panic!("Expected StmtKind::Import | StmtKind::ImportFrom");
-            }
-        }
-
-        // Store a reference to the last top-level import statement.
-        self.trailing_import = Some(import);
-    }
-
-    /// Add an import statement to import the given module.
-    ///
-    /// If there are no existing imports, the new import will be added at the top
-    /// of the file. Otherwise, it will be added after the most recent top-level
-    /// import statement.
-    pub fn add_import(&self, import: &AnyImport) -> Edit {
-        let required_import = import.to_string();
-        if let Some(stmt) = self.trailing_import {
-            // Insert after the last top-level import.
-            let Insertion {
-                prefix,
-                location,
-                suffix,
-            } = end_of_statement_insertion(stmt, self.locator, self.stylist);
-            let content = format!("{prefix}{required_import}{suffix}");
-            Edit::insertion(content, location)
-        } else {
-            // Insert at the top of the file.
-            let Insertion {
-                prefix,
-                location,
-                suffix,
-            } = top_of_file_insertion(self.python_ast, self.locator, self.stylist);
-            let content = format!("{prefix}{required_import}{suffix}");
-            Edit::insertion(content, location)
-        }
-    }
-
-    /// Return the top-level [`Stmt`] that imports the given module using `StmtKind::ImportFrom`.
-    /// if it exists.
-    pub fn get_import_from(&self, module: &str) -> Option<&Stmt> {
-        self.import_from_map.get(module).copied()
-    }
-
-    /// Add the given member to an existing `StmtKind::ImportFrom` statement.
-    pub fn add_member(&self, stmt: &Stmt, member: &str) -> Result<Edit> {
-        let mut tree = match_module(self.locator.slice(stmt))?;
-        let import_from = match_import_from(&mut tree)?;
-        let aliases = match_aliases(import_from)?;
-        aliases.push(ImportAlias {
-            name: NameOrAttribute::N(Box::new(Name {
-                value: member,
-                lpar: vec![],
-                rpar: vec![],
-            })),
-            asname: None,
-            comma: aliases.last().and_then(|alias| alias.comma.clone()),
-        });
-        let mut state = CodegenState {
-            default_newline: &self.stylist.line_ending(),
-            default_indent: self.stylist.indentation(),
-            ..CodegenState::default()
-        };
-        tree.codegen(&mut state);
-        Ok(Edit::replacement(
-            state.to_string(),
-            stmt.location,
-            stmt.end_location.unwrap(),
-        ))
-    }
-}
-
-#[derive(Debug, Clone, PartialEq, Eq)]
-struct Insertion {
-    /// The content to add before the insertion.
-    prefix: &'static str,
-    /// The location at which to insert.
-    location: Location,
-    /// The content to add after the insertion.
-    suffix: &'static str,
-}
-
-impl Insertion {
-    fn new(prefix: &'static str, location: Location, suffix: &'static str) -> Self {
-        Self {
-            prefix,
-            location,
-            suffix,
-        }
-    }
-}
-
-/// Find the end of the last docstring.
-fn match_docstring_end(body: &[Stmt]) -> Option<Location> {
-    let mut iter = body.iter();
-    let Some(mut stmt) = iter.next() else {
-        return None;
-    };
-    if !is_docstring_stmt(stmt) {
-        return None;
-    }
-    for next in iter {
-        if !is_docstring_stmt(next) {
-            break;
-        }
-        stmt = next;
-    }
-    Some(stmt.end_location.unwrap())
-}
-
-/// Find the location at which a "top-of-file" import should be inserted,
-/// along with a prefix and suffix to use for the insertion.
-///
-/// For example, given the following code:
-///
-/// ```python
-/// """Hello, world!"""
-///
-/// import os
-/// ```
-///
-/// The location returned will be the start of the `import os` statement,
-/// along with a trailing newline suffix.
-fn end_of_statement_insertion(stmt: &Stmt, locator: &Locator, stylist: &Stylist) -> Insertion {
-    let location = stmt.end_location.unwrap();
-    let mut tokens = lexer::lex_located(locator.skip(location), Mode::Module, location).flatten();
-    if let Some((.., Tok::Semi, end)) = tokens.next() {
-        // If the first token after the docstring is a semicolon, insert after the semicolon as an
-        // inline statement;
-        Insertion::new(" ", end, ";")
-    } else {
-        // Otherwise, insert on the next line.
-        Insertion::new(
-            "",
-            Location::new(location.row() + 1, 0),
-            stylist.line_ending().as_str(),
-        )
-    }
-}
-
-/// Find the location at which a "top-of-file" import should be inserted,
-/// along with a prefix and suffix to use for the insertion.
-///
-/// For example, given the following code:
-///
-/// ```python
-/// """Hello, world!"""
-///
-/// import os
-/// ```
-///
-/// The location returned will be the start of the `import os` statement,
-/// along with a trailing newline suffix.
-fn top_of_file_insertion(body: &[Stmt], locator: &Locator, stylist: &Stylist) -> Insertion {
-    // Skip over any docstrings.
-    let mut location = if let Some(location) = match_docstring_end(body) {
-        // If the first token after the docstring is a semicolon, insert after the semicolon as an
-        // inline statement;
-        let first_token = lexer::lex_located(locator.skip(location), Mode::Module, location)
-            .flatten()
-            .next();
-        if let Some((.., Tok::Semi, end)) = first_token {
-            return Insertion::new(" ", end, ";");
-        }
-
-        // Otherwise, advance to the next row.
-        Location::new(location.row() + 1, 0)
-    } else {
-        Location::default()
-    };
-
-    // Skip over any comments and empty lines.
-    for (.., tok, end) in
-        lexer::lex_located(locator.skip(location), Mode::Module, location).flatten()
-    {
-        if matches!(tok, Tok::Comment(..) | Tok::Newline) {
-            location = Location::new(end.row() + 1, 0);
-        } else {
-            break;
-        }
-    }
-
-    return Insertion::new("", location, stylist.line_ending().as_str());
-}
-
-#[cfg(test)]
-mod tests {
-    use anyhow::Result;
-    use rustpython_parser as parser;
-    use rustpython_parser::ast::Location;
-    use rustpython_parser::lexer::LexResult;
-
-    use ruff_python_ast::source_code::{LineEnding, Locator, Stylist};
-
-    use crate::importer::{top_of_file_insertion, Insertion};
-
-    fn insert(contents: &str) -> Result<Insertion> {
-        let program = parser::parse_program(contents, "<filename>")?;
-        let tokens: Vec<LexResult> = ruff_rustpython::tokenize(contents);
-        let locator = Locator::new(contents);
-        let stylist = Stylist::from_tokens(&tokens, &locator);
-        Ok(top_of_file_insertion(&program, &locator, &stylist))
-    }
-
-    #[test]
-    fn top_of_file_insertions() -> Result<()> {
-        let contents = "";
-        assert_eq!(
-            insert(contents)?,
-            Insertion::new("", Location::new(1, 0), LineEnding::default().as_str())
-        );
-
-        let contents = r#"
-"""Hello, world!""""#
-            .trim_start();
-        assert_eq!(
-            insert(contents)?,
-            Insertion::new("", Location::new(2, 0), LineEnding::default().as_str())
-        );
-
-        let contents = r#"
-"""Hello, world!"""
-"#
-        .trim_start();
-        assert_eq!(
-            insert(contents)?,
-            Insertion::new("", Location::new(2, 0), "\n")
-        );
-
-        let contents = r#"
-"""Hello, world!"""
-"""Hello, world!"""
-"#
-        .trim_start();
-        assert_eq!(
-            insert(contents)?,
-            Insertion::new("", Location::new(3, 0), "\n")
-        );
-
-        let contents = r#"
-x = 1
-"#
-        .trim_start();
-        assert_eq!(
-            insert(contents)?,
-            Insertion::new("", Location::new(1, 0), "\n")
-        );
-
-        let contents = r#"
-#!/usr/bin/env python3
-"#
-        .trim_start();
-        assert_eq!(
-            insert(contents)?,
-            Insertion::new("", Location::new(2, 0), "\n")
-        );
-
-        let contents = r#"
-#!/usr/bin/env python3
-"""Hello, world!"""
-"#
-        .trim_start();
-        assert_eq!(
-            insert(contents)?,
-            Insertion::new("", Location::new(3, 0), "\n")
-        );
-
-        let contents = r#"
-"""Hello, world!"""
-#!/usr/bin/env python3
-"#
-        .trim_start();
-        assert_eq!(
-            insert(contents)?,
-            Insertion::new("", Location::new(3, 0), "\n")
-        );
-
-        let contents = r#"
-"""%s""" % "Hello, world!"
-"#
-        .trim_start();
-        assert_eq!(
-            insert(contents)?,
-            Insertion::new("", Location::new(1, 0), "\n")
-        );
-
-        let contents = r#"
-"""Hello, world!"""; x = 1
-"#
-        .trim_start();
-        assert_eq!(
-            insert(contents)?,
-            Insertion::new(" ", Location::new(1, 20), ";")
-        );
-
-        let contents = r#"
-"""Hello, world!"""; x = 1; y = \
-    2
-"#
-        .trim_start();
-        assert_eq!(
-            insert(contents)?,
-            Insertion::new(" ", Location::new(1, 20), ";")
-        );
-
-        Ok(())
-    }
-}

crates/ruff/src/jupyter/schema.rs

@@ -226,7 +226,7 @@ pub enum CodemirrorMode {
 }
 
 /// String identifying the type of cell.
-#[derive(Debug, Serialize, Deserialize, PartialEq, Copy, Clone)]
+#[derive(Debug, Serialize, Deserialize, PartialEq)]
 pub enum CellType {
     #[serde(rename = "code")]
     Code,
@@ -236,14 +236,14 @@ pub enum CellType {
     Raw,
 }
 
-#[derive(Debug, Serialize, Deserialize, Copy, Clone)]
+#[derive(Debug, Serialize, Deserialize)]
 pub enum ScrolledEnum {
     #[serde(rename = "auto")]
     Auto,
 }
 
 /// Type of cell output.
-#[derive(Debug, Serialize, Deserialize, Copy, Clone)]
+#[derive(Debug, Serialize, Deserialize)]
 pub enum OutputType {
     #[serde(rename = "display_data")]
     DisplayData,

crates/ruff/src/lex/docstring_detection.rs

@@ -6,7 +6,7 @@
 
 use rustpython_parser::Tok;
 
-#[derive(Default, Copy, Clone)]
+#[derive(Default)]
 enum State {
     // Start of the module: first string gets marked as a docstring.
     #[default]
@@ -32,91 +32,86 @@ pub struct StateMachine {
 
 impl StateMachine {
     pub fn consume(&mut self, tok: &Tok) -> bool {
-        match tok {
-            Tok::NonLogicalNewline
-            | Tok::Newline
-            | Tok::Indent
-            | Tok::Dedent
-            | Tok::Comment(..) => false,
-
-            Tok::String { .. } => {
-                if matches!(
-                    self.state,
-                    State::ExpectModuleDocstring
-                        | State::ExpectClassDocstring
-                        | State::ExpectFunctionDocstring
-                ) {
-                    self.state = State::Other;
-                    true
-                } else {
-                    false
-                }
-            }
-            Tok::Class => {
-                self.state = State::ExpectClassColon;
-                self.bracket_count = 0;
-
-                false
-            }
-
-            Tok::Def => {
-                self.state = State::ExpectFunctionColon;
-                self.bracket_count = 0;
-
-                false
-            }
-
-            Tok::Colon => {
-                if self.bracket_count == 0 {
-                    if matches!(self.state, State::ExpectClassColon) {
-                        self.state = State::ExpectClassDocstring;
-                    } else if matches!(self.state, State::ExpectFunctionColon) {
-                        self.state = State::ExpectFunctionDocstring;
-                    }
-                }
-
-                false
-            }
-
-            Tok::Lpar | Tok::Lbrace | Tok::Lsqb => {
-                self.bracket_count += 1;
-                if matches!(
-                    self.state,
-                    State::ExpectModuleDocstring
-                        | State::ExpectClassDocstring
-                        | State::ExpectFunctionDocstring
-                ) {
-                    self.state = State::Other;
-                }
-                false
-            }
-
-            Tok::Rpar | Tok::Rbrace | Tok::Rsqb => {
-                self.bracket_count -= 1;
-                if matches!(
-                    self.state,
-                    State::ExpectModuleDocstring
-                        | State::ExpectClassDocstring
-                        | State::ExpectFunctionDocstring
-                ) {
-                    self.state = State::Other;
-                }
-
-                false
-            }
-
-            _ => {
-                if matches!(
-                    self.state,
-                    State::ExpectModuleDocstring
-                        | State::ExpectClassDocstring
-                        | State::ExpectFunctionDocstring
-                ) {
-                    self.state = State::Other;
-                }
-
-                false
-            }
+        if matches!(
+            tok,
+            Tok::NonLogicalNewline | Tok::Newline | Tok::Indent | Tok::Dedent | Tok::Comment(..)
+        ) {
+            return false;
         }
+
+        if matches!(tok, Tok::String { .. }) {
+            return if matches!(
+                self.state,
+                State::ExpectModuleDocstring
+                    | State::ExpectClassDocstring
+                    | State::ExpectFunctionDocstring
+            ) {
+                self.state = State::Other;
+                true
+            } else {
+                false
+            };
+        }
+
+        if matches!(tok, Tok::Class) {
+            self.state = State::ExpectClassColon;
+            self.bracket_count = 0;
+            return false;
+        }
+
+        if matches!(tok, Tok::Def) {
+            self.state = State::ExpectFunctionColon;
+            self.bracket_count = 0;
+            return false;
+        }
+
+        if matches!(tok, Tok::Colon) {
+            if self.bracket_count == 0 {
+                if matches!(self.state, State::ExpectClassColon) {
+                    self.state = State::ExpectClassDocstring;
+                } else if matches!(self.state, State::ExpectFunctionColon) {
+                    self.state = State::ExpectFunctionDocstring;
+                }
+            }
+            return false;
+        }
+
+        if matches!(tok, Tok::Lpar | Tok::Lbrace | Tok::Lsqb) {
+            self.bracket_count += 1;
+            if matches!(
+                self.state,
+                State::ExpectModuleDocstring
+                    | State::ExpectClassDocstring
+                    | State::ExpectFunctionDocstring
+            ) {
+                self.state = State::Other;
+            }
+            return false;
+        }
+
+        if matches!(tok, Tok::Rpar | Tok::Rbrace | Tok::Rsqb) {
+            self.bracket_count -= 1;
+            if matches!(
+                self.state,
+                State::ExpectModuleDocstring
+                    | State::ExpectClassDocstring
+                    | State::ExpectFunctionDocstring
+            ) {
+                self.state = State::Other;
+            }
+            return false;
+        }
+
+        if matches!(
+            self.state,
+            State::ExpectModuleDocstring
+                | State::ExpectClassDocstring
+                | State::ExpectFunctionDocstring
+        ) {
+            self.state = State::Other;
+            return false;
+        }
+
+        false
     }
 }

crates/ruff/src/lib.rs

@@ -17,9 +17,9 @@ mod cst;
 pub mod directives;
 mod doc_lines;
 mod docstrings;
+pub mod fix;
 pub mod flake8_to_ruff;
 pub mod fs;
-mod importer;
 pub mod jupyter;
 mod lex;
 pub mod linter;

crates/ruff/src/linter.rs

@@ -16,13 +16,14 @@ use crate::autofix::fix_file;
 use crate::checkers::ast::check_ast;
 use crate::checkers::filesystem::check_file_path;
 use crate::checkers::imports::check_imports;
+use crate::checkers::logical_lines::check_logical_lines;
 use crate::checkers::noqa::check_noqa;
 use crate::checkers::physical_lines::check_physical_lines;
 use crate::checkers::tokens::check_tokens;
 use crate::directives::Directives;
 use crate::doc_lines::{doc_lines_from_ast, doc_lines_from_tokens};
 use crate::message::{Message, Source};
-use crate::noqa::add_noqa;
+use crate::noqa::{add_noqa, rule_is_ignored};
 use crate::registry::{AsRule, Rule};
 use crate::rules::pycodestyle;
 use crate::settings::{flags, Settings};
@@ -104,8 +105,7 @@ pub fn check_path(
         .iter_enabled()
         .any(|rule_code| rule_code.lint_source().is_logical_lines())
     {
-        #[cfg(feature = "logical_lines")]
-        diagnostics.extend(crate::checkers::logical_lines::check_logical_lines(
+        diagnostics.extend(check_logical_lines(
             &tokens,
             locator,
             stylist,
@@ -159,13 +159,20 @@ pub fn check_path(
                 }
             }
             Err(parse_error) => {
-                // Always add a diagnostic for the syntax error, regardless of whether
-                // `Rule::SyntaxError` is enabled. We avoid propagating the syntax error
-                // if it's disabled via any of the usual mechanisms (e.g., `noqa`,
-                // `per-file-ignores`), and the easiest way to detect that suppression is
-                // to see if the diagnostic persists to the end of the function.
-                pycodestyle::rules::syntax_error(&mut diagnostics, &parse_error);
-                error = Some(parse_error);
+                if settings.rules.enabled(Rule::SyntaxError) {
+                    pycodestyle::rules::syntax_error(&mut diagnostics, &parse_error);
+                }
+
+                // If the syntax error is ignored, suppress it (regardless of whether
+                // `Rule::SyntaxError` is enabled).
+                if !rule_is_ignored(
+                    Rule::SyntaxError,
+                    parse_error.location.row(),
+                    &directives.noqa_line_for,
+                    locator,
+                ) {
+                    error = Some(parse_error);
+                }
             }
         }
     }
@@ -223,23 +230,6 @@ pub fn check_path(
         }
     }
 
-    // If there was a syntax error, check if it should be discarded.
-    if error.is_some() {
-        // If the syntax error was removed by _any_ of the above disablement methods (e.g., a
-        // `noqa` directive, or a `per-file-ignore`), discard it.
-        if !diagnostics
-            .iter()
-            .any(|diagnostic| diagnostic.kind.rule() == Rule::SyntaxError)
-        {
-            error = None;
-        }
-
-        // If the syntax error _diagnostic_ is disabled, discard the _diagnostic_.
-        if !settings.rules.enabled(Rule::SyntaxError) {
-            diagnostics.retain(|diagnostic| diagnostic.kind.rule() != Rule::SyntaxError);
-        }
-    }
-
     LinterResult::new(diagnostics, error)
 }
 
@@ -257,7 +247,7 @@ pub fn add_noqa_to_path(path: &Path, package: Option<&Path>, settings: &Settings
     let locator = Locator::new(&contents);
 
     // Detect the current code style (lazily).
-    let stylist = Stylist::from_tokens(&tokens, &locator);
+    let stylist = Stylist::from_contents(&contents, &locator);
 
     // Extra indices from the code.
     let indexer: Indexer = tokens.as_slice().into();
@@ -322,7 +312,7 @@ pub fn lint_only(
     let locator = Locator::new(contents);
 
     // Detect the current code style (lazily).
-    let stylist = Stylist::from_tokens(&tokens, &locator);
+    let stylist = Stylist::from_contents(contents, &locator);
 
     // Extra indices from the code.
     let indexer: Indexer = tokens.as_slice().into();
@@ -394,7 +384,7 @@ pub fn lint_fix<'a>(
         let locator = Locator::new(&transformed);
 
         // Detect the current code style (lazily).
-        let stylist = Stylist::from_tokens(&tokens, &locator);
+        let stylist = Stylist::from_contents(&transformed, &locator);
 
         // Extra indices from the code.
         let indexer: Indexer = tokens.as_slice().into();

crates/ruff/src/message.rs

@@ -12,7 +12,7 @@ pub struct Message {
     pub kind: DiagnosticKind,
     pub location: Location,
     pub end_location: Location,
-    pub fix: Fix,
+    pub fix: Option<Fix>,
     pub filename: String,
     pub source: Option<Source>,
     pub noqa_row: usize,

crates/ruff/src/noqa.rs

@@ -28,6 +28,49 @@ static NOQA_LINE_REGEX: Lazy<Regex> = Lazy::new(|| {
 });
 static SPLIT_COMMA_REGEX: Lazy<Regex> = Lazy::new(|| Regex::new(r"[,\s]").unwrap());
 
+#[derive(Debug)]
+pub enum Exemption<'a> {
+    None,
+    All,
+    Codes(Vec<&'a str>),
+}
+
+/// Return `true` if a file is exempt from checking based on the contents of the
+/// given line.
+pub fn extract_file_exemption(line: &str) -> Exemption {
+    let line = line.trim_start();
+
+    if line.starts_with("# flake8: noqa")
+        || line.starts_with("# flake8: NOQA")
+        || line.starts_with("# flake8: NoQA")
+    {
+        return Exemption::All;
+    }
+
+    if let Some(remainder) = line
+        .strip_prefix("# ruff: noqa")
+        .or_else(|| line.strip_prefix("# ruff: NOQA"))
+        .or_else(|| line.strip_prefix("# ruff: NoQA"))
+    {
+        if remainder.is_empty() {
+            return Exemption::All;
+        } else if let Some(codes) = remainder.strip_prefix(':') {
+            let codes: Vec<&str> = SPLIT_COMMA_REGEX
+                .split(codes.trim())
+                .map(str::trim)
+                .filter(|code| !code.is_empty())
+                .collect();
+            if codes.is_empty() {
+                warn!("Expected rule codes on `noqa` directive: \"{line}\"");
+            }
+            return Exemption::Codes(codes);
+        }
+        warn!("Unexpected suffix on `noqa` directive: \"{line}\"");
+    }
+
+    Exemption::None
+}
+
 #[derive(Debug)]
 pub enum Directive<'a> {
     None,
@@ -76,47 +119,6 @@ pub fn extract_noqa_directive(line: &str) -> Directive {
     }
 }
 
-enum ParsedExemption<'a> {
-    None,
-    All,
-    Codes(Vec<&'a str>),
-}
-
-/// Return a [`ParsedExemption`] for a given comment line.
-fn parse_file_exemption(line: &str) -> ParsedExemption {
-    let line = line.trim_start();
-
-    if line.starts_with("# flake8: noqa")
-        || line.starts_with("# flake8: NOQA")
-        || line.starts_with("# flake8: NoQA")
-    {
-        return ParsedExemption::All;
-    }
-
-    if let Some(remainder) = line
-        .strip_prefix("# ruff: noqa")
-        .or_else(|| line.strip_prefix("# ruff: NOQA"))
-        .or_else(|| line.strip_prefix("# ruff: NoQA"))
-    {
-        if remainder.is_empty() {
-            return ParsedExemption::All;
-        } else if let Some(codes) = remainder.strip_prefix(':') {
-            let codes: Vec<&str> = SPLIT_COMMA_REGEX
-                .split(codes.trim())
-                .map(str::trim)
-                .filter(|code| !code.is_empty())
-                .collect();
-            if codes.is_empty() {
-                warn!("Expected rule codes on `noqa` directive: \"{line}\"");
-            }
-            return ParsedExemption::Codes(codes);
-        }
-        warn!("Unexpected suffix on `noqa` directive: \"{line}\"");
-    }
-
-    ParsedExemption::None
-}
-
 /// Returns `true` if the string list of `codes` includes `code` (or an alias
 /// thereof).
 pub fn includes(needle: Rule, haystack: &[&str]) -> bool {
@@ -145,50 +147,13 @@ pub fn rule_is_ignored(
     }
 }
 
-pub enum FileExemption {
-    None,
-    All,
-    Codes(Vec<NoqaCode>),
-}
-
-/// Extract the [`FileExemption`] for a given Python source file, enumerating any rules that are
-/// globally ignored within the file.
-pub fn file_exemption(lines: &[&str], commented_lines: &[usize]) -> FileExemption {
-    let mut exempt_codes: Vec<NoqaCode> = vec![];
-
-    for lineno in commented_lines {
-        match parse_file_exemption(lines[lineno - 1]) {
-            ParsedExemption::All => {
-                return FileExemption::All;
-            }
-            ParsedExemption::Codes(codes) => {
-                exempt_codes.extend(codes.into_iter().filter_map(|code| {
-                    if let Ok(rule) = Rule::from_code(get_redirect_target(code).unwrap_or(code)) {
-                        Some(rule.noqa_code())
-                    } else {
-                        warn!("Invalid code provided to `# ruff: noqa`: {}", code);
-                        None
-                    }
-                }));
-            }
-            ParsedExemption::None => {}
-        }
-    }
-
-    if exempt_codes.is_empty() {
-        FileExemption::None
-    } else {
-        FileExemption::Codes(exempt_codes)
-    }
-}
-
 pub fn add_noqa(
     path: &Path,
     diagnostics: &[Diagnostic],
     contents: &str,
     commented_lines: &[usize],
     noqa_line_for: &IntMap<usize, usize>,
-    line_ending: LineEnding,
+    line_ending: &LineEnding,
 ) -> Result<usize> {
     let (count, output) = add_noqa_inner(
         diagnostics,
@@ -206,56 +171,71 @@ fn add_noqa_inner(
     contents: &str,
     commented_lines: &[usize],
     noqa_line_for: &IntMap<usize, usize>,
-    line_ending: LineEnding,
+    line_ending: &LineEnding,
 ) -> (usize, String) {
     // Map of line number to set of (non-ignored) diagnostic codes that are triggered on that line.
     let mut matches_by_line: FxHashMap<usize, RuleSet> = FxHashMap::default();
 
-    let lines: Vec<&str> = contents.universal_newlines().collect();
-
     // Whether the file is exempted from all checks.
+    let mut file_exempted = false;
+
     // Codes that are globally exempted (within the current file).
-    let exemption = file_exemption(&lines, commented_lines);
+    let mut file_exemptions: Vec<NoqaCode> = vec![];
+
+    let lines: Vec<&str> = contents.universal_newlines().collect();
+    for lineno in commented_lines {
+        match extract_file_exemption(lines[lineno - 1]) {
+            Exemption::All => {
+                file_exempted = true;
+            }
+            Exemption::Codes(codes) => {
+                file_exemptions.extend(codes.into_iter().filter_map(|code| {
+                    if let Ok(rule) = Rule::from_code(get_redirect_target(code).unwrap_or(code)) {
+                        Some(rule.noqa_code())
+                    } else {
+                        warn!("Invalid code provided to `# ruff: noqa`: {}", code);
+                        None
+                    }
+                }));
+            }
+            Exemption::None => {}
+        }
+    }
 
     // Mark any non-ignored diagnostics.
     for diagnostic in diagnostics {
-        match &exemption {
-            FileExemption::All => {
-                // If the file is exempted, don't add any noqa directives.
-                continue;
-            }
-            FileExemption::Codes(codes) => {
-                // If the diagnostic is ignored by a global exemption, don't add a noqa directive.
-                if codes.contains(&diagnostic.kind.rule().noqa_code()) {
-                    continue;
-                }
-            }
-            FileExemption::None => {}
+        // If the file is exempted, don't add any noqa directives.
+        if file_exempted {
+            continue;
         }
 
-        let diagnostic_lineno = diagnostic.location.row();
+        // If the diagnostic is ignored by a global exemption, don't add a noqa directive.
+        if !file_exemptions.is_empty() {
+            if file_exemptions.contains(&diagnostic.kind.rule().noqa_code()) {
+                continue;
+            }
+        }
 
         // Is the violation ignored by a `noqa` directive on the parent line?
         if let Some(parent_lineno) = diagnostic.parent.map(|location| location.row()) {
-            if parent_lineno != diagnostic_lineno {
-                let noqa_lineno = noqa_line_for.get(&parent_lineno).unwrap_or(&parent_lineno);
-                if commented_lines.contains(noqa_lineno) {
-                    match extract_noqa_directive(lines[noqa_lineno - 1]) {
-                        Directive::All(..) => {
+            let noqa_lineno = noqa_line_for.get(&parent_lineno).unwrap_or(&parent_lineno);
+            if commented_lines.contains(noqa_lineno) {
+                match extract_noqa_directive(lines[noqa_lineno - 1]) {
+                    Directive::All(..) => {
+                        continue;
+                    }
+                    Directive::Codes(.., codes, _) => {
+                        if includes(diagnostic.kind.rule(), &codes) {
                             continue;
                         }
-                        Directive::Codes(.., codes, _) => {
-                            if includes(diagnostic.kind.rule(), &codes) {
-                                continue;
-                            }
-                        }
-                        Directive::None => {}
                     }
+                    Directive::None => {}
                 }
             }
         }
 
         // Is the diagnostic ignored by a `noqa` directive on the same line?
+        let diagnostic_lineno = diagnostic.location.row();
         let noqa_lineno = noqa_line_for
             .get(&diagnostic_lineno)
             .unwrap_or(&diagnostic_lineno);
@@ -288,7 +268,7 @@ fn add_noqa_inner(
         match matches_by_line.get(&lineno) {
             None => {
                 output.push_str(line);
-                output.push_str(&line_ending);
+                output.push_str(line_ending);
             }
             Some(rules) => {
                 match extract_noqa_directive(line) {
@@ -301,13 +281,13 @@ fn add_noqa_inner(
 
                         // Add codes.
                         push_codes(&mut output, rules.iter().map(|rule| rule.noqa_code()));
-                        output.push_str(&line_ending);
+                        output.push_str(line_ending);
                         count += 1;
                     }
                     Directive::All(..) => {
                         // Leave the line as-is.
                         output.push_str(line);
-                        output.push_str(&line_ending);
+                        output.push_str(line_ending);
                     }
                     Directive::Codes(_, start_byte, _, existing, _) => {
                         // Reconstruct the line based on the preserved rule codes.
@@ -331,7 +311,7 @@ fn add_noqa_inner(
                         );
 
                         output.push_str(&formatted);
-                        output.push_str(&line_ending);
+                        output.push_str(line_ending);
 
                         // Only count if the new line is an actual edit.
                         if formatted != line {
@@ -395,7 +375,7 @@ mod tests {
             contents,
             &commented_lines,
             &noqa_line_for,
-            LineEnding::Lf,
+            &LineEnding::Lf,
         );
         assert_eq!(count, 0);
         assert_eq!(output, format!("{contents}\n"));
@@ -414,7 +394,7 @@ mod tests {
             contents,
             &commented_lines,
             &noqa_line_for,
-            LineEnding::Lf,
+            &LineEnding::Lf,
         );
         assert_eq!(count, 1);
         assert_eq!(output, "x = 1  # noqa: F841\n");
@@ -439,7 +419,7 @@ mod tests {
             contents,
             &commented_lines,
             &noqa_line_for,
-            LineEnding::Lf,
+            &LineEnding::Lf,
         );
         assert_eq!(count, 1);
         assert_eq!(output, "x = 1  # noqa: E741, F841\n");
@@ -464,7 +444,7 @@ mod tests {
             contents,
             &commented_lines,
             &noqa_line_for,
-            LineEnding::Lf,
+            &LineEnding::Lf,
         );
         assert_eq!(count, 0);
         assert_eq!(output, "x = 1  # noqa\n");

crates/ruff/src/registry.rs

@@ -14,68 +14,68 @@ pub use rule_set::{RuleSet, RuleSetIterator};
 ruff_macros::register_rules!(
     // pycodestyle errors
     rules::pycodestyle::rules::MixedSpacesAndTabs,
-    #[cfg(feature = "logical_lines")]
-    rules::pycodestyle::rules::logical_lines::IndentationWithInvalidMultiple,
-    #[cfg(feature = "logical_lines")]
-    rules::pycodestyle::rules::logical_lines::NoIndentedBlock,
-    #[cfg(feature = "logical_lines")]
-    rules::pycodestyle::rules::logical_lines::UnexpectedIndentation,
-    #[cfg(feature = "logical_lines")]
-    rules::pycodestyle::rules::logical_lines::IndentationWithInvalidMultipleComment,
-    #[cfg(feature = "logical_lines")]
-    rules::pycodestyle::rules::logical_lines::NoIndentedBlockComment,
-    #[cfg(feature = "logical_lines")]
-    rules::pycodestyle::rules::logical_lines::UnexpectedIndentationComment,
-    #[cfg(feature = "logical_lines")]
-    rules::pycodestyle::rules::logical_lines::OverIndented,
-    #[cfg(feature = "logical_lines")]
-    rules::pycodestyle::rules::logical_lines::WhitespaceAfterOpenBracket,
-    #[cfg(feature = "logical_lines")]
-    rules::pycodestyle::rules::logical_lines::WhitespaceBeforeCloseBracket,
-    #[cfg(feature = "logical_lines")]
-    rules::pycodestyle::rules::logical_lines::WhitespaceBeforePunctuation,
-    #[cfg(feature = "logical_lines")]
-    rules::pycodestyle::rules::logical_lines::MultipleSpacesBeforeOperator,
-    #[cfg(feature = "logical_lines")]
-    rules::pycodestyle::rules::logical_lines::MultipleSpacesAfterOperator,
-    #[cfg(feature = "logical_lines")]
-    rules::pycodestyle::rules::logical_lines::TabBeforeOperator,
-    #[cfg(feature = "logical_lines")]
-    rules::pycodestyle::rules::logical_lines::TabAfterOperator,
-    #[cfg(feature = "logical_lines")]
-    rules::pycodestyle::rules::logical_lines::TooFewSpacesBeforeInlineComment,
-    #[cfg(feature = "logical_lines")]
-    rules::pycodestyle::rules::logical_lines::NoSpaceAfterInlineComment,
-    #[cfg(feature = "logical_lines")]
-    rules::pycodestyle::rules::logical_lines::NoSpaceAfterBlockComment,
-    #[cfg(feature = "logical_lines")]
-    rules::pycodestyle::rules::logical_lines::MultipleLeadingHashesForBlockComment,
-    #[cfg(feature = "logical_lines")]
-    rules::pycodestyle::rules::logical_lines::MultipleSpacesAfterKeyword,
-    #[cfg(feature = "logical_lines")]
-    rules::pycodestyle::rules::logical_lines::MissingWhitespace,
-    #[cfg(feature = "logical_lines")]
-    rules::pycodestyle::rules::logical_lines::MissingWhitespaceAfterKeyword,
-    #[cfg(feature = "logical_lines")]
-    rules::pycodestyle::rules::logical_lines::MultipleSpacesBeforeKeyword,
-    #[cfg(feature = "logical_lines")]
-    rules::pycodestyle::rules::logical_lines::MissingWhitespaceAroundOperator,
-    #[cfg(feature = "logical_lines")]
-    rules::pycodestyle::rules::logical_lines::MissingWhitespaceAroundArithmeticOperator,
-    #[cfg(feature = "logical_lines")]
-    rules::pycodestyle::rules::logical_lines::MissingWhitespaceAroundBitwiseOrShiftOperator,
-    #[cfg(feature = "logical_lines")]
-    rules::pycodestyle::rules::logical_lines::MissingWhitespaceAroundModuloOperator,
-    #[cfg(feature = "logical_lines")]
-    rules::pycodestyle::rules::logical_lines::TabAfterKeyword,
-    #[cfg(feature = "logical_lines")]
-    rules::pycodestyle::rules::logical_lines::UnexpectedSpacesAroundKeywordParameterEquals,
-    #[cfg(feature = "logical_lines")]
-    rules::pycodestyle::rules::logical_lines::MissingWhitespaceAroundParameterEquals,
-    #[cfg(feature = "logical_lines")]
-    rules::pycodestyle::rules::logical_lines::WhitespaceBeforeParameters,
-    #[cfg(feature = "logical_lines")]
-    rules::pycodestyle::rules::logical_lines::TabBeforeKeyword,
+    #[cfg(debug_assertions)]
+    rules::pycodestyle::rules::IndentationWithInvalidMultiple,
+    #[cfg(debug_assertions)]
+    rules::pycodestyle::rules::NoIndentedBlock,
+    #[cfg(debug_assertions)]
+    rules::pycodestyle::rules::UnexpectedIndentation,
+    #[cfg(debug_assertions)]
+    rules::pycodestyle::rules::IndentationWithInvalidMultipleComment,
+    #[cfg(debug_assertions)]
+    rules::pycodestyle::rules::NoIndentedBlockComment,
+    #[cfg(debug_assertions)]
+    rules::pycodestyle::rules::UnexpectedIndentationComment,
+    #[cfg(debug_assertions)]
+    rules::pycodestyle::rules::OverIndented,
+    #[cfg(debug_assertions)]
+    rules::pycodestyle::rules::WhitespaceAfterOpenBracket,
+    #[cfg(debug_assertions)]
+    rules::pycodestyle::rules::WhitespaceBeforeCloseBracket,
+    #[cfg(debug_assertions)]
+    rules::pycodestyle::rules::WhitespaceBeforePunctuation,
+    #[cfg(debug_assertions)]
+    rules::pycodestyle::rules::MultipleSpacesBeforeOperator,
+    #[cfg(debug_assertions)]
+    rules::pycodestyle::rules::MultipleSpacesAfterOperator,
+    #[cfg(debug_assertions)]
+    rules::pycodestyle::rules::TabBeforeOperator,
+    #[cfg(debug_assertions)]
+    rules::pycodestyle::rules::TabAfterOperator,
+    #[cfg(debug_assertions)]
+    rules::pycodestyle::rules::TooFewSpacesBeforeInlineComment,
+    #[cfg(debug_assertions)]
+    rules::pycodestyle::rules::NoSpaceAfterInlineComment,
+    #[cfg(debug_assertions)]
+    rules::pycodestyle::rules::NoSpaceAfterBlockComment,
+    #[cfg(debug_assertions)]
+    rules::pycodestyle::rules::MultipleLeadingHashesForBlockComment,
+    #[cfg(debug_assertions)]
+    rules::pycodestyle::rules::MultipleSpacesAfterKeyword,
+    #[cfg(debug_assertions)]
+    rules::pycodestyle::rules::MissingWhitespace,
+    #[cfg(debug_assertions)]
+    rules::pycodestyle::rules::MissingWhitespaceAfterKeyword,
+    #[cfg(debug_assertions)]
+    rules::pycodestyle::rules::MultipleSpacesBeforeKeyword,
+    #[cfg(debug_assertions)]
+    rules::pycodestyle::rules::MissingWhitespaceAroundOperator,
+    #[cfg(debug_assertions)]
+    rules::pycodestyle::rules::MissingWhitespaceAroundArithmeticOperator,
+    #[cfg(debug_assertions)]
+    rules::pycodestyle::rules::MissingWhitespaceAroundBitwiseOrShiftOperator,
+    #[cfg(debug_assertions)]
+    rules::pycodestyle::rules::MissingWhitespaceAroundModuloOperator,
+    #[cfg(debug_assertions)]
+    rules::pycodestyle::rules::TabAfterKeyword,
+    #[cfg(debug_assertions)]
+    rules::pycodestyle::rules::UnexpectedSpacesAroundKeywordParameterEquals,
+    #[cfg(debug_assertions)]
+    rules::pycodestyle::rules::MissingWhitespaceAroundParameterEquals,
+    #[cfg(debug_assertions)]
+    rules::pycodestyle::rules::WhitespaceBeforeParameters,
+    #[cfg(debug_assertions)]
+    rules::pycodestyle::rules::TabBeforeKeyword,
     rules::pycodestyle::rules::MultipleImportsOnOneLine,
     rules::pycodestyle::rules::ModuleImportNotAtTopOfFile,
     rules::pycodestyle::rules::LineTooLong,
@@ -167,7 +167,7 @@ ruff_macros::register_rules!(
     rules::pylint::rules::UselessImportAlias,
     rules::pylint::rules::UnnecessaryDirectLambdaCall,
     rules::pylint::rules::NonlocalWithoutBinding,
-    rules::pylint::rules::LoadBeforeGlobalDeclaration,
+    rules::pylint::rules::UsePriorToGlobalDeclaration,
     rules::pylint::rules::AwaitOutsideAsync,
     rules::pylint::rules::PropertyWithParameters,
     rules::pylint::rules::ReturnInInit,
@@ -223,7 +223,6 @@ ruff_macros::register_rules!(
     rules::flake8_bugbear::rules::ZipWithoutExplicitStrict,
     rules::flake8_bugbear::rules::ExceptWithEmptyTuple,
     rules::flake8_bugbear::rules::ExceptWithNonExceptionClasses,
-    rules::flake8_bugbear::rules::ReuseOfGroupbyGenerator,
     rules::flake8_bugbear::rules::UnintentionalTypeAnnotation,
     // flake8-blind-except
     rules::flake8_blind_except::rules::BlindExcept,
@@ -300,7 +299,7 @@ ruff_macros::register_rules!(
     rules::flake8_simplify::rules::DuplicateIsinstanceCall,
     rules::flake8_simplify::rules::CollapsibleIf,
     rules::flake8_simplify::rules::NeedlessBool,
-    rules::flake8_simplify::rules::SuppressibleException,
+    rules::flake8_simplify::rules::UseContextlibSuppress,
     rules::flake8_simplify::rules::ReturnInTryExceptFinally,
     rules::flake8_simplify::rules::IfElseBlockInsteadOfIfExp,
     rules::flake8_simplify::rules::CompareWithTuple,
@@ -430,45 +429,25 @@ ruff_macros::register_rules!(
     rules::eradicate::rules::CommentedOutCode,
     // flake8-bandit
     rules::flake8_bandit::rules::Assert,
-    rules::flake8_bandit::rules::BadFilePermissions,
+    rules::flake8_bandit::rules::DeniedFunctionCall,
     rules::flake8_bandit::rules::ExecBuiltin,
+    rules::flake8_bandit::rules::BadFilePermissions,
     rules::flake8_bandit::rules::HardcodedBindAllInterfaces,
-    rules::flake8_bandit::rules::HardcodedPasswordDefault,
-    rules::flake8_bandit::rules::HardcodedPasswordFuncArg,
     rules::flake8_bandit::rules::HardcodedPasswordString,
+    rules::flake8_bandit::rules::HardcodedPasswordFuncArg,
+    rules::flake8_bandit::rules::HardcodedPasswordDefault,
     rules::flake8_bandit::rules::HardcodedSQLExpression,
     rules::flake8_bandit::rules::HardcodedTempFile,
-    rules::flake8_bandit::rules::HashlibInsecureHashFunction,
-    rules::flake8_bandit::rules::Jinja2AutoescapeFalse,
-    rules::flake8_bandit::rules::LoggingConfigInsecureListen,
-    rules::flake8_bandit::rules::RequestWithNoCertValidation,
+    rules::flake8_bandit::rules::TryExceptPass,
+    rules::flake8_bandit::rules::TryExceptContinue,
     rules::flake8_bandit::rules::RequestWithoutTimeout,
+    rules::flake8_bandit::rules::HashlibInsecureHashFunction,
+    rules::flake8_bandit::rules::RequestWithNoCertValidation,
+    rules::flake8_bandit::rules::UnsafeYAMLLoad,
     rules::flake8_bandit::rules::SnmpInsecureVersion,
     rules::flake8_bandit::rules::SnmpWeakCryptography,
-    rules::flake8_bandit::rules::SuspiciousEvalUsage,
-    rules::flake8_bandit::rules::SuspiciousFTPLibUsage,
-    rules::flake8_bandit::rules::SuspiciousInsecureCipherUsage,
-    rules::flake8_bandit::rules::SuspiciousInsecureCipherModeUsage,
-    rules::flake8_bandit::rules::SuspiciousInsecureHashUsage,
-    rules::flake8_bandit::rules::SuspiciousMarkSafeUsage,
-    rules::flake8_bandit::rules::SuspiciousMarshalUsage,
-    rules::flake8_bandit::rules::SuspiciousMktempUsage,
-    rules::flake8_bandit::rules::SuspiciousNonCryptographicRandomUsage,
-    rules::flake8_bandit::rules::SuspiciousPickleUsage,
-    rules::flake8_bandit::rules::SuspiciousTelnetUsage,
-    rules::flake8_bandit::rules::SuspiciousURLOpenUsage,
-    rules::flake8_bandit::rules::SuspiciousUnverifiedContextUsage,
-    rules::flake8_bandit::rules::SuspiciousXMLCElementTreeUsage,
-    rules::flake8_bandit::rules::SuspiciousXMLETreeUsage,
-    rules::flake8_bandit::rules::SuspiciousXMLElementTreeUsage,
-    rules::flake8_bandit::rules::SuspiciousXMLExpatBuilderUsage,
-    rules::flake8_bandit::rules::SuspiciousXMLExpatReaderUsage,
-    rules::flake8_bandit::rules::SuspiciousXMLMiniDOMUsage,
-    rules::flake8_bandit::rules::SuspiciousXMLPullDOMUsage,
-    rules::flake8_bandit::rules::SuspiciousXMLSaxUsage,
-    rules::flake8_bandit::rules::TryExceptContinue,
-    rules::flake8_bandit::rules::TryExceptPass,
-    rules::flake8_bandit::rules::UnsafeYAMLLoad,
+    rules::flake8_bandit::rules::LoggingConfigInsecureListen,
+    rules::flake8_bandit::rules::Jinja2AutoescapeFalse,
     // flake8-boolean-trap
     rules::flake8_boolean_trap::rules::BooleanPositionalArgInFunctionDefinition,
     rules::flake8_boolean_trap::rules::BooleanDefaultValueInFunctionDefinition,
@@ -514,18 +493,16 @@ ruff_macros::register_rules!(
     rules::flake8_errmsg::rules::FStringInException,
     rules::flake8_errmsg::rules::DotFormatInException,
     // flake8-pyi
-    rules::flake8_pyi::rules::ArgumentDefaultInStub,
-    rules::flake8_pyi::rules::AssignmentDefaultInStub,
-    rules::flake8_pyi::rules::BadVersionInfoComparison,
-    rules::flake8_pyi::rules::DocstringInStub,
-    rules::flake8_pyi::rules::NonEmptyStubBody,
-    rules::flake8_pyi::rules::PassStatementStubBody,
-    rules::flake8_pyi::rules::TypeCommentInStub,
-    rules::flake8_pyi::rules::TypedArgumentDefaultInStub,
     rules::flake8_pyi::rules::UnprefixedTypeParam,
+    rules::flake8_pyi::rules::BadVersionInfoComparison,
     rules::flake8_pyi::rules::UnrecognizedPlatformCheck,
     rules::flake8_pyi::rules::UnrecognizedPlatformName,
-    rules::flake8_pyi::rules::PassInClassBody,
+    rules::flake8_pyi::rules::PassStatementStubBody,
+    rules::flake8_pyi::rules::NonEmptyStubBody,
+    rules::flake8_pyi::rules::DocstringInStub,
+    rules::flake8_pyi::rules::TypedArgumentDefaultInStub,
+    rules::flake8_pyi::rules::ArgumentDefaultInStub,
+    rules::flake8_pyi::rules::TypeCommentInStub,
     // flake8-pytest-style
     rules::flake8_pytest_style::rules::PytestFixtureIncorrectParenthesesStyle,
     rules::flake8_pytest_style::rules::PytestFixturePositionalArgs,
@@ -590,31 +567,31 @@ ruff_macros::register_rules!(
     rules::tryceratops::rules::ErrorInsteadOfException,
     rules::tryceratops::rules::VerboseLogMessage,
     // flake8-use-pathlib
-    rules::flake8_use_pathlib::violations::OsPathAbspath,
-    rules::flake8_use_pathlib::violations::OsChmod,
-    rules::flake8_use_pathlib::violations::OsMkdir,
-    rules::flake8_use_pathlib::violations::OsMakedirs,
-    rules::flake8_use_pathlib::violations::OsRename,
+    rules::flake8_use_pathlib::violations::PathlibAbspath,
+    rules::flake8_use_pathlib::violations::PathlibChmod,
+    rules::flake8_use_pathlib::violations::PathlibMkdir,
+    rules::flake8_use_pathlib::violations::PathlibMakedirs,
+    rules::flake8_use_pathlib::violations::PathlibRename,
     rules::flake8_use_pathlib::violations::PathlibReplace,
-    rules::flake8_use_pathlib::violations::OsRmdir,
-    rules::flake8_use_pathlib::violations::OsRemove,
-    rules::flake8_use_pathlib::violations::OsUnlink,
-    rules::flake8_use_pathlib::violations::OsGetcwd,
-    rules::flake8_use_pathlib::violations::OsPathExists,
-    rules::flake8_use_pathlib::violations::OsPathExpanduser,
-    rules::flake8_use_pathlib::violations::OsPathIsdir,
-    rules::flake8_use_pathlib::violations::OsPathIsfile,
-    rules::flake8_use_pathlib::violations::OsPathIslink,
-    rules::flake8_use_pathlib::violations::OsReadlink,
-    rules::flake8_use_pathlib::violations::OsStat,
-    rules::flake8_use_pathlib::violations::OsPathIsabs,
-    rules::flake8_use_pathlib::violations::OsPathJoin,
-    rules::flake8_use_pathlib::violations::OsPathBasename,
-    rules::flake8_use_pathlib::violations::OsPathDirname,
-    rules::flake8_use_pathlib::violations::OsPathSamefile,
-    rules::flake8_use_pathlib::violations::OsPathSplitext,
-    rules::flake8_use_pathlib::violations::BuiltinOpen,
-    rules::flake8_use_pathlib::violations::PyPath,
+    rules::flake8_use_pathlib::violations::PathlibRmdir,
+    rules::flake8_use_pathlib::violations::PathlibRemove,
+    rules::flake8_use_pathlib::violations::PathlibUnlink,
+    rules::flake8_use_pathlib::violations::PathlibGetcwd,
+    rules::flake8_use_pathlib::violations::PathlibExists,
+    rules::flake8_use_pathlib::violations::PathlibExpanduser,
+    rules::flake8_use_pathlib::violations::PathlibIsDir,
+    rules::flake8_use_pathlib::violations::PathlibIsFile,
+    rules::flake8_use_pathlib::violations::PathlibIsLink,
+    rules::flake8_use_pathlib::violations::PathlibReadlink,
+    rules::flake8_use_pathlib::violations::PathlibStat,
+    rules::flake8_use_pathlib::violations::PathlibIsAbs,
+    rules::flake8_use_pathlib::violations::PathlibJoin,
+    rules::flake8_use_pathlib::violations::PathlibBasename,
+    rules::flake8_use_pathlib::violations::PathlibDirname,
+    rules::flake8_use_pathlib::violations::PathlibSamefile,
+    rules::flake8_use_pathlib::violations::PathlibSplitext,
+    rules::flake8_use_pathlib::violations::PathlibOpen,
+    rules::flake8_use_pathlib::violations::PathlibPyPath,
     // flake8-logging-format
     rules::flake8_logging_format::violations::LoggingStringFormat,
     rules::flake8_logging_format::violations::LoggingPercentFormat,
@@ -628,10 +605,6 @@ ruff_macros::register_rules!(
     rules::flake8_raise::rules::UnnecessaryParenOnRaiseException,
     // flake8-self
     rules::flake8_self::rules::PrivateMemberAccess,
-    // flake8-gettext
-    rules::flake8_gettext::rules::FStringInGetTextFuncCall,
-    rules::flake8_gettext::rules::FormatInGetTextFuncCall,
-    rules::flake8_gettext::rules::PrintfInGetTextFuncCall,
     // numpy
     rules::numpy::rules::NumpyDeprecatedTypeAlias,
     rules::numpy::rules::NumpyLegacyRandom,
@@ -649,7 +622,6 @@ ruff_macros::register_rules!(
     rules::flake8_django::rules::DjangoExcludeWithModelForm,
     rules::flake8_django::rules::DjangoAllWithModelForm,
     rules::flake8_django::rules::DjangoModelWithoutDunderStr,
-    rules::flake8_django::rules::DjangoUnorderedBodyContentInModel,
     rules::flake8_django::rules::DjangoNonLeadingReceiverDecorator,
 );
 
@@ -782,9 +754,6 @@ pub enum Linter {
     /// [flake8-type-checking](https://pypi.org/project/flake8-type-checking/)
     #[prefix = "TCH"]
     Flake8TypeChecking,
-    /// [flake8-gettext](https://pypi.org/project/flake8-gettext/)
-    #[prefix = "INT"]
-    Flake8GetText,
     /// [flake8-unused-arguments](https://pypi.org/project/flake8-unused-arguments/)
     #[prefix = "ARG"]
     Flake8UnusedArguments,
@@ -916,7 +885,7 @@ impl Rule {
             Rule::IOError => LintSource::Io,
             Rule::UnsortedImports | Rule::MissingRequiredImport => LintSource::Imports,
             Rule::ImplicitNamespacePackage | Rule::InvalidModuleName => LintSource::Filesystem,
-            #[cfg(feature = "logical_lines")]
+            #[cfg(debug_assertions)]
             Rule::IndentationWithInvalidMultiple
             | Rule::IndentationWithInvalidMultipleComment
             | Rule::MissingWhitespace

crates/ruff/src/registry/rule_set.rs

@@ -111,7 +111,7 @@ impl RuleSet {
         let mut i = 0;
 
         while i < self.0.len() {
-            self.0[i] &= !other.0[i];
+            self.0[i] ^= other.0[i];
             i += 1;
         }
 
@@ -362,15 +362,4 @@ mod tests {
         let expected_rules: Vec<_> = Rule::iter().collect();
         assert_eq!(all_rules, expected_rules);
     }
-
-    #[test]
-    fn remove_not_existing_rule_from_set() {
-        let mut set = RuleSet::default();
-
-        set.remove(Rule::AmbiguousFunctionName);
-
-        assert!(!set.contains(Rule::AmbiguousFunctionName));
-        assert!(set.is_empty());
-        assert_eq!(set.into_iter().collect::<Vec<_>>(), vec![]);
-    }
 }

crates/ruff/src/resolver.rs

@@ -41,7 +41,6 @@ impl PyprojectDiscovery {
 }
 
 /// The strategy for resolving file paths in a `pyproject.toml`.
-#[derive(Copy, Clone)]
 pub enum Relativity {
     /// Resolve file paths relative to the current working directory.
     Cwd,

crates/ruff/src/rule_selector.rs

@@ -265,7 +265,7 @@ impl RuleSelector {
     }
 }
 
-#[derive(EnumIter, PartialEq, Eq, PartialOrd, Ord, Copy, Clone)]
+#[derive(EnumIter, PartialEq, Eq, PartialOrd, Ord)]
 pub(crate) enum Specificity {
     All,
     LinterGroup,

crates/ruff/src/rules/eradicate/rules.rs

@@ -1,6 +1,6 @@
 use rustpython_parser::ast::Location;
 
-use ruff_diagnostics::{AlwaysAutofixableViolation, Diagnostic, Edit};
+use ruff_diagnostics::{AlwaysAutofixableViolation, Diagnostic, Fix};
 use ruff_macros::{derive_message_formats, violation};
 use ruff_python_ast::source_code::Locator;
 use ruff_python_ast::types::Range;
@@ -62,7 +62,7 @@ pub fn commented_out_code(
     if is_standalone_comment(line) && comment_contains_code(line, &settings.task_tags[..]) {
         let mut diagnostic = Diagnostic::new(CommentedOutCode, Range::new(start, end));
         if autofix.into() && settings.rules.should_fix(Rule::CommentedOutCode) {
-            diagnostic.set_fix(Edit::deletion(location, end_location));
+            diagnostic.amend(Fix::deletion(location, end_location));
         }
         Some(diagnostic)
     } else {

crates/ruff/src/rules/eradicate/snapshots/ruff__rules__eradicate__tests__ERA001_ERA001.py.snap

@@ -14,14 +14,13 @@ expression: diagnostics
     row: 1
     column: 10
   fix:
-    edits:
-      - content: ""
-        location:
-          row: 1
-          column: 0
-        end_location:
-          row: 2
-          column: 0
+    content: ""
+    location:
+      row: 1
+      column: 0
+    end_location:
+      row: 2
+      column: 0
   parent: ~
 - kind:
     name: CommentedOutCode
@@ -35,14 +34,13 @@ expression: diagnostics
     row: 2
     column: 22
   fix:
-    edits:
-      - content: ""
-        location:
-          row: 2
-          column: 0
-        end_location:
-          row: 3
-          column: 0
+    content: ""
+    location:
+      row: 2
+      column: 0
+    end_location:
+      row: 3
+      column: 0
   parent: ~
 - kind:
     name: CommentedOutCode
@@ -56,14 +54,13 @@ expression: diagnostics
     row: 3
     column: 6
   fix:
-    edits:
-      - content: ""
-        location:
-          row: 3
-          column: 0
-        end_location:
-          row: 4
-          column: 0
+    content: ""
+    location:
+      row: 3
+      column: 0
+    end_location:
+      row: 4
+      column: 0
   parent: ~
 - kind:
     name: CommentedOutCode
@@ -77,14 +74,13 @@ expression: diagnostics
     row: 5
     column: 13
   fix:
-    edits:
-      - content: ""
-        location:
-          row: 5
-          column: 0
-        end_location:
-          row: 6
-          column: 0
+    content: ""
+    location:
+      row: 5
+      column: 0
+    end_location:
+      row: 6
+      column: 0
   parent: ~
 - kind:
     name: CommentedOutCode
@@ -98,13 +94,12 @@ expression: diagnostics
     row: 12
     column: 16
   fix:
-    edits:
-      - content: ""
-        location:
-          row: 12
-          column: 0
-        end_location:
-          row: 13
-          column: 0
+    content: ""
+    location:
+      row: 12
+      column: 0
+    end_location:
+      row: 13
+      column: 0
   parent: ~
 

crates/ruff/src/rules/flake8_2020/snapshots/ruff__rules__flake8_2020__tests__YTT101_YTT101.py.snap

@@ -13,8 +13,7 @@ expression: diagnostics
   end_location:
     row: 6
     column: 17
-  fix:
-    edits: []
+  fix: ~
   parent: ~
 - kind:
     name: SysVersionSlice3
@@ -27,8 +26,7 @@ expression: diagnostics
   end_location:
     row: 7
     column: 13
-  fix:
-    edits: []
+  fix: ~
   parent: ~
 - kind:
     name: SysVersionSlice3
@@ -41,7 +39,6 @@ expression: diagnostics
   end_location:
     row: 8
     column: 7
-  fix:
-    edits: []
+  fix: ~
   parent: ~
 

crates/ruff/src/rules/flake8_2020/snapshots/ruff__rules__flake8_2020__tests__YTT102_YTT102.py.snap

@@ -13,8 +13,7 @@ expression: diagnostics
   end_location:
     row: 4
     column: 22
-  fix:
-    edits: []
+  fix: ~
   parent: ~
 - kind:
     name: SysVersion2
@@ -27,7 +26,6 @@ expression: diagnostics
   end_location:
     row: 5
     column: 18
-  fix:
-    edits: []
+  fix: ~
   parent: ~
 

crates/ruff/src/rules/flake8_2020/snapshots/ruff__rules__flake8_2020__tests__YTT103_YTT103.py.snap

@@ -13,8 +13,7 @@ expression: diagnostics
   end_location:
     row: 4
     column: 7
-  fix:
-    edits: []
+  fix: ~
   parent: ~
 - kind:
     name: SysVersionCmpStr3
@@ -27,8 +26,7 @@ expression: diagnostics
   end_location:
     row: 5
     column: 11
-  fix:
-    edits: []
+  fix: ~
   parent: ~
 - kind:
     name: SysVersionCmpStr3
@@ -41,8 +39,7 @@ expression: diagnostics
   end_location:
     row: 6
     column: 11
-  fix:
-    edits: []
+  fix: ~
   parent: ~
 - kind:
     name: SysVersionCmpStr3
@@ -55,8 +52,7 @@ expression: diagnostics
   end_location:
     row: 7
     column: 11
-  fix:
-    edits: []
+  fix: ~
   parent: ~
 - kind:
     name: SysVersionCmpStr3
@@ -69,7 +65,6 @@ expression: diagnostics
   end_location:
     row: 8
     column: 11
-  fix:
-    edits: []
+  fix: ~
   parent: ~
 

crates/ruff/src/rules/flake8_2020/snapshots/ruff__rules__flake8_2020__tests__YTT201_YTT201.py.snap

@@ -13,8 +13,7 @@ expression: diagnostics
   end_location:
     row: 7
     column: 25
-  fix:
-    edits: []
+  fix: ~
   parent: ~
 - kind:
     name: SysVersionInfo0Eq3
@@ -27,8 +26,7 @@ expression: diagnostics
   end_location:
     row: 8
     column: 21
-  fix:
-    edits: []
+  fix: ~
   parent: ~
 - kind:
     name: SysVersionInfo0Eq3
@@ -41,8 +39,7 @@ expression: diagnostics
   end_location:
     row: 9
     column: 25
-  fix:
-    edits: []
+  fix: ~
   parent: ~
 - kind:
     name: SysVersionInfo0Eq3
@@ -55,7 +52,6 @@ expression: diagnostics
   end_location:
     row: 10
     column: 21
-  fix:
-    edits: []
+  fix: ~
   parent: ~
 

crates/ruff/src/rules/flake8_2020/snapshots/ruff__rules__flake8_2020__tests__YTT202_YTT202.py.snap

@@ -13,8 +13,7 @@ expression: diagnostics
   end_location:
     row: 4
     column: 10
-  fix:
-    edits: []
+  fix: ~
   parent: ~
 - kind:
     name: SixPY3
@@ -27,7 +26,6 @@ expression: diagnostics
   end_location:
     row: 6
     column: 6
-  fix:
-    edits: []
+  fix: ~
   parent: ~