Skip to content

Commit

Permalink
chore: Fix clippy checks in CI (#1032)
Browse files Browse the repository at this point in the history
* chore: Fix needless borrow

* ci: Clippy requires checking out submodules

* ci: Exclude `protobuf` project from clippy
  • Loading branch information
caspermeijn authored Apr 19, 2024
1 parent e3deaa2 commit eee2de0
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 2 deletions.
4 changes: 3 additions & 1 deletion .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -25,6 +25,8 @@ jobs:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v4
with:
submodules: 'recursive'
- name: install protoc
uses: taiki-e/install-action@v2
with:
Expand All @@ -34,7 +36,7 @@ jobs:
- uses: dtolnay/rust-toolchain@stable
with:
components: clippy
- run: cargo clippy --workspace --all-features --tests -- -D warnings
- run: cargo clippy --workspace --exclude protobuf --all-features --tests -- -D warnings

machete:
runs-on: ubuntu-latest
Expand Down
2 changes: 1 addition & 1 deletion protobuf/build.rs
Original file line number Diff line number Diff line change
Expand Up @@ -131,7 +131,7 @@ fn apply_patches(src_dir: &Path) -> Result<()> {
.arg("-p1")
.arg("-i")
.arg(patch_src)
.current_dir(&src_dir)
.current_dir(src_dir)
.status()
.context("failed to apply patch")?;
// exit code: 0 means success; 1 means already applied
Expand Down

0 comments on commit eee2de0

Please sign in to comment.