Skip to content

Don't deny warnings on CI - #201

Merged
phil-opp merged 3 commits into
masterfrom
update-ci
Nov 8, 2020
Merged

phil-opp merged 3 commits into
masterfrom
update-ci

Conversation

@phil-opp

Copy link
Copy Markdown
Member

Sometimes Rust/Cargo or clippy introduce new (stylistic) warnings. We don't want to break our CI build over those. This PR changes or CI script to report these warning in the Github UI instead.

Clippy sometimes introduces new stylistic warnings and we don't want to break our build over this.
Rust/Cargo sometimes add new stylistic warnings too.
@phil-opp phil-opp mentioned this pull request Oct 31, 2020
2 tasks
@phil-opp

Copy link
Copy Markdown
Member Author

The warnings are now shown inline in the "Files Changed" tab and also on the build overview page.

@phil-opp
phil-opp requested review from a user and josephlr October 31, 2020 11:34
@phil-opp
phil-opp force-pushed the update-ci branch 3 times, most recently from 73a3f5a to 3e4c6fe Compare October 31, 2020 12:08
@phil-opp
phil-opp merged commit 7def2db into master Nov 8, 2020
@phil-opp
phil-opp deleted the update-ci branch November 8, 2020 10:41
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.

1 participant