Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

ci: unify rust.yml lints #192

Merged
merged 1 commit into from
Aug 4, 2023
Merged

ci: unify rust.yml lints #192

merged 1 commit into from
Aug 4, 2023

Conversation

dspicher
Copy link
Owner

@dspicher dspicher commented Aug 4, 2023

Since fuzz is in the workspace, just use workspace mode for the appropriate linting commands.

Also:

  • prefer the less verbose version without unhelpful names.

  • drop the cargo version stage

  • drop the unfunctional --locked flag for check

Since `fuzz` is in the workspace, just use workspace
mode for the appropriate linting commands.

Also:

 - prefer the less verbose version without unhelpful
   names.

 - drop the `cargo version` stage

 - drop the unfunctional `--locked` flag for `check`
@codecov
Copy link

codecov bot commented Aug 4, 2023

Codecov Report

Merging #192 (bef8f78) into master (a484565) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #192   +/-   ##
=======================================
  Coverage   96.25%   96.25%           
=======================================
  Files           5        5           
  Lines        1255     1255           
=======================================
  Hits         1208     1208           
  Misses         47       47           

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@dspicher dspicher merged commit e152a41 into master Aug 4, 2023
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant