clippy-check icon indicating copy to clipboard operation
clippy-check copied to clipboard

Bring clippy-check up to date with modern APIs

Open kristof-mattei opened this issue 2 years ago • 4 comments

  • Use the integrated way of emitting annotations, removing the need for token, solves this Gitter question
  • Fix the issue of duplicated annotations, fixes https://github.com/actions-rs/clippy/issues/1 (also an issue here)
  • Posts summary with the summary API, closes #164
  • Update packages to latest version

kristof-mattei avatar May 12 '22 16:05 kristof-mattei

FWIW, this works great, but I'll note two things as a bystander:

  • Your changes didn't update README.md
  • Not sure whether this applies to the original version, because I only tried it briefly and I don't remember how it was, but warning annotations for dependencies from crates.io are showing up on the github actions summary.

glandium avatar Jun 21 '22 01:06 glandium

Would be great to merge this, but I'd also note https://github.com/actions-rs/clippy-check/issues/162

adam-azarchs avatar Oct 10 '22 06:10 adam-azarchs

might be time to fork this repo...

danieleades avatar Oct 12 '22 07:10 danieleades

Looks like @kristof-mattei already did that: https://github.com/actions-rs-plus/clippy-check

adam-azarchs avatar Oct 13 '22 05:10 adam-azarchs