Skip to content

Commit

Permalink
ci: add --frozen flag to cargo check in CI
Browse files Browse the repository at this point in the history
  • Loading branch information
@iTrooz
iTrooz committed Apr 28, 2024
1 parent 2a9e699 commit d63ed60
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion .github/workflows/build.yml
View file
Original file line number Diff line number Diff line change
Expand Up @@ -44,7 +44,7 @@ jobs:
run: cargo fmt -- --check

- name: Run cargo check
run: cargo check --all-features
run: cargo check --all-features --frozen

- name: Run cargo clippy
run: cargo clippy --all-features
Expand Down

2 comments on commit d63ed60

@MOZGIII
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please actually check if this works: you might have to specify the --frozen for the first command (or better all of them really) that touches Cargo.lock, otherwise the subsequent commands won't actually have any changes to apply (as the changes would be applied by the prev. command that didn't have --frozen passed to it). Not sure if cargo fetch or cargo fmt -- --check need --frozen - but you should definitely test this solution.

@iTrooz
Copy link
Owner Author

@iTrooz iTrooz commented on d63ed60 May 14, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Indeed ! I fixed that in 1c22eba

Please sign in to comment.