Use rustup-show instead of actions-rs

.cargo/config.toml

@@ -1,28 +1,2 @@
 [alias]
 dev = "run --package ruff_dev --bin ruff_dev"
-
-[target.'cfg(all())']
-rustflags = [
-  # CLIPPY LINT SETTINGS
-  # This is a workaround to configure lints for the entire workspace, pending the ability to configure this via TOML.
-  # See: `https://github.com/rust-lang/cargo/issues/5034`
-  #      `https://github.com/EmbarkStudios/rust-ecosystem/issues/22#issuecomment-947011395`
-  "-Dunsafe_code",
-  "-Wclippy::pedantic",
-  # Allowed pedantic lints
-  "-Wclippy::char_lit_as_u8",
-  "-Aclippy::collapsible_else_if",
-  "-Aclippy::collapsible_if",
-  "-Aclippy::implicit_hasher",
-  "-Aclippy::match_same_arms",
-  "-Aclippy::missing_errors_doc",
-  "-Aclippy::missing_panics_doc",
-  "-Aclippy::module_name_repetitions",
-  "-Aclippy::must_use_candidate",
-  "-Aclippy::similar_names",
-  "-Aclippy::too_many_lines",
-  # Disallowed restriction lints
-  "-Wclippy::print_stdout",
-  "-Wclippy::print_stderr",
-  "-Wclippy::dbg_macro",
-]

.github/CODE_OF_CONDUCT.md

@@ -106,7 +106,7 @@ Violating these terms may lead to a permanent ban.
 ### 4. Permanent Ban
 
 **Community Impact**: Demonstrating a pattern of violation of community
-standards, including sustained inappropriate behavior, harassment of an
+standards, including sustained inappropriate behavior,  harassment of an
 individual, or aggression toward or disparagement of classes of individuals.
 
 **Consequence**: A permanent ban from any sort of public interaction within
@@ -115,12 +115,14 @@ the community.
 ## Attribution
 
 This Code of Conduct is adapted from the [Contributor Covenant][homepage],
