Skip to content
This repository has been archived by the owner on Oct 13, 2023. It is now read-only.

Bring clippy-check up to date with modern APIs #165

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

kristof-mattei
Copy link

@kristof-mattei kristof-mattei commented May 12, 2022

@glandium
Copy link

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.

@adam-azarchs
Copy link

Would be great to merge this, but I'd also note #162

@danieleades
Copy link

might be time to fork this repo...

@adam-azarchs
Copy link

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

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
4 participants