Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Add clippy #26

Merged
merged 2 commits into from
Aug 30, 2024
Merged

Add clippy #26

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
52a6be0
Add clippy
thejpster Aug 30, 2024
09c83b6
Make Clippy fail on warnings
thejpster Aug 30, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
21 changes: 19 additions & 2 deletions .github/workflows/rust.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,19 +2,36 @@ name: Rust

on: [push, pull_request]

# Make sure CI fails on all warnings, including Clippy lints
env:
RUSTFLAGS: "-Dwarnings"

jobs:
formatting:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v1
- uses: actions/checkout@v4
- name: Check formatting
run: cargo fmt -- --check

build-test:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v1
- uses: actions/checkout@v4
- name: Build
run: cargo build
- name: Run Tests
run: cargo test

clippy:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v4
- name: Clippy (default features)
run: cargo clippy
- name: Clippy (no features)
run: cargo clippy --no-default-features
- name: Clippy (all features)
run: cargo clippy --all-features