-version 2.0, available [here](https://www.contributor-covenant.org/version/2/0/code_of_conduct.html).
+version 2.0, available at
+https://www.contributor-covenant.org/version/2/0/code_of_conduct.html.
 
 Community Impact Guidelines were inspired by [Mozilla's code of conduct
 enforcement ladder](https://github.com/mozilla/diversity).
 
 [homepage]: https://www.contributor-covenant.org
 
-For answers to common questions about this code of conduct, see the [FAQ](https://www.contributor-covenant.org/faq).
-Translations are available [here](https://www.contributor-covenant.org/translations).
+For answers to common questions about this code of conduct, see the FAQ at
+https://www.contributor-covenant.org/faq. Translations are available at
+https://www.contributor-covenant.org/translations.

.github/CONTRIBUTING.md

@@ -0,0 +1,125 @@
+# Contributing to Ruff
+
+Welcome! We're happy to have you here. Thank you in advance for your contribution to Ruff.
+
+## The basics
+
+Ruff welcomes contributions in the form of Pull Requests. For small changes (e.g., bug fixes), feel
+free to submit a PR. For larger changes (e.g., new lint rules, new functionality, new configuration
+options), consider submitting an [Issue](https://github.com/charliermarsh/ruff/issues) outlining
+your proposed change.
+
+If you're looking for a place to start, we recommend implementing a new lint rule (see:
+[_Adding a new lint rule_](#example-adding-a-new-lint-rule), which will allow you to learn from and
+pattern-match against the examples in the existing codebase. Many lint rules are inspired by
+existing Python plugins, which can be used as a reference implementation.
+
+As a concrete example: consider taking on one of the rules in [`flake8-simplify`](https://github.com/charliermarsh/ruff/issues/998),
+and looking to the originating [Python source](https://github.com/MartinThoma/flake8-simplify) for
+guidance.
+
+### Prerequisites
+
+Ruff is written in Rust. You'll need to install the
+[Rust toolchain](https://www.rust-lang.org/tools/install) for development.
+
+You'll also need [Insta](https://insta.rs/docs/) to update snapshot tests:
+
+```shell
+cargo install cargo-insta
+```
+
+### Development
+
+After cloning the repository, run Ruff locally with:
+
+```shell
+cargo run resources/test/fixtures --no-cache
+```
+
+Prior to opening a pull request, ensure that your code has been auto-formatted, and that it passes
+both the lint and test validation checks:
+
+```shell
+cargo +nightly fmt --all     # Auto-formatting...
+cargo +nightly clippy --all  # Linting...
+cargo +nightly test --all    # Testing...
+```
+
+These checks will run on GitHub Actions when you open your Pull Request, but running them locally
+will save you time and expedite the merge process.
+
+Your Pull Request will be reviewed by a maintainer, which may involve a few rounds of iteration
+prior to merging.
+
+### Example: Adding a new lint rule
+
+There are four phases to adding a new lint rule:
+
+1. Define the violation struct in `src/violations.rs` (e.g., `ModuleImportNotAtTopOfFile`).
+2. Map the violation struct to a rule code in `src/registry.rs` (e.g., `E402`).
+3. Define the logic for triggering the violation in `src/checkers/ast.rs` (for AST-based checks),
+   `src/checkers/tokens.rs` (for token-based checks), or `src/checkers/lines.rs` (for text-based checks).
+4. Add a test fixture.
+5. Update the generated files (documentation and generated code).
+
+To define the violation, open up `src/violations.rs`, and define a new struct using the
+`define_violation!` macro. There are plenty of examples in that file, so feel free to pattern-match
+against the existing structs.
+
+To trigger the violation, you'll likely want to augment the logic in `src/checkers/ast.rs`, which
+defines the Python AST visitor, responsible for iterating over the abstract syntax tree and
+collecting diagnostics as it goes.
+
+If you need to inspect the AST, you can run `cargo +nightly dev print-ast` with a Python file. Grep
+for the `Check::new` invocations to understand how other, similar rules are implemented.
+
+To add a test fixture, create a file under `resources/test/fixtures/[origin]`, named to match
+the code you defined earlier (e.g., `resources/test/fixtures/pycodestyle/E402.py`). This file should
+contain a variety of violations and non-violations designed to evaluate and demonstrate the behavior
+of your lint rule.
+
+Run `cargo +nightly dev generate-all` to generate the code for your new fixture. Then run Ruff
+locally with (e.g.) `cargo run resources/test/fixtures/pycodestyle/E402.py --no-cache --select E402`.
+
+Once you're satisfied with the output, codify the behavior as a snapshot test by adding a new
+`test_case` macro in the relevant `src/[origin]/mod.rs` file. Then, run `cargo test --all`.
+Your test will fail, but you'll be prompted to follow-up with `cargo insta review`. Accept the
+generated snapshot, then commit the snapshot file alongside the rest of your changes.
+
+Finally, regenerate the documentation and generated code with `cargo +nightly dev generate-all`.
+
+### Example: Adding a new configuration option
+
+Ruff's user-facing settings live in a few different places.
+
+First, the command-line options are defined via the `Cli` struct in `src/cli.rs`.
+
+Second, the `pyproject.toml` options are defined in `src/settings/options.rs` (via the `Options`
+struct), `src/settings/configuration.rs` (via the `Configuration` struct), and `src/settings/mod.rs`
+(via the `Settings` struct). These represent, respectively: the schema used to parse the
+`pyproject.toml` file; an internal, intermediate representation; and the final, internal
+representation used to power Ruff.
+
+To add a new configuration option, you'll likely want to modify these latter few files (along with
+`cli.rs`, if appropriate). If you want to pattern-match against an existing example, grep for
+`dummy_variable_rgx`, which defines a regular expression to match against acceptable unused
+variables (e.g., `_`).
+
+Note that plugin-specific configuration options are defined in their own modules (e.g.,
+`src/flake8_unused_arguments/settings.rs`).
+
+You may also want to add the new configuration option to the `flake8-to-ruff` tool, which is
+responsible for converting `flake8` configuration files to Ruff's TOML format. This logic
+lives in `flake8_to_ruff/src/converter.rs`.
+
+Finally, regenerate the documentation and generated code with `cargo +nightly dev generate-all`.
+
+## Release process
+
+As of now, Ruff has an ad hoc release process: releases are cut with high frequency via GitHub
+Actions, which automatically generates the appropriate wheels across architectures and publishes
+them to [PyPI](https://pypi.org/project/ruff/).
+
+Ruff follows the [semver](https://semver.org/) versioning standard. However, as pre-1.0 software,
+even patch releases may contain [non-backwards-compatible changes](https://semver.org/#spec-item-4).

.github/ISSUE_TEMPLATE.md

@@ -3,8 +3,8 @@ Thank you for taking the time to report an issue! We're glad to have you involve
 
 If you're filing a bug report, please consider including the following information:
 
-* A minimal code snippet that reproduces the bug.
-* The command you invoked (e.g., `ruff /path/to/file.py --fix`), ideally including the `--isolated` flag.
-* The current Ruff settings (any relevant sections from your `pyproject.toml`).
-* The current Ruff version (`ruff --version`).
+- A minimal code snippet that reproduces the bug.
+- The command you invoked (e.g., `ruff /path/to/file.py --fix`), ideally including the `--isolated` flag.
+- The current Ruff settings (any relevant sections from your `pyproject.toml`).
+- The current Ruff version (`ruff --version`).
 -->

.github/release.yml

@@ -1,19 +0,0 @@
-# https://docs.github.com/en/repositories/releasing-projects-on-github/automatically-generated-release-notes#configuring-automatically-generated-release-notes
-changelog:
-  categories:
-    - title: Breaking Changes
-      labels:
-        - breaking
-    - title: Rules
-      labels:
-        - rule
-        - autofix
-    - title: Settings
-      labels:
-        - configuration
-    - title: Bug Fixes
-      labels:
-        - bug
-    - title: Other Changes
-      labels:
-        - "*"

.github/workflows/ci.yaml

@@ -4,7 +4,7 @@ on:
   push:
     branches: [main]
   pull_request:
-  workflow_dispatch:
+    branches: [main]
 
 concurrency:
   group: ${{ github.workflow }}-${{ github.ref_name }}-${{ github.event.pull_request.number || github.sha }}
@@ -13,7 +13,6 @@ concurrency:
 env:
   CARGO_INCREMENTAL: 0
   CARGO_NET_RETRY: 10
-  CARGO_TERM_COLOR: always
   RUSTUP_MAX_RETRIES: 10
 
 jobs:
@@ -22,13 +21,12 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v3
-      - name: "Install Rust toolchain"
-        run: rustup show
+      - run: rustup show
       - uses: Swatinem/rust-cache@v1
       - run: cargo build --all
       - run: ./target/debug/ruff_dev generate-all
-      - run: git diff --quiet README.md || echo "::error file=README.md::This file is outdated. Run 'cargo dev generate-all'."
-      - run: git diff --quiet ruff.schema.json || echo "::error file=ruff.schema.json::This file is outdated. Run 'cargo dev generate-all'."
+      - run: git diff --quiet README.md || echo "::error file=README.md::This file is outdated. Run 'cargo +nightly dev generate-all'."
+      - run: git diff --quiet ruff.schema.json || echo "::error file=ruff.schema.json::This file is outdated. Run 'cargo +nightly dev generate-all'."
       - run: git diff --exit-code -- README.md ruff.schema.json
 
   cargo-fmt:
@@ -36,8 +34,7 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v3
-      - name: "Install Rust toolchain"
-        run: rustup component add rustfmt
+      - run: rustup show
       - run: cargo fmt --all --check
 
   cargo_clippy:
@@ -45,80 +42,73 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v3
-      - name: "Install Rust toolchain"
-        run: |
-          rustup component add clippy
-          rustup target add wasm32-unknown-unknown
+      - run: rustup show
       - uses: Swatinem/rust-cache@v1
-      - run: cargo clippy --workspace --all-targets --all-features -- -D warnings
-      - run: cargo clippy -p ruff --target wasm32-unknown-unknown --all-features -- -D warnings
+      - run: cargo clippy --workspace --all-targets --all-features -- -D warnings -W clippy::pedantic
+      - run: cargo clippy --workspace --target wasm32-unknown-unknown --all-features -- -D warnings -W clippy::pedantic
 
   cargo-test:
-    strategy:
-      matrix:
-        os: [ubuntu-latest, windows-latest]
-    runs-on: ${{ matrix.os }}
-    name: "cargo test | ${{ matrix.os }}"
-    steps:
-      - uses: actions/checkout@v3
-      - name: "Install Rust toolchain"
-        run: rustup show
-      - uses: Swatinem/rust-cache@v1
-      - run: cargo install cargo-insta
-      - run: pip install black[d]==22.12.0
-      - name: "Run tests (Ubuntu)"
-        if: ${{ matrix.os == 'ubuntu-latest' }}
-        run: |
-          cargo insta test --all --all-features --delete-unreferenced-snapshots
-          git diff --exit-code
-      - name: "Run tests (Windows)"
-        if: ${{ matrix.os == 'windows-latest' }}
-        shell: bash
-        run: |
-          cargo insta test --all --all-features
-          git diff --exit-code
-      - run: cargo test --package ruff_cli --test black_compatibility_test -- --ignored
-      # Check for broken links in the documentation.
-      - run: cargo doc --all --no-deps
-        env:
-          # Setting RUSTDOCFLAGS because `cargo doc --check` isn't yet implemented (https://github.com/rust-lang/cargo/issues/10025).
-          RUSTDOCFLAGS: "-D warnings"
-
-  scripts:
-    name: "test scripts"
+    name: "cargo test"
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v3
-      - name: "Install Rust toolchain"
-        run: rustup show
+      - run: rustup show
       - uses: Swatinem/rust-cache@v1
-      - run: ./scripts/add_rule.py --name DoTheThing --code PLC999 --linter pylint
-      - run: cargo check
-      - run: |
-          ./scripts/add_plugin.py test --url https://pypi.org/project/-test/0.1.0/ --prefix TST
-          ./scripts/add_rule.py --name FirstRule --code TST001 --linter test
-      - run: cargo check
+      - run: cargo install cargo-insta
+      - run: pip install black[d]==22.12.0
+      - name: Run tests
+        run: |
+          cargo insta test --all --delete-unreferenced-snapshots
+          git diff --exit-code
+      - run: cargo test --package ruff --test black_compatibility_test -- --ignored
+
+# TODO(charlie): Re-enable the `wasm-pack` tests.
+# See: https://github.com/charliermarsh/ruff/issues/1425
+#  wasm-pack-test:
+#    name: "wasm-pack test"
+#    runs-on: ubuntu-latest
+#    env:
+#      WASM_BINDGEN_TEST_TIMEOUT: 60
+#    steps:
+#      - uses: actions/checkout@v3
+#      - run: rustup show
+#      - uses: actions/cache@v3
+#        env:
+#          cache-name: cache-cargo
+#        with:
+#          path: |
+#            ~/.cargo/registry
+#            ~/.cargo/git
+#          key: ${{ runner.os }}-build-${{ env.cache-name }}-${{ hashFiles('**/Cargo.lock') }}
+#          restore-keys: |
+#            ${{ runner.os }}-build-${{ env.cache-name }}-
+#            ${{ runner.os }}-build-
+#            ${{ runner.os }}-
+#      - uses: jetli/wasm-pack-action@v0.4.0
+#      - uses: jetli/wasm-bindgen-action@v0.2.0
+#      - run: wasm-pack test --node
 
   maturin-build:
     name: "maturin build"
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v3
-      - name: "Install Rust toolchain"
-        run: rustup show
+      - run: rustup show
       - uses: Swatinem/rust-cache@v1
       - uses: actions/setup-python@v4
         with:
           python-version: "3.11"
       - run: pip install maturin
       - run: maturin build -b bin
-      - run: python scripts/transform_readme.py --target pypi
 
   typos:
-    name: "spell check"
+    name: Spell Check with Typos
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@v3
-      - uses: crate-ci/typos@master
-        with:
-          files: .
+    - name: Checkout Actions Repository
+      uses: actions/checkout@v2
+
+    - name: Check spelling of file.txt
+      uses: crate-ci/typos@master
+      with:
+        files: .

.github/workflows/docs.yaml

@@ -1,34 +0,0 @@
-name: mkdocs
-
-on:
-  push:
-    paths:
-      - README.md
-      - mkdocs.template.yml
-      - .github/workflows/docs.yaml
-    branches: [main]
-  workflow_dispatch:
-
-jobs:
-  mkdocs:
-    runs-on: ubuntu-latest
-    env:
-      CF_API_TOKEN_EXISTS: ${{ secrets.CF_API_TOKEN != '' }}
-    steps:
-      - uses: actions/checkout@v3
-      - uses: actions/setup-python@v4
-      - name: "Install dependencies"
-        run: |
-          pip install "mkdocs~=1.4.2" "mkdocs-material~=9.0.6"
-      - name: "Copy README File"
-        run: |
-          python scripts/transform_readme.py --target mkdocs
-          python scripts/generate_mkdocs.py
-          mkdocs build --strict
-      - name: "Deploy to Cloudflare Pages"
-        if: ${{ env.CF_API_TOKEN_EXISTS == 'true' }}
-        uses: cloudflare/wrangler-action@2.0.0
-        with:
-          apiToken: ${{ secrets.CF_API_TOKEN }}
-          accountId: ${{ secrets.CF_ACCOUNT_ID }}
-          command: pages publish site --project-name=ruff-docs --branch ${GITHUB_HEAD_REF} --commit-hash ${GITHUB_SHA}

.github/workflows/flake8-to-ruff.yaml

@@ -12,7 +12,6 @@ env:
   PYTHON_VERSION: "3.7" # to build abi3 wheels
   CARGO_INCREMENTAL: 0
   CARGO_NET_RETRY: 10
-  CARGO_TERM_COLOR: always
   RUSTUP_MAX_RETRIES: 10
 
 jobs:
@@ -24,18 +23,18 @@ jobs:
         with:
           python-version: ${{ env.PYTHON_VERSION }}
           architecture: x64
-      - name: "Install Rust toolchain"
+      - name: Install Rust toolchain
         run: rustup show
-      - name: "Build wheels - x86_64"
+      - name: Build wheels - x86_64
         uses: messense/maturin-action@v1
         with:
           target: x86_64
           args: --release --out dist --sdist -m ./${{ env.CRATE_NAME }}/Cargo.toml
-      - name: "Install built wheel - x86_64"
+      - name: Install built wheel - x86_64
         run: |
           pip install dist/${{ env.CRATE_NAME }}-*.whl --force-reinstall
-      - name: "Upload wheels"
-        uses: actions/upload-artifact@v3
+      - name: Upload wheels
+        uses: actions/upload-artifact@v2
         with:
           name: wheels
           path: dist
@@ -47,17 +46,17 @@ jobs:
         with:
           python-version: ${{ env.PYTHON_VERSION }}
           architecture: x64
-      - name: "Install Rust toolchain"
+      - name: Install Rust toolchain
         run: rustup show
-      - name: "Build wheels - universal2"
+      - name: Build wheels - universal2
         uses: messense/maturin-action@v1
         with:
           args: --release --universal2 --out dist -m ./${{ env.CRATE_NAME }}/Cargo.toml
-      - name: "Install built wheel - universal2"
+      - name: Install built wheel - universal2
         run: |
           pip install dist/${{ env.CRATE_NAME }}-*universal2.whl --force-reinstall
-      - name: "Upload wheels"
-        uses: actions/upload-artifact@v3
+      - name: Upload wheels
+        uses: actions/upload-artifact@v2
         with:
           name: wheels
           path: dist
@@ -73,19 +72,19 @@ jobs:
         with:
           python-version: ${{ env.PYTHON_VERSION }}
           architecture: ${{ matrix.target }}
-      - name: "Install Rust toolchain"
+      - name: Install Rust toolchain
         run: rustup show
-      - name: "Build wheels"
+      - name: Build wheels
         uses: messense/maturin-action@v1
         with:
           target: ${{ matrix.target }}
           args: --release --out dist -m ./${{ env.CRATE_NAME }}/Cargo.toml
-      - name: "Install built wheel"
+      - name: Install built wheel
         shell: bash
         run: |
           python -m pip install dist/${{ env.CRATE_NAME }}-*.whl --force-reinstall
-      - name: "Upload wheels"
-        uses: actions/upload-artifact@v3
+      - name: Upload wheels
+        uses: actions/upload-artifact@v2
         with:
           name: wheels
           path: dist
@@ -101,18 +100,18 @@ jobs:
         with:
           python-version: ${{ env.PYTHON_VERSION }}
           architecture: x64
-      - name: "Build wheels"
+      - name: Build wheels
         uses: messense/maturin-action@v1
         with:
           target: ${{ matrix.target }}
           manylinux: auto
           args: --release --out dist -m ./${{ env.CRATE_NAME }}/Cargo.toml
-      - name: "Install built wheel"
+      - name: Install built wheel
         if: matrix.target == 'x86_64'
         run: |
           pip install dist/${{ env.CRATE_NAME }}-*.whl --force-reinstall
-      - name: "Upload wheels"
-        uses: actions/upload-artifact@v3
+      - name: Upload wheels
+        uses: actions/upload-artifact@v2
         with:
           name: wheels
           path: dist
@@ -127,13 +126,13 @@ jobs:
       - uses: actions/setup-python@v4
         with:
           python-version: ${{ env.PYTHON_VERSION }}
-      - name: "Build wheels"
+      - name: Build wheels
         uses: messense/maturin-action@v1
         with:
           target: ${{ matrix.target }}
           manylinux: auto
           args: --no-default-features --release --out dist -m ./${{ env.CRATE_NAME }}/Cargo.toml
-      - uses: uraimo/run-on-arch-action@v2.5.0
+      - uses: uraimo/run-on-arch-action@v2.0.5
         if: matrix.target != 'ppc64'
         name: Install built wheel
         with:
@@ -146,8 +145,8 @@ jobs:
             pip3 install -U pip
           run: |
             pip3 install ${{ env.PACKAGE_NAME }} --no-index --find-links dist/ --force-reinstall
-      - name: "Upload wheels"
-        uses: actions/upload-artifact@v3
+      - name: Upload wheels
+        uses: actions/upload-artifact@v2
         with:
           name: wheels
           path: dist
@@ -165,13 +164,13 @@ jobs:
         with:
           python-version: ${{ env.PYTHON_VERSION }}
           architecture: x64
-      - name: "Build wheels"
+      - name: Build wheels
         uses: messense/maturin-action@v1
         with:
           target: ${{ matrix.target }}
           manylinux: musllinux_1_2
           args: --release --out dist -m ./${{ env.CRATE_NAME }}/Cargo.toml
-      - name: "Install built wheel"
+      - name: Install built wheel
         if: matrix.target == 'x86_64-unknown-linux-musl'
         uses: addnab/docker-run-action@v3
         with:
@@ -180,8 +179,8 @@ jobs:
           run: |
             apk add py3-pip
             pip3 install ${{ env.PACKAGE_NAME }} --no-index --find-links /io/dist/ --force-reinstall
-      - name: "Upload wheels"
-        uses: actions/upload-artifact@v3
+      - name: Upload wheels
+        uses: actions/upload-artifact@v2
         with:
           name: wheels
           path: dist
@@ -200,7 +199,7 @@ jobs:
       - uses: actions/setup-python@v4
         with:
           python-version: ${{ env.PYTHON_VERSION }}
-      - name: "Build wheels"
+      - name: Build wheels
         uses: messense/maturin-action@v1
         with:
           target: ${{ matrix.platform.target }}
@@ -216,8 +215,8 @@ jobs:
             apk add py3-pip
           run: |
             pip3 install ${{ env.PACKAGE_NAME }} --no-index --find-links dist/ --force-reinstall
-      - name: "Upload wheels"
-        uses: actions/upload-artifact@v3
+      - name: Upload wheels
+        uses: actions/upload-artifact@v2
         with:
           name: wheels
           path: dist
@@ -240,18 +239,18 @@ jobs:
       - uses: actions/setup-python@v4
         with:
           python-version: pypy${{ matrix.python-version }}
-      - name: "Build wheels"
+      - name: Build wheels
         uses: messense/maturin-action@v1
         with:
           target: ${{ matrix.target }}
           manylinux: auto
           args: --release --out dist -i pypy${{ matrix.python-version }} -m ./${{ env.CRATE_NAME }}/Cargo.toml
-      - name: "Install built wheel"
+      - name: Install built wheel
         if: matrix.target == 'x86_64'
         run: |
           pip install dist/${{ env.CRATE_NAME }}-*.whl --force-reinstall
-      - name: "Upload wheels"
-        uses: actions/upload-artifact@v3
+      - name: Upload wheels
+        uses: actions/upload-artifact@v2
         with:
           name: wheels
           path: dist
@@ -269,11 +268,11 @@ jobs:
       - musllinux-cross
       - pypy
     steps:
-      - uses: actions/download-artifact@v3
+      - uses: actions/download-artifact@v2
         with:
           name: wheels
       - uses: actions/setup-python@v4
-      - name: "Publish to PyPi"
+      - name: Publish to PyPi
         env:
           TWINE_USERNAME: __token__
           TWINE_PASSWORD: ${{ secrets.FLAKE8_TO_RUFF_TOKEN }}

.github/workflows/playground.yaml

@@ -8,18 +8,16 @@ on:
 env:
   CARGO_INCREMENTAL: 0
   CARGO_NET_RETRY: 10
-  CARGO_TERM_COLOR: always
   RUSTUP_MAX_RETRIES: 10
 
 jobs:
   publish:
     runs-on: ubuntu-latest
     env:
-      CF_API_TOKEN_EXISTS: ${{ secrets.CF_API_TOKEN != '' }}
+       CF_API_TOKEN_EXISTS: ${{ secrets.CF_API_TOKEN != '' }}
     steps:
       - uses: actions/checkout@v3
-      - name: "Install Rust toolchain"
-        run: rustup target add wasm32-unknown-unknown
+      - run: rustup show
       - uses: actions/setup-node@v3
         with:
           node-version: 18
@@ -28,7 +26,7 @@ jobs:
       - uses: jetli/wasm-pack-action@v0.4.0
       - uses: jetli/wasm-bindgen-action@v0.2.0
       - name: "Run wasm-pack"
-        run: wasm-pack build --target web --out-dir ../../playground/src/pkg crates/ruff
+        run: wasm-pack build --target web --out-dir playground/src/pkg
       - name: "Install Node dependencies"
         run: npm ci
         working-directory: playground

.github/workflows/ruff.yaml

@@ -1,7 +1,6 @@
 name: "[ruff] Release"
 
 on:
-  workflow_dispatch:
   release:
     types: [published]
 
@@ -14,7 +13,6 @@ env:
   PYTHON_VERSION: "3.7" # to build abi3 wheels
   CARGO_INCREMENTAL: 0
   CARGO_NET_RETRY: 10
-  CARGO_TERM_COLOR: always
   RUSTUP_MAX_RETRIES: 10
 
 jobs:
@@ -26,18 +24,18 @@ jobs:
         with:
           python-version: ${{ env.PYTHON_VERSION }}
           architecture: x64
-      - name: "Prep README.md"
-        run: python scripts/transform_readme.py --target pypi
-      - name: "Build wheels - x86_64"
+      - name: Install Rust toolchain
+        run: rustup show
+      - name: Build wheels - x86_64
         uses: messense/maturin-action@v1
         with:
           target: x86_64
           args: --release --out dist --sdist
-      - name: "Install built wheel - x86_64"
+      - name: Install built wheel - x86_64
         run: |
           pip install dist/${{ env.PACKAGE_NAME }}-*.whl --force-reinstall
-      - name: "Upload wheels"
-        uses: actions/upload-artifact@v3
+      - name: Upload wheels
+        uses: actions/upload-artifact@v2
         with:
           name: wheels
           path: dist
@@ -49,17 +47,17 @@ jobs:
         with:
           python-version: ${{ env.PYTHON_VERSION }}
           architecture: x64
-      - name: "Prep README.md"
-        run: python scripts/transform_readme.py --target pypi
-      - name: "Build wheels - universal2"
+      - name: Install Rust toolchain
+        run: rustup show
+      - name: Build wheels - universal2
         uses: messense/maturin-action@v1
         with:
           args: --release --universal2 --out dist
-      - name: "Install built wheel - universal2"
+      - name: Install built wheel - universal2
         run: |
           pip install dist/${{ env.PACKAGE_NAME }}-*universal2.whl --force-reinstall
-      - name: "Upload wheels"
-        uses: actions/upload-artifact@v3
+      - name: Upload wheels
+        uses: actions/upload-artifact@v2
         with:
           name: wheels
           path: dist
@@ -75,19 +73,19 @@ jobs:
         with:
           python-version: ${{ env.PYTHON_VERSION }}
           architecture: ${{ matrix.target }}
-      - name: "Prep README.md"
-        run: python scripts/transform_readme.py --target pypi
-      - name: "Build wheels"
+      - name: Install Rust toolchain
+        run: rustup show
+      - name: Build wheels
         uses: messense/maturin-action@v1
         with:
           target: ${{ matrix.target }}
           args: --release --out dist
-      - name: "Install built wheel"
+      - name: Install built wheel
         shell: bash
         run: |
           python -m pip install dist/${{ env.PACKAGE_NAME }}-*.whl --force-reinstall
-      - name: "Upload wheels"
-        uses: actions/upload-artifact@v3
+      - name: Upload wheels
+        uses: actions/upload-artifact@v2
         with:
           name: wheels
           path: dist
@@ -103,20 +101,18 @@ jobs:
         with:
           python-version: ${{ env.PYTHON_VERSION }}
           architecture: x64
-      - name: "Prep README.md"
-        run: python scripts/transform_readme.py --target pypi
-      - name: "Build wheels"
+      - name: Build wheels
         uses: messense/maturin-action@v1
         with:
           target: ${{ matrix.target }}
           manylinux: auto
           args: --release --out dist
-      - name: "Install built wheel"
+      - name: Install built wheel
         if: matrix.target == 'x86_64'
         run: |
           pip install dist/${{ env.PACKAGE_NAME }}-*.whl --force-reinstall
-      - name: "Upload wheels"
-        uses: actions/upload-artifact@v3
+      - name: Upload wheels
+        uses: actions/upload-artifact@v2
         with:
           name: wheels
           path: dist
@@ -131,15 +127,13 @@ jobs:
       - uses: actions/setup-python@v4
         with:
           python-version: ${{ env.PYTHON_VERSION }}
-      - name: "Prep README.md"
-        run: python scripts/transform_readme.py --target pypi
-      - name: "Build wheels"
+      - name: Build wheels
         uses: messense/maturin-action@v1
         with:
           target: ${{ matrix.target }}
           manylinux: auto
-          args: --release --out dist
-      - uses: uraimo/run-on-arch-action@v2.5.0
+          args: --no-default-features --release --out dist
+      - uses: uraimo/run-on-arch-action@v2.0.5
         if: matrix.target != 'ppc64'
         name: Install built wheel
         with:
@@ -152,8 +146,8 @@ jobs:
             pip3 install -U pip
           run: |
             pip3 install ${{ env.PACKAGE_NAME }} --no-index --find-links dist/ --force-reinstall
-      - name: "Upload wheels"
-        uses: actions/upload-artifact@v3
+      - name: Upload wheels
+        uses: actions/upload-artifact@v2
         with:
           name: wheels
           path: dist
@@ -171,15 +165,13 @@ jobs:
         with:
           python-version: ${{ env.PYTHON_VERSION }}
           architecture: x64
-      - name: "Prep README.md"
-        run: python scripts/transform_readme.py --target pypi
-      - name: "Build wheels"
+      - name: Build wheels
         uses: messense/maturin-action@v1
         with:
           target: ${{ matrix.target }}
           manylinux: musllinux_1_2
           args: --release --out dist
-      - name: "Install built wheel"
+      - name: Install built wheel
         if: matrix.target == 'x86_64-unknown-linux-musl'
         uses: addnab/docker-run-action@v3
         with:
@@ -188,8 +180,8 @@ jobs:
           run: |
             apk add py3-pip
             pip3 install ${{ env.PACKAGE_NAME }} --no-index --find-links /io/dist/ --force-reinstall
-      - name: "Upload wheels"
-        uses: actions/upload-artifact@v3
+      - name: Upload wheels
+        uses: actions/upload-artifact@v2
         with:
           name: wheels
           path: dist
@@ -208,9 +200,7 @@ jobs:
       - uses: actions/setup-python@v4
         with:
           python-version: ${{ env.PYTHON_VERSION }}
-      - name: "Prep README.md"
-        run: python scripts/transform_readme.py --target pypi
-      - name: "Build wheels"
+      - name: Build wheels
         uses: messense/maturin-action@v1
         with:
           target: ${{ matrix.platform.target }}
@@ -226,8 +216,8 @@ jobs:
             apk add py3-pip
           run: |
             pip3 install ${{ env.PACKAGE_NAME }} --no-index --find-links dist/ --force-reinstall
-      - name: "Upload wheels"
-        uses: actions/upload-artifact@v3
+      - name: Upload wheels
+        uses: actions/upload-artifact@v2
         with:
           name: wheels
           path: dist
@@ -250,20 +240,18 @@ jobs:
       - uses: actions/setup-python@v4
         with:
           python-version: pypy${{ matrix.python-version }}
-      - name: "Prep README.md"
-        run: python scripts/transform_readme.py --target pypi
-      - name: "Build wheels"
+      - name: Build wheels
         uses: messense/maturin-action@v1
         with:
           target: ${{ matrix.target }}
           manylinux: auto
           args: --release --out dist -i pypy${{ matrix.python-version }}
-      - name: "Install built wheel"
+      - name: Install built wheel
         if: matrix.target == 'x86_64'
         run: |
           pip install dist/${{ env.PACKAGE_NAME }}-*.whl --force-reinstall
-      - name: "Upload wheels"
-        uses: actions/upload-artifact@v3
+      - name: Upload wheels
+        uses: actions/upload-artifact@v2
         with:
           name: wheels
           path: dist
@@ -282,17 +270,14 @@ jobs:
       - pypy
     if: "startsWith(github.ref, 'refs/tags/')"
     steps:
-      - uses: actions/download-artifact@v3
+      - uses: actions/download-artifact@v2
         with:
           name: wheels
       - uses: actions/setup-python@v4
-      - name: "Publish to PyPi"
+      - name: Publish to PyPi
         env:
           TWINE_USERNAME: __token__
           TWINE_PASSWORD: ${{ secrets.RUFF_TOKEN }}
         run: |
           pip install --upgrade twine
           twine upload --skip-existing *
-      - name: "Update pre-commit mirror"
-        run: |
-          curl -X POST -H "Accept: application/vnd.github+json" -H "Authorization: Bearer ${{ secrets.RUFF_PRE_COMMIT_PAT }}" -H "X-GitHub-Api-Version: 2022-11-28" https://api.github.com/repos/charliermarsh/ruff-pre-commit/dispatches --data '{"event_type": "pypi_release"}'

.gitignore

@@ -1,9 +1,6 @@
 # Local cache
 .ruff_cache
-crates/ruff/resources/test/cpython
-docs/
-mkdocs.yml
-.overrides
+resources/test/cpython
 
 ###
 # Rust.gitignore
@@ -185,6 +182,3 @@ cython_debug/
 #  option (not recommended) you can uncomment the following to ignore the entire idea folder.
 .idea/
 .vimspector.json
-
-# Visual Studio Code
-.vscode/

.pre-commit-config.yaml

@@ -1,44 +1,10 @@
 repos:
+  - repo: https://github.com/charliermarsh/ruff-pre-commit
+    rev: v0.0.217
+    hooks:
+      - id: ruff
+
   - repo: https://github.com/abravalheri/validate-pyproject
     rev: v0.10.1
     hooks:
       - id: validate-pyproject
-
-  - repo: https://github.com/igorshubovych/markdownlint-cli
-    rev: v0.33.0
-    hooks:
-      - id: markdownlint-fix
-        args:
-          - --disable
-          - MD013 # line-length
-          - MD033 # no-inline-html
-          - --
-
-  - repo: local
-    hooks:
-      - id: cargo-fmt
-        name: cargo fmt
-        entry: cargo fmt --
-        language: rust
-        types: [rust]
-      - id: clippy
-        name: clippy
-        entry: cargo clippy --workspace --all-targets --all-features -- -D warnings
-        language: rust
-        pass_filenames: false
-      - id: ruff
-        name: ruff
-        entry: cargo run -- --no-cache --fix
-        language: rust
-        types_or: [python, pyi]
-        require_serial: true
-        exclude: ^crates/ruff/resources
-      - id: dev-generate-all
-        name: dev-generate-all
-        entry: cargo dev generate-all
-        language: rust
-        pass_filenames: false
-        exclude: target
-
-ci:
-  skip: [cargo-fmt, clippy, dev-generate-all]

BREAKING_CHANGES.md

@@ -1,116 +1,5 @@
 # Breaking Changes
 
-## 0.0.238
-
-### `select`, `extend-select`, `ignore`, and `extend-ignore` have new semantics ([#2312](https://github.com/charliermarsh/ruff/pull/2312))
-
-Previously, the interplay between `select` and its related options could lead to unexpected
-behavior. For example, `ruff --select E501 --ignore ALL` and `ruff --select E501 --extend-ignore
-ALL` behaved differently. (See [#2312](https://github.com/charliermarsh/ruff/pull/2312) for more
-examples.)
-
-When Ruff determines the enabled rule set, it has to reconcile `select` and `ignore` from a variety
-of sources, including the current `pyproject.toml`, any inherited `pyproject.toml` files, and the
-CLI.
-
-The new semantics are such that Ruff uses the "highest-priority" `select` as the basis for the rule
-set, and then applies any `extend-select`, `ignore`, and `extend-ignore` adjustments. CLI options
-are given higher priority than `pyproject.toml` options, and the current `pyproject.toml` file is
-given higher priority than any inherited `pyproject.toml` files.
-
-`extend-select` and `extend-ignore` are no longer given "top priority"; instead, they merely append
-to the `select` and `ignore` lists, as in Flake8.
-
-This change is largely backwards compatible -- most users should experience no change in behavior.
-However, as an example of a breaking change, consider the following:
-
-```toml
-[tool.ruff]
-ignore = ["F401"]
-```
-
-Running `ruff --select F` would previously have enabled all `F` rules, apart from `F401`. Now, it
-will enable all `F` rules, including `F401`, as the command line's `--select` resets the resolution.
-
-### `remove-six-compat` (`UP016`) has been removed ([#2332](https://github.com/charliermarsh/ruff/pull/2332))
-
-The `remove-six-compat` rule has been removed. This rule was only useful for one-time Python 2-to-3
-upgrades.
-
-## 0.0.237
-
-### `--explain`, `--clean`, and `--generate-shell-completion` are now subcommands ([#2190](https://github.com/charliermarsh/ruff/pull/2190))
-
-`--explain`, `--clean`, and `--generate-shell-completion` are now implemented as subcommands:
-
-```console
-ruff .         # Still works! And will always work.
-ruff check .   # New! Also works.
-
-ruff --explain E402   # Still works.
-ruff rule E402        # New! Also works. (And preferred.)
-
-# Oops! The command has to come first.
-ruff --format json --explain E402   # No longer works.
-ruff --explain E402 --format json   # Still works!
-ruff rule E402   --format json      # Works! (And preferred.)
-```
-
-This change is largely backwards compatible -- most users should experience
-no change in behavior. However, please note the following exceptions:
-
-* Subcommands will now fail when invoked with unsupported arguments, instead
-  of silently ignoring them. For example, the following will now fail:
-
-  ```console
-  ruff --clean --respect-gitignore
-  ```
-
-  (the `clean` command doesn't support `--respect-gitignore`.)
-
-* The semantics of `ruff <arg>` have changed slightly when `<arg>` is a valid subcommand.
-  For example, prior to this release, running `ruff rule` would run `ruff` over a file or
-  directory called `rule`. Now, `ruff rule` would invoke the `rule` subcommand. This should
-  only impact projects with files or directories named `rule`, `check`, `explain`, `clean`,
-  or `generate-shell-completion`.
-
-* Scripts that invoke ruff should supply `--` before any positional arguments.
-  (The semantics of `ruff -- <arg>` have not changed.)
-
-* `--explain` previously treated `--format grouped` as a synonym for `--format text`.
-  This is no longer supported; instead, use `--format text`.
-
-## 0.0.226
-
-### `misplaced-comparison-constant` (`PLC2201`) was deprecated in favor of `SIM300` ([#1980](https://github.com/charliermarsh/ruff/pull/1980))
-
-These two rules contain (nearly) identical logic. To deduplicate the rule set, we've upgraded
-`SIM300` to handle a few more cases, and deprecated `PLC2201` in favor of `SIM300`.
-
-## 0.0.225
-
-### `@functools.cache` rewrites have been moved to a standalone rule (`UP033`) ([#1938](https://github.com/charliermarsh/ruff/pull/1938))
-
-Previously, `UP011` handled both `@functools.lru_cache()`-to-`@functools.lru_cache` conversions,
-_and_ `@functools.lru_cache(maxsize=None)`-to-`@functools.cache` conversions. The latter has been
-moved out to its own rule (`UP033`). As such, some `# noqa: UP011` comments may need to be updated
-to reflect the change in rule code.
-
-## 0.0.222
-
-### `--max-complexity` has been removed from the CLI ([#1877](https://github.com/charliermarsh/ruff/pull/1877))
-
-The McCabe plugin's `--max-complexity` setting has been removed from the CLI, for consistency with
-the treatment of other, similar settings.
-
-To set the maximum complexity, use the `max-complexity` property in your `pyproject.toml` file,
-like so:
-
-```toml
-[tool.ruff.mccabe]
-max-complexity = 10
-```
-
 ## 0.0.181
 
 ### Files excluded by `.gitignore` are now ignored ([#1234](https://github.com/charliermarsh/ruff/pull/1234))

CONTRIBUTING.md

@@ -1,288 +0,0 @@
-# Contributing to Ruff
-
-Welcome! We're happy to have you here. Thank you in advance for your contribution to Ruff.
-
-## The basics
-
-Ruff welcomes contributions in the form of Pull Requests.
-
-For small changes (e.g., bug fixes), feel free to submit a PR.
-
-For larger changes (e.g., new lint rules, new functionality, new configuration options), consider
-creating an [**issue**](https://github.com/charliermarsh/ruff/issues) outlining your proposed
-change. You can also join us on [**Discord**](https://discord.gg/Z8KbeK24) to discuss your idea with
-the community.
-
-If you're looking for a place to start, we recommend implementing a new lint rule (see:
-[_Adding a new lint rule_](#example-adding-a-new-lint-rule), which will allow you to learn from and
-pattern-match against the examples in the existing codebase. Many lint rules are inspired by
-existing Python plugins, which can be used as a reference implementation.
-
-As a concrete example: consider taking on one of the rules from the [`tryceratops`](https://github.com/charliermarsh/ruff/issues/2056)
-plugin, and looking to the originating [Python source](https://github.com/guilatrova/tryceratops)
-for guidance.
-
-### Prerequisites
-
-Ruff is written in Rust. You'll need to install the
-[Rust toolchain](https://www.rust-lang.org/tools/install) for development.
-
-You'll also need [Insta](https://insta.rs/docs/) to update snapshot tests:
-
-```shell
-cargo install cargo-insta
-```
-
-### Development
-
-After cloning the repository, run Ruff locally with:
-
-```shell
-cargo run /path/to/file.py --no-cache
-```
-
-Prior to opening a pull request, ensure that your code has been auto-formatted,
-and that it passes both the lint and test validation checks:
-
-```shell
-cargo fmt --all  # Auto-formatting...
-cargo clippy --fix --workspace --all-targets --all-features  # Linting...
-cargo test --all  # Testing...
-```
-
-These checks will run on GitHub Actions when you open your Pull Request, but running them locally
-will save you time and expedite the merge process.
-
-Note that many code changes also require updating the snapshot tests, which is done interactively
-after running `cargo test` like so:
-
-```shell
-cargo insta review
-```
-
-If you have `pre-commit` [installed](https://pre-commit.com/#installation) then you can use it to
-assist with formatting and linting. The following command will run the `pre-commit` hooks:
-
-```shell
-pre-commit run --all-files
-```
-
-Your Pull Request will be reviewed by a maintainer, which may involve a few rounds of iteration
-prior to merging.
-
-### Project structure
-
-Ruff is structured as a monorepo with a [flat crate structure](https://matklad.github.io/2021/08/22/large-rust-workspaces.html),
-such that all crates are contained in a flat `crates` directory.
-
-The vast majority of the code, including all lint rules, lives in the `ruff` crate (located at
-`crates/ruff`). As a contributor, that's the crate that'll be most relevant to you.
-
-At time of writing, the repository includes the following crates:
-
-- `crates/ruff`: library crate containing all lint rules and the core logic for running them.
-- `crates/ruff_cli`: binary crate containing Ruff's command-line interface.
-- `crates/ruff_dev`: binary crate containing utilities used in the development of Ruff itself (e.g., `cargo dev generate-all`).
-- `crates/ruff_macros`: library crate containing macros used by Ruff.
-- `crates/ruff_python`: library crate implementing Python-specific functionality (e.g., lists of standard library modules by versionb).
-- `crates/flake8_to_ruff`: binary crate for generating Ruff configuration from Flake8 configuration.
-
-### Example: Adding a new lint rule
-
-At a high level, the steps involved in adding a new lint rule are as follows:
-
-1. Create a file for your rule (e.g., `crates/ruff/src/rules/flake8_bugbear/rules/abstract_base_class.rs`).
-2. In that file, define a violation struct. You can grep for `define_violation!` to see examples.
-3. Map the violation struct to a rule code in `crates/ruff/src/registry.rs` (e.g., `E402`).
-4. Define the logic for triggering the violation in `crates/ruff/src/checkers/ast.rs` (for AST-based
-   checks), `crates/ruff/src/checkers/tokens.rs` (for token-based checks), `crates/ruff/src/checkers/lines.rs`
-   (for text-based checks), or `crates/ruff/src/checkers/filesystem.rs` (for filesystem-based
-   checks).
-5. Add a test fixture.
-6. Update the generated files (documentation and generated code).
-
-To define the violation, start by creating a dedicated file for your rule under the appropriate
-rule linter (e.g., `crates/ruff/src/rules/flake8_bugbear/rules/abstract_base_class.rs`). That file should
-contain a struct defined via `define_violation!`, along with a function that creates the violation
-based on any required inputs. (Many of the existing examples live in `crates/ruff/src/violations.rs`,
-but we're looking to place new rules in their own files.)
-
-To trigger the violation, you'll likely want to augment the logic in `crates/ruff/src/checkers/ast.rs`,
-which defines the Python AST visitor, responsible for iterating over the abstract syntax tree and
-collecting diagnostics as it goes.
-
-If you need to inspect the AST, you can run `cargo dev print-ast` with a Python file. Grep
-for the `Check::new` invocations to understand how other, similar rules are implemented.
-
-To add a test fixture, create a file under `crates/ruff/resources/test/fixtures/[linter]`, named to match
-the code you defined earlier (e.g., `crates/ruff/resources/test/fixtures/pycodestyle/E402.py`). This file should
-contain a variety of violations and non-violations designed to evaluate and demonstrate the behavior
-of your lint rule.
-
-Run `cargo dev generate-all` to generate the code for your new fixture. Then run Ruff
-locally with (e.g.) `cargo run crates/ruff/resources/test/fixtures/pycodestyle/E402.py --no-cache --select E402`.
-
-Once you're satisfied with the output, codify the behavior as a snapshot test by adding a new
-`test_case` macro in the relevant `crates/ruff/src/[linter]/mod.rs` file. Then, run `cargo test --all`.
-Your test will fail, but you'll be prompted to follow-up with `cargo insta review`. Accept the
-generated snapshot, then commit the snapshot file alongside the rest of your changes.
-
-Finally, regenerate the documentation and generated code with `cargo dev generate-all`.
-
-### Example: Adding a new configuration option
-
-Ruff's user-facing settings live in a few different places.
-
-First, the command-line options are defined via the `Cli` struct in `crates/ruff/src/cli.rs`.
-
-Second, the `pyproject.toml` options are defined in `crates/ruff/src/settings/options.rs` (via the
-`Options` struct), `crates/ruff/src/settings/configuration.rs` (via the `Configuration` struct), and
-`crates/ruff/src/settings/mod.rs` (via the `Settings` struct). These represent, respectively: the
-schema used to parse the `pyproject.toml` file; an internal, intermediate representation; and the
-final, internal representation used to power Ruff.
-
-To add a new configuration option, you'll likely want to modify these latter few files (along with
-`cli.rs`, if appropriate). If you want to pattern-match against an existing example, grep for
-`dummy_variable_rgx`, which defines a regular expression to match against acceptable unused
-variables (e.g., `_`).
-
-Note that plugin-specific configuration options are defined in their own modules (e.g.,
-`crates/ruff/src/flake8_unused_arguments/settings.rs`).
-
-You may also want to add the new configuration option to the `flake8-to-ruff` tool, which is
-responsible for converting `flake8` configuration files to Ruff's TOML format. This logic
-lives in `crates/ruff/src/flake8_to_ruff/converter.rs`.
-
-Finally, regenerate the documentation and generated code with `cargo dev generate-all`.
-
-## Release process
-
-As of now, Ruff has an ad hoc release process: releases are cut with high frequency via GitHub
-Actions, which automatically generates the appropriate wheels across architectures and publishes
-them to [PyPI](https://pypi.org/project/ruff/).
-
-Ruff follows the [semver](https://semver.org/) versioning standard. However, as pre-1.0 software,
-even patch releases may contain [non-backwards-compatible changes](https://semver.org/#spec-item-4).
-
-## Benchmarks
-
-First, clone [CPython](https://github.com/python/cpython). It's a large and diverse Python codebase,
-which makes it a good target for benchmarking.
-
-```shell
-git clone --branch 3.10 https://github.com/python/cpython.git crates/ruff/resources/test/cpython
-```
-
-To benchmark the release build:
-
-```shell
-cargo build --release && hyperfine --ignore-failure --warmup 10 \
-  "./target/release/ruff ./crates/ruff/resources/test/cpython/ --no-cache" \
-  "./target/release/ruff ./crates/ruff/resources/test/cpython/"
-
-Benchmark 1: ./target/release/ruff ./crates/ruff/resources/test/cpython/ --no-cache
-  Time (mean ± σ):     293.8 ms ±   3.2 ms    [User: 2384.6 ms, System: 90.3 ms]
-  Range (min … max):   289.9 ms … 301.6 ms    10 runs
-
-  Warning: Ignoring non-zero exit code.
-
-Benchmark 2: ./target/release/ruff ./crates/ruff/resources/test/cpython/
-  Time (mean ± σ):      48.0 ms ±   3.1 ms    [User: 65.2 ms, System: 124.7 ms]
-  Range (min … max):    45.0 ms …  66.7 ms    62 runs
-
-  Warning: Ignoring non-zero exit code.
-
-Summary
-  './target/release/ruff ./crates/ruff/resources/test/cpython/' ran
-    6.12 ± 0.41 times faster than './target/release/ruff ./crates/ruff/resources/test/cpython/ --no-cache'
-```
-
-To benchmark against the ecosystem's existing tools:
-
-```shell
-hyperfine --ignore-failure --warmup 5 \
-  "./target/release/ruff ./crates/ruff/resources/test/cpython/ --no-cache" \
-  "pyflakes crates/ruff/resources/test/cpython" \
-  "autoflake --recursive --expand-star-imports --remove-all-unused-imports --remove-unused-variables --remove-duplicate-keys resources/test/cpython" \
-  "pycodestyle crates/ruff/resources/test/cpython" \
-  "flake8 crates/ruff/resources/test/cpython"
-
-Benchmark 1: ./target/release/ruff ./crates/ruff/resources/test/cpython/ --no-cache
-  Time (mean ± σ):     294.3 ms ±   3.3 ms    [User: 2467.5 ms, System: 89.6 ms]
-  Range (min … max):   291.1 ms … 302.8 ms    10 runs
-
-  Warning: Ignoring non-zero exit code.
-
-Benchmark 2: pyflakes crates/ruff/resources/test/cpython
-  Time (mean ± σ):     15.786 s ±  0.143 s    [User: 15.560 s, System: 0.214 s]
-  Range (min … max):   15.640 s … 16.157 s    10 runs
-
-  Warning: Ignoring non-zero exit code.
-
-Benchmark 3: autoflake --recursive --expand-star-imports --remove-all-unused-imports --remove-unused-variables --remove-duplicate-keys resources/test/cpython
-  Time (mean ± σ):      6.175 s ±  0.169 s    [User: 54.102 s, System: 1.057 s]
-  Range (min … max):    5.950 s …  6.391 s    10 runs
-
-Benchmark 4: pycodestyle crates/ruff/resources/test/cpython
-  Time (mean ± σ):     46.921 s ±  0.508 s    [User: 46.699 s, System: 0.202 s]
-  Range (min … max):   46.171 s … 47.863 s    10 runs
-
-  Warning: Ignoring non-zero exit code.
-
-Benchmark 5: flake8 crates/ruff/resources/test/cpython
-  Time (mean ± σ):     12.260 s ±  0.321 s    [User: 102.934 s, System: 1.230 s]
-  Range (min … max):   11.848 s … 12.933 s    10 runs
-
-  Warning: Ignoring non-zero exit code.
-
-Summary
-  './target/release/ruff ./crates/ruff/resources/test/cpython/ --no-cache' ran
-   20.98 ± 0.62 times faster than 'autoflake --recursive --expand-star-imports --remove-all-unused-imports --remove-unused-variables --remove-duplicate-keys resources/test/cpython'
-   41.66 ± 1.18 times faster than 'flake8 crates/ruff/resources/test/cpython'
-   53.64 ± 0.77 times faster than 'pyflakes crates/ruff/resources/test/cpython'
-  159.43 ± 2.48 times faster than 'pycodestyle crates/ruff/resources/test/cpython'
-```
-
-You can run `poetry install` from `./scripts` to create a working environment for the above. All
-reported benchmarks were computed using the versions specified by `./scripts/pyproject.toml`
-on Python 3.11.
-
-To benchmark Pylint, remove the following files from the CPython repository:
-
-```shell
-rm Lib/test/bad_coding.py \
-  Lib/test/bad_coding2.py \
-  Lib/test/bad_getattr.py \
-  Lib/test/bad_getattr2.py \
-  Lib/test/bad_getattr3.py \
-  Lib/test/badcert.pem \
-  Lib/test/badkey.pem \
-  Lib/test/badsyntax_3131.py \
-  Lib/test/badsyntax_future10.py \
-  Lib/test/badsyntax_future3.py \
-  Lib/test/badsyntax_future4.py \
-  Lib/test/badsyntax_future5.py \
-  Lib/test/badsyntax_future6.py \
-  Lib/test/badsyntax_future7.py \
-  Lib/test/badsyntax_future8.py \
-  Lib/test/badsyntax_future9.py \
-  Lib/test/badsyntax_pep3120.py \
-  Lib/test/test_asyncio/test_runners.py \
-  Lib/test/test_copy.py \
-  Lib/test/test_inspect.py \
-  Lib/test/test_typing.py
-```
-
-Then, from `crates/ruff/resources/test/cpython`, run: `time pylint -j 0 -E $(git ls-files '*.py')`. This
-will execute Pylint with maximum parallelism and only report errors.
-
-To benchmark Pyupgrade, run the following from `crates/ruff/resources/test/cpython`:
-
-```shell
-hyperfine --ignore-failure --warmup 5 --prepare "git reset --hard HEAD" \
-  "find . -type f -name \"*.py\" | xargs -P 0 pyupgrade --py311-plus"
-
-Benchmark 1: find . -type f -name "*.py" | xargs -P 0 pyupgrade --py311-plus
-  Time (mean ± σ):     30.119 s ±  0.195 s    [User: 28.638 s, System: 0.390 s]
-  Range (min … max):   29.813 s … 30.356 s    10 runs
-```

Cargo.toml

@@ -1,6 +1,102 @@
 [workspace]
-members = ["crates/*"]
-default-members = ["crates/ruff", "crates/ruff_cli"]
+members = [
+  "flake8_to_ruff",
+  "ruff_dev",
+]
+
+[package]
+name = "ruff"
+version = "0.0.217"
+authors = ["Charlie Marsh <charlie.r.marsh@gmail.com>"]
+edition = "2021"
+rust-version = "1.65.0"
+documentation = "https://github.com/charliermarsh/ruff"
+homepage = "https://github.com/charliermarsh/ruff"
+repository = "https://github.com/charliermarsh/ruff"
+readme = "README.md"
+license = "MIT"
+
+[lib]
+name = "ruff"
+crate-type = ["cdylib", "rlib"]
+
+[dependencies]
+annotate-snippets = { version = "0.9.1", features = ["color"] }
+anyhow = { version = "1.0.66" }
+atty = { version = "0.2.14" }
+bincode = { version = "1.3.3" }
+bitflags = { version = "1.3.2" }
+cachedir = { version = "0.3.0" }
+cfg-if = { version = "1.0.0" }
+chrono = { version = "0.4.21", default-features = false, features = ["clock"] }
+clap = { version = "4.0.1", features = ["derive", "env"] }
+clap_complete_command = { version = "0.4.0" }
+colored = { version = "2.0.0" }
+dirs = { version = "4.0.0" }
+fern = { version = "0.6.1" }
+filetime = { version = "0.2.17" }
+glob = { version = "0.3.0" }
+globset = { version = "0.4.9" }
+ignore = { version = "0.4.18" }
+itertools = { version = "0.10.5" }
+libcst = { git = "https://github.com/charliermarsh/LibCST", rev = "f2f0b7a487a8725d161fe8b3ed73a6758b21e177" }
+log = { version = "0.4.17" }
+natord = { version = "1.0.9" }
+nohash-hasher = { version = "0.2.0" }
+notify = { version = "5.0.0" }
+num-bigint = { version = "0.4.3" }
+num-traits = "0.2.15"
+once_cell = { version = "1.16.0" }
+path-absolutize = { version = "3.0.14", features = ["once_cell_cache", "use_unix_paths_on_wasm"] }
+quick-junit = { version = "0.3.2" }
+regex = { version = "1.6.0" }
+ropey = { version = "1.5.0", features = ["cr_lines", "simd"], default-features = false }
+ruff_macros = { version = "0.0.217", path = "ruff_macros" }
+rustc-hash = { version = "1.1.0" }
+rustpython-ast = { features = ["unparse"], git = "https://github.com/RustPython/RustPython.git", rev = "d532160333ffeb6dbeca2c2728c2391cd1e53b7f" }
+rustpython-common = { git = "https://github.com/RustPython/RustPython.git", rev = "d532160333ffeb6dbeca2c2728c2391cd1e53b7f" }
+rustpython-parser = { features = ["lalrpop"], git = "https://github.com/RustPython/RustPython.git", rev = "d532160333ffeb6dbeca2c2728c2391cd1e53b7f" }
+schemars = { version = "0.8.11" }
+semver = { version = "1.0.16" }
+serde = { version = "1.0.147", features = ["derive"] }
+serde_json = { version = "1.0.87" }
+shellexpand = { version = "3.0.0" }
+similar = { version = "2.2.1" }
+strum = { version = "0.24.1", features = ["strum_macros"] }
+strum_macros = { version = "0.24.3" }
+textwrap = { version = "0.16.0" }
+titlecase = { version = "2.2.1" }
+toml_edit = { version = "0.17.1", features = ["easy"] }
+walkdir = { version = "2.3.2" }
+
+[target.'cfg(not(target_family = "wasm"))'.dependencies]
+clearscreen = { version = "2.0.0" }
+rayon = { version = "1.5.3" }
+update-informer = { version = "0.6.0", default-features = false, features = ["pypi"], optional = true }
+
+# https://docs.rs/getrandom/0.2.7/getrandom/#webassembly-support
+# For (future) wasm-pack support
+[target.'cfg(all(target_family = "wasm", target_os = "unknown"))'.dependencies]
+getrandom = { version = "0.2.7", features = ["js"] }
+console_error_panic_hook = { version = "0.1.7" }
+console_log = { version = "0.2.0" }
+serde-wasm-bindgen = { version = "0.4" }
+js-sys = { version = "0.3.60" }
+wasm-bindgen = { version = "0.2.83" }
+
+[dev-dependencies]
+insta = { version = "1.19.1", features = ["yaml"] }
+test-case = { version = "2.2.2" }
+ureq = { version = "2.5.0", features = [] }
+wasm-bindgen-test = { version = "0.3.33" }
+
+[target.'cfg(not(target_family = "wasm"))'.dev-dependencies]
+assert_cmd = { version = "2.0.4" }
+criterion = { version = "0.4.0" }
+
+[features]
+default = ["update-informer"]
+update-informer = ["dep:update-informer"]
 
 [profile.release]
 panic = "abort"
@@ -18,3 +114,7 @@ opt-level = 3
 # https://github.com/bytecodealliance/wasm-tools/blob/b5c3d98e40590512a3b12470ef358d5c7b983b15/crates/wasmparser/src/limits.rs#L29
 [profile.dev.package.rustpython-parser]
 opt-level = 1
+
+[[bench]]
+name = "source_code_locator"
+harness = false

_typos.toml

@@ -1,2 +0,0 @@
-[files]
-extend-exclude = ["snapshots"]

benches/source_code_locator.rs

@@ -0,0 +1,18 @@
+use std::fs;
+use std::path::Path;
+
+use criterion::{black_box, criterion_group, criterion_main, Criterion};
+use ropey::Rope;
+
+fn criterion_benchmark(c: &mut Criterion) {
+    let contents = fs::read_to_string(Path::new("resources/test/fixtures/D.py")).unwrap();
+    c.bench_function("rope", |b| {
+        b.iter(|| {
+            let rope = Rope::from_str(black_box(&contents));
+            rope.line_to_char(black_box(4));
+        });
+    });
+}
+
+criterion_group!(benches, criterion_benchmark);
+criterion_main!(benches);

crates/flake8_to_ruff/Cargo.toml

@@ -1,19 +0,0 @@
-[package]
-name = "flake8-to-ruff"
-version = "0.0.243"
-edition = "2021"
-
-[dependencies]
-anyhow = { version = "1.0.66" }
-clap = { version = "4.0.1", features = ["derive"] }
-colored = { version = "2.0.0" }
-configparser = { version = "3.0.2" }
-once_cell = { version = "1.16.0" }
-regex = { version = "1.6.0" }
-ruff = { path = "../ruff", default-features = false }
-rustc-hash = { version = "1.1.0" }
-serde = { version = "1.0.147", features = ["derive"] }
-serde_json = { version = "1.0.87" }
-strum = { version = "0.24.1", features = ["strum_macros"] }
-strum_macros = { version = "0.24.3" }
-toml = { version = "0.6.0" }

crates/flake8_to_ruff/src/main.rs

@@ -1,57 +0,0 @@
-//! Utility to generate Ruff's `pyproject.toml` section from a Flake8 INI file.
-
-use std::path::PathBuf;
-
-use anyhow::Result;
-use clap::Parser;
-use configparser::ini::Ini;
-use ruff::flake8_to_ruff::{self, ExternalConfig};
-
-#[derive(Parser)]
-#[command(
-    about = "Convert existing Flake8 configuration to Ruff.",
-    long_about = None
-)]
-struct Args {
-    /// Path to the Flake8 configuration file (e.g., `setup.cfg`, `tox.ini`, or
-    /// `.flake8`).
-    #[arg(required = true)]
-    file: PathBuf,
-    /// Optional path to a `pyproject.toml` file, used to ensure compatibility
-    /// with Black.
-    #[arg(long)]
-    pyproject: Option<PathBuf>,
-    /// List of plugins to enable.
-    #[arg(long, value_delimiter = ',')]
-    plugin: Option<Vec<flake8_to_ruff::Plugin>>,
-}
-
-fn main() -> Result<()> {
-    let args = Args::parse();
-
-    // Read the INI file.
-    let mut ini = Ini::new_cs();
-    ini.set_multiline(true);
-    let config = ini.load(args.file).map_err(|msg| anyhow::anyhow!(msg))?;
-
-    // Read the pyproject.toml file.
-    let pyproject = args.pyproject.map(flake8_to_ruff::parse).transpose()?;
-    let external_config = pyproject
-        .as_ref()
-        .and_then(|pyproject| pyproject.tool.as_ref())
-        .map(|tool| ExternalConfig {
-            black: tool.black.as_ref(),
-            isort: tool.isort.as_ref(),
-        })
-        .unwrap_or_default();
-
-    // Create Ruff's pyproject.toml section.
-    let pyproject = flake8_to_ruff::convert(&config, &external_config, args.plugin)?;
-
-    #[allow(clippy::print_stdout)]
-    {
-        println!("{}", toml::to_string_pretty(&pyproject)?);
-    }
-
-    Ok(())
-}

crates/ruff/Cargo.toml

@@ -1,83 +0,0 @@
-[package]
-name = "ruff"
-version = "0.0.243"
-authors = ["Charlie Marsh <charlie.r.marsh@gmail.com>"]
-edition = "2021"
-rust-version = "1.65.0"
-documentation = "https://github.com/charliermarsh/ruff"
-homepage = "https://github.com/charliermarsh/ruff"
-repository = "https://github.com/charliermarsh/ruff"
-readme = "README.md"
-license = "MIT"
-
-[lib]
-name = "ruff"
-crate-type = ["cdylib", "rlib"]
-doctest = false
-
-[dependencies]
-anyhow = { version = "1.0.66" }
-bisection = { version = "0.1.0" }
-bitflags = { version = "1.3.2" }
-cfg-if = { version = "1.0.0" }
-chrono = { version = "0.4.21", default-features = false, features = ["clock"] }
-clap = { version = "4.0.1", features = ["derive", "env"] }
-colored = { version = "2.0.0" }
-dirs = { version = "4.0.0" }
-fern = { version = "0.6.1" }
-glob = { version = "0.3.0" }
-globset = { version = "0.4.9" }
-ignore = { version = "0.4.18" }
-imperative = { version = "1.0.3" }
-itertools = { version = "0.10.5" }
-libcst = { git = "https://github.com/charliermarsh/LibCST", rev = "f2f0b7a487a8725d161fe8b3ed73a6758b21e177" }
-log = { version = "0.4.17" }
-natord = { version = "1.0.9" }
-nohash-hasher = { version = "0.2.0" }
-num-bigint = { version = "0.4.3" }
-num-traits = "0.2.15"
-once_cell = { version = "1.16.0" }
-path-absolutize = { version = "3.0.14", features = ["once_cell_cache", "use_unix_paths_on_wasm"] }
-regex = { version = "1.6.0" }
-ruff_macros = { version = "0.0.243", path = "../ruff_macros" }
-ruff_python = { version = "0.0.243", path = "../ruff_python" }
-rustc-hash = { version = "1.1.0" }
-rustpython-ast = { features = ["unparse"], git = "https://github.com/RustPython/RustPython.git", rev = "adc23253e4b58980b407ba2760dbe61681d752fc" }
-rustpython-common = { git = "https://github.com/RustPython/RustPython.git", rev = "adc23253e4b58980b407ba2760dbe61681d752fc" }
-rustpython-parser = { features = ["lalrpop"], git = "https://github.com/RustPython/RustPython.git", rev = "adc23253e4b58980b407ba2760dbe61681d752fc" }
-schemars = { version = "0.8.11" }
-semver = { version = "1.0.16" }
-serde = { version = "1.0.147", features = ["derive"] }
-shellexpand = { version = "3.0.0" }
-smallvec = { version = "1.10.0" }
-strum = { version = "0.24.1", features = ["strum_macros"] }
-strum_macros = { version = "0.24.3" }
-textwrap = { version = "0.16.0" }
-thiserror = { version = "1.0" }
-titlecase = { version = "2.2.1" }
-toml = { version = "0.6.0" }
-
-# https://docs.rs/getrandom/0.2.7/getrandom/#webassembly-support
-# For (future) wasm-pack support
-[target.'cfg(all(target_family = "wasm", target_os = "unknown"))'.dependencies]
-getrandom = { version = "0.2.7", features = ["js"] }
-console_error_panic_hook = { version = "0.1.7" }
-console_log = { version = "0.2.0" }
-serde-wasm-bindgen = { version = "0.4" }
-js-sys = { version = "0.3.60" }
-wasm-bindgen = { version = "0.2.83" }
-
-[target.'cfg(not(target_family = "wasm"))'.dependencies]
-is_executable = "1.0.1"
-
-[dev-dependencies]
-insta = { version = "1.19.0", features = ["yaml", "redactions"] }
-test-case = { version = "2.2.2" }
-wasm-bindgen-test = { version = "0.3.33" }
-
-[target.'cfg(not(target_family = "wasm"))'.dev-dependencies]
-criterion = { version = "0.4.0" }
-
-[features]
-default = []
-logical_lines = []

crates/ruff/resources/test/fixtures/flake8_annotations/allow_nested_overload.py

@@ -1,9 +0,0 @@
-class C:
-    from typing import overload
-
-    @overload
-    def f(self, x: int, y: int) -> None:
-        ...
-
-    def f(self, x, y):
-        pass

crates/ruff/resources/test/fixtures/flake8_annotations/mypy_init_return.py

@@ -1,48 +0,0 @@
-"""Test case expected to be run with `mypy_init_return = True`."""
-
-# Error
-class Foo:
-    def __init__(self):
-        ...
-
-
-# Error
-class Foo:
-    def __init__(self, foo):
-        ...
-
-
-# OK
-class Foo:
-    def __init__(self, foo) -> None:
-        ...
-
-
-# OK
-class Foo:
-    def __init__(self) -> None:
-        ...
-
-
-# OK
-class Foo:
-    def __init__(self, foo: int):
-        ...
-
-
-# OK
-class Foo:
-    def __init__(self, foo: int) -> None:
-        ...
-
-
-# Error
-def __init__(self, foo: int):
-    ...
-
-
-# Error – used to be ok for a moment since the mere presence
-# of a vararg falsely indicated that the function has a typed argument.
-class Foo:
-    def __init__(self, *arg):
-        ...

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

@@ -1,14 +0,0 @@
-try:
-    pass
-except Exception:
-    pass
-
-try:
-    pass
-except:
-    pass
-
-try:
-    pass
-except ValueError:
-    pass

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

@@ -1,6 +0,0 @@
-from pysnmp.hlapi import CommunityData
-
-CommunityData("public", mpModel=0)  # S508
-CommunityData("public", mpModel=1)  # S508
-
-CommunityData("public", mpModel=2)  # OK

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

@@ -1,7 +0,0 @@
-from pysnmp.hlapi import UsmUserData
-
-
-insecure = UsmUserData("securityName")  # S509
-auth_no_priv = UsmUserData("securityName", "authName")  # S509
-
-less_insecure = UsmUserData("securityName", "authName", "privName")  # OK

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

@@ -1,8 +0,0 @@
-import logging.config
-
-t = logging.config.listen(9999)
-
-def verify_func():
-    pass
-
-l = logging.config.listen(9999, verify=verify_func)

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

@@ -1,29 +0,0 @@
-import jinja2
-from jinja2 import Environment, select_autoescape
-templateLoader = jinja2.FileSystemLoader( searchpath="/" )
-something = ''
-
-Environment(loader=templateLoader, load=templateLoader, autoescape=True)
-templateEnv = jinja2.Environment(autoescape=True,
-        loader=templateLoader )
-Environment(loader=templateLoader, load=templateLoader, autoescape=something)  # S701
-templateEnv = jinja2.Environment(autoescape=False, loader=templateLoader )  # S701
-Environment(loader=templateLoader,
-            load=templateLoader,
-            autoescape=False)  # S701
-
-Environment(loader=templateLoader,  # S701
-            load=templateLoader)
-
-Environment(loader=templateLoader, autoescape=select_autoescape())
-
-Environment(loader=templateLoader,
-            autoescape=select_autoescape(['html', 'htm', 'xml']))
-
-Environment(loader=templateLoader,
-            autoescape=jinja2.select_autoescape(['html', 'htm', 'xml']))
-
-
-def fake_func():
-    return 'foobar'
-Environment(loader=templateLoader, autoescape=fake_func())  # S701

crates/ruff/resources/test/fixtures/flake8_blind_except/BLE.py

@@ -1,94 +0,0 @@
-try:
-    pass
-except ValueError:
-    pass
-except Exception as e:
-    raise e
-finally:
-    pass
-
-
-try:
-    pass
-except BaseException as e:
-    raise e
-except TypeError:
-    pass
-else:
-    pass
-
-
-try:
-    pass
-except Exception as e:
-    raise e
-except BaseException:
-    pass
-
-
-try:
-    pass
-except Exception:
-    pass
-finally:
-    try:
-        pass
-    except BaseException as e:
-        raise e
-
-
-try:
-    pass
-except Exception as e:
-    try:
-        raise e
-    except BaseException:
-        pass
-
-
-try:
-    try:
-        pass
-    except BaseException as e:
-        raise e
-except Exception:
-    pass
-
-
-try:
-    pass
-except Exception as e:
-    raise bad
-except BaseException:
-    pass
-
-import logging
-
-try:
-    pass
-except Exception:
-    logging.error("...")
-
-
-try:
-    pass
-except Exception:
-    logging.error("...", exc_info=False)
-
-
-try:
-    pass
-except Exception:
-    logging.error("...", exc_info=None)
-
-
-try:
-    pass
-except Exception:
-    logging.exception("...")
-
-
-try:
-    pass
-except Exception:
-    logging.error("...", exc_info=True)

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

@@ -1,88 +0,0 @@
-for i in range(10):
-    print(i)
-
-print(i)  # name no longer defined on Python 3; no warning yet
-
-for i in range(10):  # name not used within the loop; B007
-    print(10)
-
-print(i)  # name no longer defined on Python 3; no warning yet
-
-
-for _ in range(10):  # _ is okay for a throw-away variable
-    print(10)
-
-
-for i in range(10):
-    for j in range(10):
-        for k in range(10):  # k not used, i and j used transitively
-            print(i + j)
-
-
-def strange_generator():
-    for i in range(10):
-        for j in range(10):
-            for k in range(10):
-                for l in range(10):
-                    yield i, (j, (k, l))
-
-
-for i, (j, (k, l)) in strange_generator():  # i, k not used
-    print(j, l)
-
-FMT = "{foo} {bar}"
-for foo, bar in [(1, 2)]:
-    if foo:
-        print(FMT.format(**locals()))
-
-for foo, bar in [(1, 2)]:
-    if foo:
-        print(FMT.format(**globals()))
-
-for foo, bar in [(1, 2)]:
-    if foo:
-        print(FMT.format(**vars()))
-
-for foo, bar in [(1, 2)]:
-    print(FMT.format(foo=foo, bar=eval("bar")))
-
-
-def f():
-    # Fixable.
-    for foo, bar, baz in (["1", "2", "3"],):
-        if foo or baz:
-            break
-
-
-def f():
-    # Unfixable due to usage of `bar` outside of loop.
-    for foo, bar, baz in (["1", "2", "3"],):
-        if foo or baz:
-            break
-
-    print(bar)
-
-
-def f():
-    # Fixable.
-    for foo, bar, baz in (["1", "2", "3"],):
-        if foo or baz:
-            break
-
-    bar = 1
-
-
-def f():
-    # Fixable.
-    for foo, bar, baz in (["1", "2", "3"],):
-        if foo or baz:
-            break
-
-    bar = 1
-    print(bar)
-
-
-# Unfixable due to trailing underscore (`_line_` wouldn't be considered an ignorable
-# variable name).
-for line_ in range(self.header_lines):
-     fp.readline()

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

@@ -1,174 +0,0 @@
-"""
-Should emit:
-B023 - on lines 12, 13, 16, 28, 29, 30, 31, 40, 42, 50, 51, 52, 53, 61, 68.
-"""
-
-functions = []
-z = 0
-
-for x in range(3):
-    y = x + 1
-    # Subject to late-binding problems
-    functions.append(lambda: x)
-    functions.append(lambda: y)  # not just the loop var
-
-    def f_bad_1():
-        return x
-
-    # Actually OK
-    functions.append(lambda x: x * 2)
-    functions.append(lambda x=x: x)
-    functions.append(lambda: z)  # OK because not assigned in the loop
-
-    def f_ok_1(x):
-        return x * 2
-
-
-def check_inside_functions_too():
-    ls = [lambda: x for x in range(2)]  # error
-    st = {lambda: x for x in range(2)}  # error
-    gn = (lambda: x for x in range(2))  # error
-    dt = {x: lambda: x for x in range(2)}  # error
-
-
-async def pointless_async_iterable():
-    yield 1
-
-
-async def container_for_problems():
-    async for x in pointless_async_iterable():
-        functions.append(lambda: x)  # error
-
-    [lambda: x async for x in pointless_async_iterable()]  # error
-
-
-a = 10
-b = 0
-while True:
-    a = a_ = a - 1
-    b += 1
-    functions.append(lambda: a)  # error
-    functions.append(lambda: a_)  # error
-    functions.append(lambda: b)  # error
-    functions.append(lambda: c)  # error, but not a name error due to late binding
-    c: bool = a > 3
-    if not c:
-        break
-
-# Nested loops should not duplicate reports
-for j in range(2):
-    for k in range(3):
-        lambda: j * k  # error
-
-
-for j, k, l in [(1, 2, 3)]:
-
-    def f():
-        j = None  # OK because it's an assignment
-        [l for k in range(2)]  # error for l, not for k
-
-        assert a and functions
-
-    a.attribute = 1  # modifying an attribute doesn't make it a loop variable
-    functions[0] = lambda: None  # same for an element
-
-for var in range(2):
-
-    def explicit_capture(captured=var):
-        return captured
-
-
-for i in range(3):
-    lambda: f"{i}"
-
-
-# `query` is defined in the function, so also defining it in the loop should be OK.
-for name in ["a", "b"]:
-    query = name
-
-    def myfunc(x):
-        query = x
-        query_post = x
-        _ = query
-        _ = query_post
-
-    query_post = name  # in case iteration order matters
-
-
-# Bug here because two dict comprehensions reference `name`, one of which is inside
-# the lambda.  This should be totally fine, of course.
-_ = {
-    k: v
-    for k, v in reduce(
-        lambda data, event: merge_mappings(
-            [data, {name: f(caches, data, event) for name, f in xx}]
-        ),
-        events,
-        {name: getattr(group, name) for name in yy},
-    ).items()
-    if k in backfill_fields
-}
-
-
-# OK to define lambdas if they're immediately consumed, typically as the `key=`
-# argument or in a consumed `filter()` (even if a comprehension is better style)
-for x in range(2):
-    # It's not a complete get-out-of-linting-free construct - these should fail:
-    min([None, lambda: x], key=repr)
-    sorted([None, lambda: x], key=repr)
-    any(filter(bool, [None, lambda: x]))
-    list(filter(bool, [None, lambda: x]))
-    all(reduce(bool, [None, lambda: x]))
-
-    # But all these should be OK:
-    min(range(3), key=lambda y: x * y)
-    max(range(3), key=lambda y: x * y)
-    sorted(range(3), key=lambda y: x * y)
-
-    any(map(lambda y: x < y, range(3)))
-    all(map(lambda y: x < y, range(3)))
-    set(map(lambda y: x < y, range(3)))
-    list(map(lambda y: x < y, range(3)))
-    tuple(map(lambda y: x < y, range(3)))
-    sorted(map(lambda y: x < y, range(3)))
-    frozenset(map(lambda y: x < y, range(3)))
-
-    any(filter(lambda y: x < y, range(3)))
-    all(filter(lambda y: x < y, range(3)))
-    set(filter(lambda y: x < y, range(3)))
-    list(filter(lambda y: x < y, range(3)))
-    tuple(filter(lambda y: x < y, range(3)))
-    sorted(filter(lambda y: x < y, range(3)))
-    frozenset(filter(lambda y: x < y, range(3)))
-
-    any(reduce(lambda y: x | y, range(3)))
-    all(reduce(lambda y: x | y, range(3)))
-    set(reduce(lambda y: x | y, range(3)))
-    list(reduce(lambda y: x | y, range(3)))
-    tuple(reduce(lambda y: x | y, range(3)))
-    sorted(reduce(lambda y: x | y, range(3)))
-    frozenset(reduce(lambda y: x | y, range(3)))
-
-    import functools
-
-    any(functools.reduce(lambda y: x | y, range(3)))
-    all(functools.reduce(lambda y: x | y, range(3)))
-    set(functools.reduce(lambda y: x | y, range(3)))
-    list(functools.reduce(lambda y: x | y, range(3)))
-    tuple(functools.reduce(lambda y: x | y, range(3)))
-    sorted(functools.reduce(lambda y: x | y, range(3)))
-    frozenset(functools.reduce(lambda y: x | y, range(3)))
-
-# OK because the lambda which references a loop variable is defined in a `return`
-# statement, and after we return the loop variable can't be redefined.
-# In principle we could do something fancy with `break`, but it's not worth it.
-def iter_f(names):
-    for name in names:
-        if exists(name):
-            return lambda: name if exists(name) else None
-
-        if foo(name):
-            return [lambda: name]  # known false alarm
-
-        if False:
-            return [lambda: i for i in range(3)]  # error

crates/ruff/resources/test/fixtures/flake8_builtins/A003.py

@@ -1,12 +0,0 @@
-class MyClass:
-    ImportError = 4
-    id = 5
-    dir = "/"
-
-    def __init__(self):
-        self.float = 5  # is fine
-        self.id = 10
-        self.dir = "."
-
-    def str(self):
-        pass

crates/ruff/resources/test/fixtures/flake8_commas/COM81.py

@@ -1,628 +0,0 @@
-# ==> bad_function_call.py <==
-bad_function_call(
-    param1='test',
-    param2='test'
-)
-# ==> bad_list.py <==
-bad_list = [
-    1,
-    2,
-    3
-]
-
-bad_list_with_comment = [
-    1,
-    2,
-    3
-    # still needs a comma!
-]
-
-bad_list_with_extra_empty = [
-    1,
-    2,
-    3
-
-
-
-]
-
-# ==> bare.py <==
-bar = 1, 2
-
-foo = 1
-
-foo = (1,)
-
-foo = 1,
-
-bar = 1; foo = bar,
-
-foo = (
-3,
-4,
-)
-
-foo = 3, 
-
-class A(object):
- foo = 3
- bar = 10,
- foo_bar = 2
-
-a = ('a',)
-
-from foo import bar, baz
-
-group_by = function_call('arg'),
-
-group_by = ('foobar' * 3),
-
-def foo():
-    return False,
-
-# ==> callable_before_parenth_form.py <==
-def foo(
-    bar,
-):
-    pass
-
-{'foo': foo}['foo'](
-    bar
-)
-
-{'foo': foo}['foo'](
-    bar,
-)
-
-(foo)(
-    bar
-)
-
-(foo)[0](
-    bar,
-)
-
-[foo][0](
-    bar
-)
-
-[foo][0](
-    bar,
-)
-
-# ==> comment_good_dict.py <==
-multiline_good_dict = {
-    "good": 123,  # this is a good number
-}
-
-# ==> dict_comprehension.py <==
-not_a_dict = {
-    x: y
-    for x, y in ((1, 2), (3, 4))
-}
-
-# ==> good_empty_comma_context.py <==
-def func2(
-):
-    pass
-
-
-func2(
-
-)
-
-func2(
-)
-
-[
-]
-
-[
-
-]
-
-(
-)
-
-(
-
-)
-
-{
-
-}
-
-# ==> good_list.py <==
-stuff = [
-   'a',
-   'b',
-   # more stuff will go here
-]
-
-more_stuff = [
-    'a',
-    'b',
-
-
-
-]
-
-# ==> keyword_before_parenth_form/base_bad.py <==
-from x import (
-    y
-)
-
-assert(
-    SyntaxWarning,
-    ThrownHere,
-    Anyway
-)
-
-# async await is fine outside an async def
-# ruff: RustPython tokenizer treats async/await as keywords, not applicable.
-
-# def await(
-#     foo
-# ):
-#     async(
-#         foo
-#     )
-
-# def async(
-#     foo
-# ):
-#     await(
-#         foo
-#     )
-
-# ==> keyword_before_parenth_form/base.py <==
-from x import (
-    y,
-)
-
-assert(
-    SyntaxWarning,
-    ThrownHere,
-    Anyway,
-)
-
-assert (
-    foo
-)
-
-assert (
-    foo and
-    bar
-)
-
-if(
-    foo and
-    bar
-):
-    pass
-elif(
-    foo and
-    bar
-):
-    pass
-
-for x in(
-    [1,2,3]
-):
-    print(x)
-
-
-(x for x in (
-    [1, 2, 3]
-))
-
-(
-    'foo'
-) is (
-    'foo'
-)
-
-if (
-    foo and
-    bar
-) or not (
-    foo
-) or (
-    spam
-):
-    pass
-
-def xyz():
-    raise(
-        Exception()
-    )
-
-def abc():
-    return(
-        3
-    )
-
-while(
-    False
-):
-    pass
-
-with(
-    loop
-):
-    pass
-
-def foo():
-    yield (
-        "foo"
-    )
-
-# async await is fine outside an async def
-# ruff: RustPython tokenizer treats async/await as keywords, not applicable.
-
-# def await(
-#     foo,
-# ):
-#     async(
-#         foo,
-#     )
-
-# def async(
-#     foo,
-# ):
-#     await(
-#         foo,
-#     )
-
-# ==> keyword_before_parenth_form/py3.py <==
-# Syntax error in Py2
-
-def foo():
-    yield from (
-        foo
-    )
-
-# ==> list_comprehension.py <==
-not_a_list = [
-    s.strip()
-    for s in 'foo, bar, baz'.split(',')
-]
-
-# ==> multiline_bad_dict.py <==
-multiline_bad_dict = {
-    "bad": 123
-}
-# ==> multiline_bad_function_def.py <==
-def func_good(
-        a = 3,
-        b = 2):
-    pass
-
-
-def func_bad(
-        a = 3,
-        b = 2
-    ):
-    pass
-
-# ==> multiline_bad_function_one_param.py <==
-def func(
-        a = 3
-    ):
-    pass
-
-
-func(
-    a = 3
-)
-
-# ==> multiline_bad_or_dict.py <==
-multiline_bad_or_dict = {
-    "good": True or False,
-    "bad": 123
-}
-
-# ==> multiline_good_dict.py <==
-multiline_good_dict = {
-    "good": 123,
-}
-# ==> multiline_good_single_keyed_for_dict.py <==
-good_dict = {
-    "good": x for x in y
-}
-
-# ==> multiline_if.py <==
-if (
-    foo
-    and bar
-):
-    print("Baz")
-
-# ==> multiline_index_access.py <==
-multiline_index_access[
-    "good"
-]
-
-multiline_index_access_after_function()[
-    "good"
-]
-
-multiline_index_access_after_inline_index_access['first'][
-    "good"
-]
-
-multiline_index_access[
-    "probably fine",
-]
-
-[0, 1, 2][
-    "good"
-]
-
-[0, 1, 2][
-    "probably fine",
-]
-
-multiline_index_access[
-    "probably fine",
-    "not good"
-]
-
-multiline_index_access[
-    "fine",
-    "fine",
-    :
-    "not good"
-]
-
-# ==> multiline_string.py <==
-s = (
-    'this' +
-    'is a string'
-)
-
-s2 = (
-    'this'
-    'is also a string'
-)
-
-t = (
-    'this' +
-    'is a tuple',
-)
-
-t2 = (
-    'this'
-    'is also a tuple',
-)
-
-# ==> multiline_subscript_slice.py <==
-multiline_index_access[
-    "fine",
-    "fine"
-    :
-    "not fine"
-]
-
-multiline_index_access[
-    "fine"
-    "fine"
-    :
-    "fine"
-    :
-    "fine"
-]
-
-multiline_index_access[
-    "fine"
-    "fine",
-    :
-    "fine",
-    :
-    "fine",
-]
-
-multiline_index_access[
-    "fine"
-    "fine",
-    :
-    "fine"
-    :
-    "fine",
-    "not fine"
-]
-
-multiline_index_access[
-    "fine"
-    "fine",
-    :
-    "fine",
-    "fine"
-    :
-    "fine",
-]
-
-multiline_index_access[
-    lambda fine,
-    fine,
-    fine: (0,)
-    :
-    lambda fine,
-    fine,
-    fine: (0,),
-    "fine"
-    :
-    "fine",
-]
-
-# ==> one_line_dict.py <==
-one_line_dict = {"good": 123}
-# ==> parenth_form.py <==
-parenth_form = (
-    a +
-    b +
-    c
-)
-
-parenth_form_with_lambda = (
-    lambda x, y: 0
-)
-
-parenth_form_with_default_lambda = (
-    lambda x=(
-        lambda
-        x,
-        y,
-        :
-        0
-    ),
-    y = {a: b},
-    :
-    0
-)
-
-# ==> prohibited.py <==
-foo = ['a', 'b', 'c',]
-
-bar = { a: b,}
-
-def bah(ham, spam,):
-    pass
-
-(0,)
-
-(0, 1,)
-
-foo = ['a', 'b', 'c', ]
-
-bar = { a: b, }
-
-def bah(ham, spam, ):
-    pass
-
-(0, )
-
-(0, 1, )
-
-image[:, :, 0]
-
-image[:,]
-
-image[:,:,]
-
-lambda x, :
-
-# ==> unpack.py <==
-def function(
-    foo,
-    bar,
-    **kwargs
-):
-    pass
-
-def function(
-    foo,
-    bar,
-    *args
-):
-    pass
-
-def function(
-    foo,
-    bar,
-    *args,
-    extra_kwarg
-    ):
-    pass
-
-result = function(
-    foo,
-    bar,
-    **kwargs
-)
-
-result = function(
-    foo,
-    bar,
-    **not_called_kwargs
-)
-
-def foo(
-    ham,
-    spam,
-    *args,
-    kwarg_only
-    ):
-    pass
-
-# In python 3.5 if it's not a function def, commas are mandatory.
-
-foo(
-    **kwargs
-)
-
-{
-    **kwargs
-}
-
-(
-    *args
-)
-
-{
-    *args
-}
-
-[
-    *args
-]
-
-def foo(
-    ham,
-    spam,
-    *args
-    ):
-    pass
-
-def foo(
-    ham,
-    spam,
-    **kwargs
-    ):
-    pass
-
-def foo(
-    ham,
-    spam,
-    *args,
-    kwarg_only
-    ):
-    pass
-
-# In python 3.5 if it's not a function def, commas are mandatory.
-
-foo(
-    **kwargs,
-)
-
-{
-    **kwargs,
-}
-
-(
-    *args,
-)
-
-{
-    *args,
-}
-
-[
-    *args,
-]
-
-result = function(
-    foo,
-    bar,
-    **{'ham': spam}
-)

crates/ruff/resources/test/fixtures/flake8_comprehensions/C400.py

@@ -1,11 +0,0 @@
-x = list(x for x in range(3))
-x = list(
-    x for x in range(3)
-)
-
-
-def list(*args, **kwargs):
-    return None
-
-
-list(x for x in range(3))

crates/ruff/resources/test/fixtures/flake8_comprehensions/C401.py

@@ -1,11 +0,0 @@
-x = set(x for x in range(3))
-x = set(
-    x for x in range(3)
-)
-
-
-def set(*args, **kwargs):
-    return None
-
-
-set(x for x in range(3))

crates/ruff/resources/test/fixtures/flake8_comprehensions/C405.py

@@ -1,18 +0,0 @@
-set([1, 2])
-set((1, 2))
-set([])
-set(())
-set()
-set((1,))
-set((
-    1,
-))
-set([
-    1,
-])
-set(
-    (1,)
-)
-set(
-    [1,]
-)

crates/ruff/resources/test/fixtures/flake8_comprehensions/C408.py

@@ -1,12 +0,0 @@
-t = tuple()
-l = list()
-d1 = dict()
-d2 = dict(a=1)
-d3 = dict(**d2)
-
-
-def list():
-    return [1, 2, 3]
-
-
-a = list()

crates/ruff/resources/test/fixtures/flake8_comprehensions/C413.py

@@ -1,13 +0,0 @@
-x = [2, 3, 1]
-list(x)
-list(sorted(x))
-reversed(sorted(x))
-reversed(sorted(x, key=lambda e: e))
-reversed(sorted(x, reverse=True))
-
-
-def reversed(*args, **kwargs):
-    return None
-
-
-reversed(sorted(x, reverse=True))

crates/ruff/resources/test/fixtures/flake8_executable/EXE001_1.py

@@ -1,4 +0,0 @@
-#!/usr/bin/python
-
-if __name__ == '__main__':
-    print('I should be executable.')

crates/ruff/resources/test/fixtures/flake8_executable/EXE001_2.py

@@ -1,2 +0,0 @@
-if __name__ == '__main__':
-    print('I should be executable.')

crates/ruff/resources/test/fixtures/flake8_executable/EXE001_3.py

@@ -1,4 +0,0 @@
-#!/usr/bin/python
-
-if __name__ == '__main__':
-    print('I should be executable.')

crates/ruff/resources/test/fixtures/flake8_executable/EXE002_1.py

@@ -1,2 +0,0 @@
-if __name__ == '__main__':
-    print('I should be executable.')

crates/ruff/resources/test/fixtures/flake8_executable/EXE002_2.py

@@ -1,2 +0,0 @@
-if __name__ == '__main__':
-    print('I should be executable.')

crates/ruff/resources/test/fixtures/flake8_executable/EXE002_3.py

@@ -1,4 +0,0 @@
-#!/usr/bin/python
-
-if __name__ == '__main__':
-    print('I should be executable.')

crates/ruff/resources/test/fixtures/flake8_executable/EXE003.py

@@ -1,2 +0,0 @@
-#!/usr/bin/bash
-print("hello world")

crates/ruff/resources/test/fixtures/flake8_executable/EXE004_1.py

@@ -1 +0,0 @@
-    #!/usr/bin/python

crates/ruff/resources/test/fixtures/flake8_executable/EXE004_2.py

@@ -1,2 +0,0 @@
-print("  #!/usr/bin/python")
-# shebang outside of comments should be ignored

crates/ruff/resources/test/fixtures/flake8_executable/EXE004_3.py

@@ -1,2 +0,0 @@
-
-pass  #!/usr/bin/env python

crates/ruff/resources/test/fixtures/flake8_executable/EXE005_1.py

@@ -1,3 +0,0 @@
-
-# A python comment
-#!/usr/bin/python

crates/ruff/resources/test/fixtures/flake8_executable/EXE005_2.py

@@ -1,4 +0,0 @@
-print("code prior to shebang")
-
-# A python comment
-#!/usr/bin/python

crates/ruff/resources/test/fixtures/flake8_executable/EXE005_3.py

@@ -1,6 +0,0 @@
-"""
-With a docstring
-print("commented out code")
-"""
-# A python comment
-#!/usr/bin/python

crates/ruff/resources/test/fixtures/flake8_import_conventions/custom.py

@@ -1,48 +0,0 @@
-import math  # not checked
-
-import altair  # unconventional
-import dask.array  # unconventional
-import dask.dataframe  # unconventional
-import matplotlib.pyplot  # unconventional
-import numpy  # unconventional
-import pandas  # unconventional
-import seaborn  # unconventional
-import tensorflow  # unconventional
-import holoviews  # unconventional
-import panel  # unconventional
-import plotly.express  # unconventional
-import matplotlib  # unconventional
-import polars  # unconventional
-import pyarrow  # unconventional
-
-import altair as altr  # unconventional
-import matplotlib.pyplot as plot  # unconventional
-import dask.array as darray  # unconventional
-import dask.dataframe as ddf  # unconventional
-import numpy as nmp  # unconventional
-import pandas as pdas  # unconventional
-import seaborn as sbrn  # unconventional
-import tensorflow as tfz  # unconventional
-import holoviews as hsv  # unconventional
-import panel as pns  # unconventional
-import plotly.express as pltx  # unconventional
-import matplotlib as ml  # unconventional
-import polars as ps # unconventional
-import pyarrow as arr  # unconventional
-
-import altair as alt  # conventional
-import dask.array as da  # conventional
-import dask.dataframe as dd  # conventional
-import matplotlib.pyplot as plt  # conventional
-import numpy as np  # conventional
-import pandas as pd  # conventional
-import seaborn as sns  # conventional
-import tensorflow as tf # conventional
-import holoviews as hv  # conventional
-import panel as pn  # conventional
-import plotly.express as px  # conventional
-import matplotlib as mpl  # conventional
-import polars as pl # conventional
-import pyarrow as pa  # conventional
-
-from tensorflow.keras import Model  # conventional

crates/ruff/resources/test/fixtures/flake8_import_conventions/from_imports.py

@@ -1,21 +0,0 @@
-# Test absolute imports
-# Violation cases
-import xml.dom.minidom
-import xml.dom.minidom as wrong
-from xml.dom import minidom as wrong
-from xml.dom import minidom
-from xml.dom.minidom import parseString as wrong  # Ensure ICN001 throws on function import.
-from xml.dom.minidom import parseString
-from xml.dom.minidom import parse, parseString
-from xml.dom.minidom import parse as ps, parseString as wrong
-
-# No ICN001 violations
-import xml.dom.minidom as md
-from xml.dom import minidom as md
-from xml.dom.minidom import parseString as pstr
-from xml.dom.minidom import parse, parseString as pstr
-from xml.dom.minidom import parse as ps, parseString as pstr
-
-
-# Test relative imports
-from ..xml.dom import minidom as okay  # Ensure config "xml.dom.minidom" doesn't catch relative imports

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

@@ -1,3 +0,0 @@
-import logging
-
-logging.info("Hello {}".format("World!"))

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

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

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

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

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

@@ -1,4 +0,0 @@
-import logging
-
-name = "world"
-logging.info(f"Hello {name}")

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

@@ -1,3 +0,0 @@
-import logging
-
-logging.warn("Hello World!")

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

@@ -1,8 +0,0 @@
-import logging
-
-logging.info(
-    "Hello world!",
-    extra={
-        "name": "foobar",
-    },
-)

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

@@ -1,8 +0,0 @@
-import logging
-
-logging.info(
-    "Hello world!",
-    extra=dict(
-        name="foobar",
-    ),
-)

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

@@ -1,21 +0,0 @@
-import logging
-import sys
-
-# G201
-try:
-    pass
-except:
-    logging.error("Hello World", exc_info=True)
-
-try:
-    pass
-except:
-    logging.error("Hello World", exc_info=sys.exc_info())
-
-# OK
-try:
-    pass
-except:
-    logging.error("Hello World", exc_info=False)
-
-logging.error("Hello World", exc_info=sys.exc_info())

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

@@ -1,21 +0,0 @@
-import logging
-import sys
-
-# G202
-try:
-    pass
-except:
-    logging.exception("Hello World", exc_info=True)
-
-try:
-    pass
-except:
-    logging.exception("Hello World", exc_info=sys.exc_info())
-
-# OK
-try:
-    pass
-except:
-    logging.exception("Hello World", exc_info=False)
-
-logging.exception("Hello World", exc_info=True)

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

@@ -1,7 +0,0 @@
-import argparse
-from pathlib import Path
-
-parser = argparse.ArgumentParser()
-parser.add_argument("target_dir", type=Path)
-args = parser.parse_args()
-parser.error(f"Target directory {args.target_dir} does not exist")

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

@@ -1,8 +0,0 @@
-import logging
-
-logging.info(
-    "Hello {world}!",
-    extra=dict(
-        world="World",
-    ),
-)

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

@@ -1,8 +0,0 @@
-import logging
-
-logging.info(
-    "Hello {world}!",
-    extra=dict(
-        world="{}".format("World"),
-    ),
-)

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

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

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

@@ -1,3 +0,0 @@
-import warnings
-
-warnings.warn("Hello World!")

crates/ruff/resources/test/fixtures/flake8_no_pep420/test_fail_nonempty/example.py

@@ -1 +0,0 @@
-print('hi')

crates/ruff/resources/test/fixtures/flake8_no_pep420/test_fail_shebang/example.py

@@ -1,2 +0,0 @@
-#!/bin/env/python
-print('hi')

crates/ruff/resources/test/fixtures/flake8_no_pep420/test_ignored/example.py

@@ -1 +0,0 @@
-import os  # noqa: INP001

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

@@ -1,66 +0,0 @@
-import enum
-from enum import Enum, unique
-
-
-class FakeEnum1(enum.Enum):
-    A = "A"
-    B = "B"
-    C = "B"  # PIE796
-
-
-class FakeEnum2(Enum):
-    A = 1
-    B = 2
-    C = 2  # PIE796
-
-
-class FakeEnum3(str, Enum):
-    A = "1"
-    B = "2"
-    C = "2"  # PIE796
-
-
-class FakeEnum4(Enum):
-    A = 1.0
-    B = 2.5
-    C = 2.5  # PIE796
-
-
-class FakeEnum5(Enum):
-    A = 1.0
-    B = True
-    C = False
-    D = False  # PIE796
-
-
-class FakeEnum6(Enum):
-    A = 1
-    B = 2
-    C = None
-    D = None  # PIE796
-
-
-@enum.unique
-class FakeEnum7(enum.Enum):
-    A = "A"
-    B = "B"
-    C = "C"
-
-
-@unique
-class FakeEnum8(Enum):
-    A = 1
-    B = 2
-    C = 2  # PIE796
-
-
-class FakeEnum9(enum.Enum):
-    A = "A"
-    B = "B"
-    C = "C"
-
-
-class FakeEnum10(enum.Enum):
-    A = enum.auto()
-    B = enum.auto()
-    C = enum.auto()

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

@@ -1,17 +0,0 @@
-{"foo": 1, **{"bar": 1}}  # PIE800
-
-foo({**foo, **{"bar": True}})  # PIE800
-
-{**foo, **{"bar": 10}}  # PIE800
-
-{**foo, **buzz, **{bar: 10}}  # PIE800
-
-{**foo,    "bar": True  }  # OK
-
-{"foo": 1, "buzz": {"bar": 1}}  # OK
-
-{**foo,    "bar": True  }  # OK
-
-Table.objects.filter(inst=inst, **{f"foo__{bar}__exists": True})  # OK
-
-buzz = {**foo, "bar": { 1: 2 }}  # OK

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

@@ -1,19 +0,0 @@
-foo(**{"bar": True})  # PIE804
-
-foo(**{"r2d2": True})  # PIE804
-
-Foo.objects.create(**{"bar": True})  # PIE804
-
-Foo.objects.create(**{"_id": some_id})  # PIE804
-
-Foo.objects.create(**{**bar})  # PIE804
-
-
-foo(**{**data, "foo": "buzz"})
-foo(**buzz)
-foo(**{"bar-foo": True})
-foo(**{"bar foo": True})
-foo(**{"1foo": True})
-foo(**{buzz: True})
-foo(**{"": True})
-foo(**{f"buzz__{bar}": True})

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

@@ -1,17 +0,0 @@
-# error
-obj.startswith("foo") or obj.startswith("bar")
-# error
-obj.endswith("foo") or obj.endswith("bar")
-# error
-obj.startswith(foo) or obj.startswith(bar)
-# error
-obj.startswith(foo) or obj.startswith("foo")
-
-# ok
-obj.startswith(("foo",  "bar"))
-# ok
-obj.endswith(("foo",  "bar"))
-# ok
-obj.startswith("foo") or obj.endswith("bar")
-# ok
-obj.startswith("foo") or abc.startswith("bar")

crates/ruff/resources/test/fixtures/flake8_pytest_style/PT003.py

@@ -1,73 +0,0 @@
-import pytest
-
-
-@pytest.fixture()
-def ok_no_scope():
-    ...
-
-
-@pytest.fixture(scope="module")
-def ok_other_scope():
-    ...
-
-
-@pytest.fixture(scope="function")
-def error():
-    ...
-
-
-@pytest.fixture(scope="function", name="my_fixture")
-def error_multiple_args():
-    ...
-
-
-@pytest.fixture(name="my_fixture", scope="function")
-def error_multiple_args():
-    ...
-
-
-@pytest.fixture(name="my_fixture", scope="function", **kwargs)
-def error_second_arg():
-    ...
-
-
-# pytest.fixture does not take positional arguments, however this 
-# tests the general case as we use a helper function that should
-# work for all cases.
-@pytest.fixture("my_fixture", scope="function")
-def error_arg():
-    ...
-
-
-@pytest.fixture(
-    scope="function",
-    name="my_fixture",
-)
-def error_multiple_args():
-    ...
-
-
-@pytest.fixture(
-    name="my_fixture",
-    scope="function",
-)
-def error_multiple_args():
-    ...
-
-
-@pytest.fixture(
-    "hello", 
-    name, 
-    *args
-    , 
-
-    # another comment ,)
-    
-    scope=\
-        "function"  # some comment ),
-    ,
-    
-    name2=name, name3="my_fixture", **kwargs
-)
-def error_multiple_args():
-    ...

crates/ruff/resources/test/fixtures/flake8_pytest_style/PT007.py

@@ -1,77 +0,0 @@
-import pytest
-
-
-@pytest.mark.parametrize("param", (1, 2))
-def test_tuple(param):
-    ...
-
-
-@pytest.mark.parametrize(
-    ("param1", "param2"),
-    (
-        (1, 2),
-        (3, 4),
-    ),
-)
-def test_tuple_of_tuples(param1, param2):
-    ...
-
-
-@pytest.mark.parametrize(
-    ("param1", "param2"),
-    (
-        [1, 2],
-        [3, 4],
-    ),
-)
-def test_tuple_of_lists(param1, param2):
-    ...
-
-
-@pytest.mark.parametrize("param", [1, 2])
-def test_list(param):
-    ...
-
-
-@pytest.mark.parametrize(
-    ("param1", "param2"),
-    [
-        (1, 2),
-        (3, 4),
-    ],
-)
-def test_list_of_tuples(param1, param2):
-    ...
-
-
-@pytest.mark.parametrize(
-    ("param1", "param2"),
-    [
-        [1, 2],
-        [3, 4],
-    ],
-)
-def test_list_of_lists(param1, param2):
-    ...
-
-
-@pytest.mark.parametrize(
-    "param1,param2",
-    [
-        [1, 2],
-        [3, 4],
-    ],
-)
-def test_csv_name_list_of_lists(param1, param2):
-    ...
-
-
-@pytest.mark.parametrize(
-    "param",
-    [
-        [1, 2],
-        [3, 4],
-    ],
-)
-def test_single_list_of_lists(param):
-    ...

crates/ruff/resources/test/fixtures/flake8_quotes/docstring_singles_function.py

@@ -1,31 +0,0 @@
-def foo():
-    '''function without params, single line docstring'''
-    ''' not a docstring'''
-    return
-
-
-def foo2():
-    '''
-        function without params, multiline docstring
-    '''
-    ''' not a docstring'''
-    return
-
-
-def fun_with_params_no_docstring(a, b='''
-    not a
-''' '''docstring'''):
-    pass
-
-
-def fun_with_params_no_docstring2(a, b=c[foo():], c=\
-    ''' not a docstring '''):
-    pass
-
-
-def function_with_single_docstring(a):
-    'Single line docstring'
-
-
-def double_inside_single(a):
-    "Double inside 'single '"

crates/ruff/resources/test/fixtures/flake8_quotes/doubles.py

@@ -1,4 +0,0 @@
-this_should_be_linted = "double quote string"
-this_should_be_linted = u"double quote string"
-this_should_be_linted = f"double quote string"
-this_should_be_linted = f"double {'quote'} string"

crates/ruff/resources/test/fixtures/flake8_quotes/doubles_escaped.py

@@ -1,11 +0,0 @@
-this_should_raise_Q003 = 'This is a \'string\''
-this_should_raise_Q003 = 'This is \\ a \\\'string\''
-this_is_fine = '"This" is a \'string\''
-this_is_fine = "This is a 'string'"
-this_is_fine = "\"This\" is a 'string'"
-this_is_fine = r'This is a \'string\''
-this_is_fine = R'This is a \'string\''
-this_should_raise = (
-    'This is a'
-    '\'string\''
-)

crates/ruff/resources/test/fixtures/flake8_quotes/doubles_implicit.py

@@ -1,27 +0,0 @@
-x = (
-    "This"
-    "is"
-    "not"
-)
-
-x = (
-    "This" \
-    "is" \
-    "not"
-)
-
-x = (
-    "This"
-    "is 'actually'"
-    "fine"
-)
-
-x = (
-    "This" \
-    "is 'actually'" \
-    "fine"
-)
-
-if True:
-    "This can use 'double' quotes"
-"But this needs to be changed"

crates/ruff/resources/test/fixtures/flake8_quotes/singles.py

@@ -1,4 +0,0 @@
-this_should_be_linted = 'single quote string'
-this_should_be_linted = u'double quote string'
-this_should_be_linted = f'double quote string'
-this_should_be_linted = f'double {"quote"} string'

crates/ruff/resources/test/fixtures/flake8_quotes/singles_escaped.py

@@ -1,10 +0,0 @@
-this_should_raise_Q003 = "This is a \"string\""
-this_is_fine = "'This' is a \"string\""
-this_is_fine = 'This is a "string"'
-this_is_fine = '\'This\' is a "string"'
-this_is_fine = r"This is a \"string\""
-this_is_fine = R"This is a \"string\""
-this_should_raise = (
-    "This is a"
-    "\"string\""
-)

crates/ruff/resources/test/fixtures/flake8_quotes/singles_implicit.py

@@ -1,27 +0,0 @@
-x = (
-    'This'
-    'is'
-    'not'
-)
-
-x = (
-    'This' \
-    'is' \
-    'not'
-)
-
-x = (
-    'This'
-    'is "actually"'
-    'fine'
-)
-
-x = (
-    'This' \
-    'is "actually"' \
-    'fine'
-)
-
-if True:
-    'This can use "single" quotes'
-'But this needs to be changed'

crates/ruff/resources/test/fixtures/flake8_raise/RSE102.py

@@ -1,34 +0,0 @@
-try:
-    y = 6 + "7"
-except TypeError:
-    # RSE102
-    raise ValueError()
-
-try:
-    x = 1 / 0
-except ZeroDivisionError:
-    raise
-
-# RSE102
-raise TypeError()
-
-# RSE102
-raise TypeError ()
-
-# RSE102
-raise TypeError \
-    ()
-
-# RSE102
-raise TypeError(
-
-)
-
-# RSE102
-raise TypeError(
-    # Hello, world!
-)
-
-raise AssertionError
-
-raise AttributeError("test message")

crates/ruff/resources/test/fixtures/flake8_self/SLF001.py

@@ -1,59 +0,0 @@
-class BazMeta(type):
-    _private_count = 1
-
-    def __new__(mcs, name, bases, attrs):
-        if mcs._private_count <= 5:
-            mcs.some_method()
-
-        return super().__new__(mcs, name, bases, attrs)
-
-    def some_method():
-        pass
-
-
-class Bar:
-    _private = True
-
-    @classmethod
-    def is_private(cls):
-        return cls._private
-
-
-class Foo(metaclass=BazMeta):
-
-    def __init__(self):
-        self.public_thing = "foo"
-        self._private_thing = "bar"
-        self.__really_private_thing = "baz"
-        self.bar = Bar()
-
-    def __str__(self):
-        return "foo"
-
-    def get_bar():
-        if self.bar._private:  # SLF001
-            return None
-        return self.bar
-
-    def public_func(self):
-        pass
-
-    def _private_func(self):
-        pass
-
-    def __really_private_func(self, arg):
-        pass
-
-
-foo = Foo()
-
-print(foo.public_thing)
-print(foo.public_func())
-print(foo.__dict__)
-print(foo.__str__())
-
-print(foo._private_thing)  # SLF001
-print(foo.__really_private_thing)  # SLF001
-print(foo._private_func())  # SLF001
-print(foo.__really_private_func(1))  # SLF001
-print(foo.bar._private)  # SLF001

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

@@ -1,131 +0,0 @@
-# SIM102
-if a:
-    if b:
-        c
-
-# SIM102
-if a:
-    if b:
-        if c:
-            d
-
-# SIM102
-if a:
-    pass
-elif b:
-    if c:
-        d
-
-# SIM102
-if a:
-    # Unfixable due to placement of this comment.
-    if b:
-        c
-
-# SIM102
-if a:
-    if b:
-        # Fixable due to placement of this comment.
-        c
-
-# OK
-if a:
-    if b:
-        c
-    else:
-        d
-
-# OK
-if __name__ == "__main__":
-    if foo():
-        ...
-
-# OK
-if a:
-    d
-    if b:
-        c
-
-while True:
-    # SIM102
-    if True:
-        if True:
-            """this
-is valid"""
-
-            """the indentation on
-            this line is significant"""
-
-            "this is" \
-"allowed too"
-
-            ("so is"
-"this for some reason")
-
-
-# SIM102
-if True:
-    if True:
-        """this
-is valid"""
-
-        """the indentation on
-        this line is significant"""
-
-        "this is" \
-"allowed too"
-
-        ("so is"
-"this for some reason")
-
-while True:
-    # SIM102
-    if node.module:
-        if node.module == "multiprocessing" or node.module.startswith(
-            "multiprocessing."
-        ):
-            print("Bad module!")
-
-# SIM102
-if node.module:
-    if node.module == "multiprocessing" or node.module.startswith(
-        "multiprocessing."
-    ):
-        print("Bad module!")
-
-
-# OK
-if a:
-    if b:
-        print("foo")
-else:
-    print("bar")
-
-# OK
-if a:
-    if b:
-        if c:
-            print("foo")
-        else:
-            print("bar")
-else:
-    print("bar")
-
-# OK
-if a:
-    # SIM 102
-    if b:
-        if c:
-            print("foo")
-else:
-    print("bar")
-
-
-# OK
-if a:
-    if b:
-        if c:
-            print("foo")
-        print("baz")
-else:
-    print("bar")

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

@@ -1,86 +0,0 @@
-def f():
-    # SIM103
-    if a:
-        return True
-    else:
-        return False
-
-
-def f():
-    # SIM103
-    if a == b:
-        return True
-    else:
-        return False
-
-
-def f():
-    # SIM103
-    if a:
-        return 1
-    elif b:
-        return True
-    else:
-        return False
-
-
-def f():
-    # SIM103
-    if a:
-        return 1
-    else:
-        if b:
-            return True
-        else:
-            return False
-
-
-def f():
-    # OK
-    if a:
-        foo()
-        return True
-    else:
-        return False
-
-
-def f():
-    # OK
-    if a:
-        return "foo"
-    else:
-        return False
-
-
-def f():
-    # SIM103 (but not fixable)
-    if a:
-        return False
-    else:
-        return True
-
-
-def f():
-    # OK
-    if a:
-        return False
-    else:
-        return False
-
-
-def f():
-    # OK
-    if a:
-        return True
-    else:
-        return True
-
-
-def f():
-    # OK
-    def bool():
-        return False
-    if a:
-        return True
-    else:
-        return False

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

@@ -1,61 +0,0 @@
-def foo():
-    pass
-
-try:
-    foo()
-except ValueError:  # SIM105
-    pass
-
-try:
-    foo()
-except (ValueError, OSError):  # SIM105
-    pass
-
-try:
-    foo()
-except:  # SIM105
-    pass
-
-try:
-    foo()
-except (a.Error, b.Error):  # SIM105
-    pass
-
-try:
-    foo()
-except ValueError:
-    print('foo')
-except OSError:
-    pass
-
-try:
-    foo()
-except ValueError:
-    pass
-else:
-    print('bar')
-
-try:
-    foo()
-except ValueError:
-    pass
-finally:
-    print('bar')
-
-try:
-    foo()
-    foo()
-except ValueError:
-    pass
-
-try:
-    for i in range(3):
-        foo()
-except ValueError:
-    pass
-
-def bar():
-    try:
-        return foo()
-    except ValueError:
-        pass

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

@@ -1,112 +0,0 @@
-# SIM108
-if a:
-    b = c
-else:
-    b = d
-
-# OK
-b = c if a else d
-
-# OK
-if a:
-    b = c
-elif c:
-    b = a
-else:
-    b = d
-
-# OK
-if True:
-    pass
-elif a:
-    b = 1
-else:
-    b = 2
-
-# OK (false negative)
-if True:
-    pass
-else:
-    if a:
-        b = 1
-    else:
-        b = 2
-
-
-import sys
-
-# OK
-if sys.version_info >= (3, 9):
-    randbytes = random.randbytes
-else:
-    randbytes = _get_random_bytes
-
-# OK
-if sys.platform == "darwin":
-    randbytes = random.randbytes
-else:
-    randbytes = _get_random_bytes
-
-# OK
-if sys.platform.startswith("linux"):
-    randbytes = random.randbytes
-else:
-    randbytes = _get_random_bytes
-
-
-# SIM108 (without fix due to comments)
-if x > 0:
-    # test test
-    abc = x
-else:
-    # test test test
-    abc = -x
-
-
-# OK (too long)
-if parser.errno == BAD_FIRST_LINE:
-    req = wrappers.Request(sock, server=self._server)
-else:
-    req = wrappers.Request(
-        sock,
-        parser.get_method(),
-        parser.get_scheme() or _scheme,
-        parser.get_path(),
-        parser.get_version(),
-        parser.get_query_string(),
-        server=self._server,
-    )
-
-
-# SIM108
-if a:
-    b = cccccccccccccccccccccccccccccccccccc
-else:
-    b = ddddddddddddddddddddddddddddddddddddd
-
-
-# OK (too long)
-if True:
-    if a:
-        b = cccccccccccccccccccccccccccccccccccc
-    else:
-        b = ddddddddddddddddddddddddddddddddddddd
-
-
-# SIM108 (without fix due to trailing comment)
-if True:
-    exitcode = 0
-else:
-    exitcode = 1  # Trailing comment
-
-
-# SIM108
-if True: x = 3  # Foo
-else: x = 5
-
-
-# SIM108
-if True:  # Foo
-    x = 3
-else:
-    x = 5

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

@@ -1,31 +0,0 @@
-# SIM109
-if a == b or a == c:
-    d
-
-# SIM109
-if (a == b or a == c) and None:
-    d
-
-# SIM109
-if a == b or a == c or None:
-    d
-
-# SIM109
-if a == b or None or a == c:
-    d
-
-# OK
-if a in (b, c):
-    d
-
-# OK
-if a == b or a == c():
-    d
-
-# OK
-if (
-    a == b
-    # This comment prevents us from raising SIM109
-    or a == c
-):
-    d

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

@@ -1,157 +0,0 @@
-def f():
-    # SIM110
-    for x in iterable:
-        if check(x):
-            return True
-    return False
-
-
-def f():
-    for x in iterable:
-        if check(x):
-            return True
-    return True
-
-
-def f():
-    for el in [1, 2, 3]:
-        if is_true(el):
-            return True
-    raise Exception
-
-
-def f():
-    # SIM111
-    for x in iterable:
-        if check(x):
-            return False
-    return True
-
-
-def f():
-    # SIM111
-    for x in iterable:
-        if not x.is_empty():
-            return False
-    return True
-
-
-def f():
-    for x in iterable:
-        if check(x):
-            return False
-    return False
-
-
-def f():
-    for x in iterable:
-        if check(x):
-            return "foo"
-    return "bar"
-
-
-def f():
-    # SIM110
-    for x in iterable:
-        if check(x):
-            return True
-    else:
-        return False
-
-
-def f():
-    # SIM111
-    for x in iterable:
-        if check(x):
-            return False
-    else:
-        return True
-
-
-def f():
-    # SIM110
-    for x in iterable:
-        if check(x):
-            return True
-    else:
-        return False
-    return True
-
-
-def f():
-    # SIM111
-    for x in iterable:
-        if check(x):
-            return False
-    else:
-        return True
-    return False
-
-
-def f():
-    for x in iterable:
-        if check(x):
-            return True
-        elif x.is_empty():
-            return True
-    return False
-
-
-def f():
-    for x in iterable:
-        if check(x):
-            return True
-        else:
-            return True
-    return False
-
-
-def f():
-    for x in iterable:
-        if check(x):
-            return True
-        elif x.is_empty():
-            return True
-    else:
-        return True
-    return False
-
-
-def f():
-    def any(exp):
-        pass
-
-    for x in iterable:
-        if check(x):
-            return True
-    return False
-
-
-def f():
-    def all(exp):
-        pass
-
-    for x in iterable:
-        if check(x):
-            return False
-    return True
-
-
-def f():
-    x = 1
-
-    # SIM110
-    for x in iterable:
-        if check(x):
-            return True
-    return False
-
-
-def f():
-    x = 1
-
-    # SIM111
-    for x in iterable:
-        if check(x):
-            return False
-    return True

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

@@ -1,157 +0,0 @@
-def f():
-    # SIM110
-    for x in iterable:
-        if check(x):
-            return True
-    return False
-
-
-def f():
-    for x in iterable:
-        if check(x):
-            return True
-    return True
-
-
-def f():
-    for el in [1, 2, 3]:
-        if is_true(el):
-            return True
-    raise Exception
-
-
-def f():
-    # SIM111
-    for x in iterable:
-        if check(x):
-            return False
-    return True
-
-
-def f():
-    # SIM111
-    for x in iterable:
-        if not x.is_empty():
-            return False
-    return True
-
-
-def f():
-    for x in iterable:
-        if check(x):
-            return False
-    return False
-
-
-def f():
-    for x in iterable:
-        if check(x):
-            return "foo"
-    return "bar"
-
-
-def f():
-    # SIM110
-    for x in iterable:
-        if check(x):
-            return True
-    else:
-        return False
-
-
-def f():
-    # SIM111
-    for x in iterable:
-        if check(x):
-            return False
-    else:
-        return True
-
-
-def f():
-    # SIM110
-    for x in iterable:
-        if check(x):
-            return True
-    else:
-        return False
-    return True
-
-
-def f():
-    # SIM111
-    for x in iterable:
-        if check(x):
-            return False
-    else:
-        return True
-    return False
-
-
-def f():
-    for x in iterable:
-        if check(x):
-            return True
-        elif x.is_empty():
-            return True
-    return False
-
-
-def f():
-    for x in iterable:
-        if check(x):
-            return True
-        else:
-            return True
-    return False
-
-
-def f():
-    for x in iterable:
-        if check(x):
-            return True
-        elif x.is_empty():
-            return True
-    else:
-        return True
-    return False
-
-
-def f():
-    def any(exp):
-        pass
-
-    for x in iterable:
-        if check(x):
-            return True
-    return False
-
-
-def f():
-    def all(exp):
-        pass
-
-    for x in iterable:
-        if check(x):
-            return False
-    return True
-
-
-def f():
-    x = 1
-
-    # SIM110
-    for x in iterable:
-        if check(x):
-            return True
-    return False
-
-
-def f():
-    x = 1
-
-    # SIM111
-    for x in iterable:
-        if check(x):
-            return False
-    return True

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

@@ -1,36 +0,0 @@
-import contextlib
-
-# SIM115
-f = open("foo.txt")
-data = f.read()
-f.close()
-
-# OK
-with open("foo.txt") as f:
-    data = f.read()
-
-# OK
-with contextlib.ExitStack() as exit_stack:
-    f = exit_stack.enter_context(open("filename"))
-
-# OK
-with contextlib.ExitStack() as stack:
-    files = [stack.enter_context(open(fname)) for fname in filenames]
-    close_files = stack.pop_all().close
-
-# OK
-with contextlib.AsyncExitStack() as exit_stack:
-    f = await exit_stack.enter_async_context(open("filename"))
-
-# OK (false negative)
-with contextlib.ExitStack():
-    f = exit_stack.enter_context(open("filename"))
-
-# SIM115
-with contextlib.ExitStack():
-    f = open("filename")
-
-# OK
-with contextlib.ExitStack() as exit_stack:
-    exit_stack_ = exit_stack
-    f = exit_stack_.enter_context(open("filename"))

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

@@ -1,92 +0,0 @@
-# SIM117
-with A() as a:
-    with B() as b:
-        print("hello")
-
-# SIM117
-with A():
-    with B():
-        with C():
-            print("hello")
-
-# SIM117
-with A() as a:
-    # Unfixable due to placement of this comment.
-    with B() as b:
-        print("hello")
-
-# SIM117
-with A() as a:
-    with B() as b:
-        # Fixable due to placement of this comment.
-        print("hello")
-
-# OK
-with A() as a:
-    a()
-    with B() as b:
-        print("hello")
-
-# OK
-with A() as a:
-    with B() as b:
-        print("hello")
-    a()
-
-# OK
-async with A() as a:
-    with B() as b:
-        print("hello")
-
-# OK
-with A() as a:
-    async with B() as b:
-        print("hello")
-
-# OK
-async with A() as a:
-    async with B() as b:
-        print("hello")
-
-while True:
-    # SIM117
-    with A() as a:
-        with B() as b:
-            """this
-is valid"""
-
-            """the indentation on
-            this line is significant"""
-
-            "this is" \
-"allowed too"
-
-            ("so is"
-"this for some reason")
-
-# SIM117
-with (
-    A() as a,
-    B() as b,
-):
-    with C() as c:
-        print("hello")
-
-# SIM117
-with A() as a:
-    with (
-        B() as b,
-        C() as c,
-    ):
-        print("hello")
-
-# SIM117
-with (
-    A() as a,
-    B() as b,
-):
-    with (
-        C() as c,
-        D() as d,
-    ):
-        print("hello")