zerocopy icon indicating copy to clipboard operation
zerocopy copied to clipboard

Test Clippy in CI

Open joshlf opened this issue 3 years ago • 2 comments

Exactly what it says on the tin.

@frazar is working on this in #51

joshlf avatar Oct 11 '22 18:10 joshlf

@frazar, if you leave a comment here, then I can assign this to you. Otherwise, it won't let me.

joshlf avatar Oct 16 '22 18:10 joshlf

Sure thing! I'll be working towards closing this issue

frazar avatar Oct 16 '22 19:10 frazar

Hahaha you know what I just realized? That PR only runs Clippy on zerocopy, but not on zerocopy-derive. Mind putting up another one for zerocopy-derive? It's just the exact same thing, but with an extra --manifest-path argument. E.g.:

https://github.com/google/zerocopy/blob/e5e9b1fa05933750db526a489f73050032c5eb70/.github/workflows/ci.yml#L90-L91

joshlf avatar Oct 17 '22 19:10 joshlf

Sure!

frazar avatar Oct 17 '22 19:10 frazar