summaryrefslogtreecommitdiff
path: root/.github/workflows/clippy.yaml
diff options
context:
space:
mode:
authorseth <[email protected]>2024-08-16 22:14:16 -0400
committerGitHub <[email protected]>2024-08-17 02:14:16 +0000
commitbbc00edc6508ea1910c4d9e6a272f7859900950d (patch)
tree538fe23940cb6bca8afa48059e502bfa7d5608cd /.github/workflows/clippy.yaml
parent66ea6f8c225cc2273cfbfb1b948604c170a78e8c (diff)
end of summer refactor (#31)
* crates: `bot-*` -> `discord-bot` I didn't really need all these crates to be split :/ * discord-bot: revamp http impl also handles the new errors reported by teawieAPI * crates: split http backend this can be reused easily * git-tracker: short-circuit boolean logic We don't need to check if the commit is a descendant of the HEAD of the branch if it *is* the HEAD * nix: fenix -> nixpkgs * treefmt: add actionlint * nix: use docker arch names for containers * ci: use actions-rust-lang actions * nix: drop ci dev shell * git-tracker: init ManagedRepository this logic can be shared * ci: use nix for clippy scan * discord-bot: better handle unmerged PRs * ci: fix treefmt check * nix: fix clippy check * .env.template: update crate names * git-tracker: use remote name for remote name i was half asleep * discord-bot: handle merged PRs that aren't found in any tracked branches * git-tracker: make collect_statuses_in() return a Vec * discord-bot: add more PR info with response fixes #18
Diffstat (limited to '.github/workflows/clippy.yaml')
-rw-r--r--.github/workflows/clippy.yaml40
1 files changed, 15 insertions, 25 deletions
diff --git a/.github/workflows/clippy.yaml b/.github/workflows/clippy.yaml
index 2d3ea70..c94f6ce 100644
--- a/.github/workflows/clippy.yaml
+++ b/.github/workflows/clippy.yaml
@@ -3,15 +3,17 @@ name: Clippy
on:
push:
paths:
- - 'Cargo.toml'
- - 'Cargo.lock'
- '**.rs'
+ - '.github/workflows/clippy.yaml'
+ - 'Cargo.lock'
+ - 'Cargo.toml'
branches: [main]
pull_request:
paths:
- - 'Cargo.toml'
- - 'Cargo.lock'
- '**.rs'
+ - '.github/workflows/clippy.yaml'
+ - 'Cargo.lock'
+ - 'Cargo.toml'
workflow_dispatch:
jobs:
@@ -27,33 +29,21 @@ jobs:
- name: Checkout repository
uses: actions/checkout@v4
- - name: Install Rust
- uses: dtolnay/rust-toolchain@stable
- with:
- components: "clippy"
-
- - name: Setup Rust cache
- uses: Swatinem/rust-cache@v2
+ - name: Install Nix
+ uses: DeterminateSystems/nix-installer-action@v13
- - name: Install SARIF tools
- run: |
- cargo install clippy-sarif sarif-fmt
-
- - name: Fetch Cargo deps
- run: |
- cargo fetch --locked
+ - name: Setup Nix cache
+ uses: DeterminateSystems/magic-nix-cache-action@v7
- name: Run Clippy
- continue-on-error: true
+ id: clippy-run
run: |
- cargo clippy \
- --all-features \
- --all-targets \
- --message-format=json \
- | clippy-sarif | tee /tmp/clippy.sarif | sarif-fmt
+ nix build --print-build-logs .#checks.x86_64-linux.clippy-sarif
+ [ -L result ] || exit 1
+ echo "sarif-file=$(readlink -f result)" >> "$GITHUB_OUTPUT"
- name: Upload results
uses: github/codeql-action/upload-sarif@v3
with:
- sarif_file: /tmp/clippy.sarif
+ sarif_file: ${{ steps.clippy-run.outputs.sarif-file }}
wait-for-processing: true