It's now a required job before merge is allowed. Unfortunately GitHub now blocks any non-Rust PR, because they require cargo clippy but don't trigger it to run. Solution is simple, just always run cargo clippy, so it can pass, so that merge is allowed.
44 lines
1.1 KiB
YAML
44 lines
1.1 KiB
YAML
on:
|
|
push:
|
|
branches:
|
|
- main
|
|
paths:
|
|
- '**/Cargo.toml'
|
|
- '**/Cargo.lock'
|
|
- '**/rust-toolchain.toml'
|
|
- '**.rs'
|
|
- .github/workflows/cargo-clippy.yml
|
|
pull_request:
|
|
concurrency:
|
|
group: ${{ github.workflow }}-${{ github.head_ref || github.run_id }}
|
|
cancel-in-progress: true
|
|
name: cargo clippy
|
|
jobs:
|
|
cargoclippy:
|
|
name: cargo clippy
|
|
runs-on: ubuntu-latest
|
|
strategy:
|
|
matrix:
|
|
dir: ['src/wasm-lib']
|
|
steps:
|
|
- uses: actions/checkout@v4
|
|
- name: Install latest rust
|
|
uses: actions-rs/toolchain@v1
|
|
with:
|
|
toolchain: stable
|
|
override: true
|
|
components: clippy
|
|
|
|
- name: install dependencies
|
|
if: matrix.dir == 'src-tauri'
|
|
run: |
|
|
sudo apt-get update
|
|
sudo apt-get install -y libgtk-3-dev libwebkit2gtk-4.0-dev libappindicator3-dev librsvg2-dev patchelf
|
|
- name: Rust Cache
|
|
uses: Swatinem/rust-cache@v2.6.1
|
|
|
|
- name: Run clippy
|
|
run: |
|
|
cd "${{ matrix.dir }}"
|
|
cargo clippy --all --tests --benches -- -D warnings
|