Skip to content

Commit

Permalink
Add clippy
Browse files Browse the repository at this point in the history
  • Loading branch information
manzt committed Nov 7, 2024
1 parent 76ddfff commit aa250b5
Show file tree
Hide file tree
Showing 2 changed files with 19 additions and 0 deletions.
18 changes: 18 additions & 0 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,9 @@ jobs:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v4
- uses: Swatinem/rust-cache@v2
with:
save-if: ${{ github.ref == 'refs/heads/main' }}
- name: Build
run: cargo build --verbose
- name: Run tests
Expand All @@ -24,7 +27,22 @@ jobs:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v4
- uses: Swatinem/rust-cache@v2
with:
save-if: ${{ github.ref == 'refs/heads/main' }}
- name: "Install Rustfmt"
run: rustup component add rustfmt
- name: "rustfmt"
run: cargo fmt --all --check

Clippy:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v4
- uses: Swatinem/rust-cache@v2
with:
save-if: ${{ github.ref == 'refs/heads/main' }}
- name: "Install Clippy"
run: rustup component add clippy
- name: "Clippy"
run: cargo clippy --all-targets --all-features --locked -- -D warnings
1 change: 1 addition & 0 deletions src/script.rs
Original file line number Diff line number Diff line change
Expand Up @@ -87,6 +87,7 @@ impl Runtime {
}

/// Dynamically generates a script for uv to run the notebook/lab/nbclassic in an isolated environment
#[allow(clippy::format_in_format_args)]
pub fn prepare_run_script(
&self,
path: &Path,
Expand Down

0 comments on commit aa250b5

Please sign in to comment.