diff options
| author | seth <[email protected]> | 2024-08-16 22:14:16 -0400 |
|---|---|---|
| committer | GitHub <[email protected]> | 2024-08-17 02:14:16 +0000 |
| commit | bbc00edc6508ea1910c4d9e6a272f7859900950d (patch) | |
| tree | 538fe23940cb6bca8afa48059e502bfa7d5608cd /.github/workflows/clippy.yaml | |
| parent | 66ea6f8c225cc2273cfbfb1b948604c170a78e8c (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.yaml | 40 |
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 |
