Add option to fail check on warnings about clippy-check HOT 3 OPEN

Hi, @sffc! Is there any reason not to add -D warnings directly to the args input, as in args: --all-targets --all-features -- -D warnings?
That should (as usual) make clippy exit if there were any warnings, and as a consequence, fail the check too.

Speaking about the clippy annotations, there is a "nextgen" clippy Action exists, which should solve some of the problems you mentioned in the original issue (to be more specific, it will work for all PRs).
Unfortunately, the only way to make it work right now is to use unstable GitHub feature (I'm actually not sure in what state it is right now, as the tracking issue is in stale state since October and GH staff is not very helpful in providing any sort of ETA), but if you up to some "potentially broken" scripts in your CI, you can give it a shot.

from clippy-check.

@svartalf Earlier testing iterations of the workflow did indeed include ... -- -D warnings in the args to the clippy Action, but it did not fail the job / workflow as expected (test ex 1, test ex 2). The next test was successful after dropping down from the clippy Action to running Cargo at the CLI.

@sffc 's pasted snippet was taken from the current iteration of the workflow, after the clippy step had been split up into 2 jobs -- one uses the clippy Action to get the awesome Github UI integrations, the other job runs cargo clippy at the CLI in strict mode to get the job / workflow failures.

from clippy-check.

When it comes to the warnings being treated as errors, I just realized that this functionality fix was already made in v1.0.2. Sorry about that.

That leaves us with the annotations concern, and that is too bad that we can't tell how the progress is coming along. Since that is the only part remaining here, I think we might be able to close this issue as a duplicate of #2 .

from clippy-check.