ensure that linting check fails on warning as well as removing cargo check since it is included in clippy. Updated name back to static analysis due to removal of check.

This commit is contained in:
Aayush 2024-07-15 14:43:13 -07:00
parent a5661d9f52
commit f5d6cfbcc0
2 changed files with 27 additions and 40 deletions

View file

@ -1,40 +0,0 @@
name: Checks on PR
on: pull_request
jobs:
# Check and Linting are seperate jobs because linting does not fail
#when there are errors outside of the changed files in the pr
check:
name: Check
runs-on: ubuntu-latest
steps:
- name: Setup | Checkout
uses: actions/checkout@v4
- name: Setup | Rust
run: rustup toolchain install stable --profile minimal
- name: Run Check
run: cd envoyfilter && cargo check --lib --bins --examples --all-features
lint:
name: Lint
runs-on: ubuntu-latest
steps:
- name: Setup | Checkout
uses: actions/checkout@v4
- name: Setup | Rust
run: rustup toolchain install stable --profile minimal
- name: Run Clippy
run: cd envoyfilter && cargo clippy --all-targets --all-features
format:
name: Rustfmt
runs-on: ubuntu-latest
steps:
- name: Setup | Checkout
uses: actions/checkout@v4
- name: Setup | Rust
run: rustup toolchain install stable --profile minimal
- name: Run Rustfmt
run: cd envoyfilter && cargo fmt --all -- --check

27
.github/workflows/static-analysis.yml vendored Normal file
View file

@ -0,0 +1,27 @@
name: Static Analysis
on: pull_request
jobs:
lint:
name: Lint
runs-on: ubuntu-latest
steps:
- name: Setup | Checkout
uses: actions/checkout@v4
- name: Setup | Rust
run: rustup toolchain install stable --profile minimal
- name: Run Clippy
run: cd envoyfilter && cargo clippy --all-targets --all-features -- -Dwarnings
format:
name: Rustfmt
runs-on: ubuntu-latest
steps:
- name: Setup | Checkout
uses: actions/checkout@v4
- name: Setup | Rust
run: rustup toolchain install stable --profile minimal
- name: Run Rustfmt
run: cd envoyfilter && cargo fmt --all -- --check