-
Notifications
You must be signed in to change notification settings - Fork 409
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Build crate in a workspace #430
Build crate in a workspace #430
Conversation
r=me, but looks like rustfmt on CI disagrees with your local rustfmt? |
Huh. I did a rustup and still I'm not getting any more diffs... I wonder if Travis is using an old Rust?? |
Stable vs nightly perhaps? |
I just double checked that our CI config was using nightly (and did #432 at the same time) and confirmed that yes it is. And I am also using nightly... |
Added some logging of the rustfmt version to #432 |
Looks like I am using an out-of-date rustfmt! Mine:
But I can't update mine further...
|
Huh, looks like it was clippy holding me back
|
2d141d6
to
1f799dc
Compare
1f799dc
to
8abedfe
Compare
Make sure these boxes are checked! 📦✅
rustfmt
installed and have yourcloned directory set to nightly
$ rustup override set nightly $ rustup component add rustfmt-preview --toolchain nightly
rustfmt
on the code base before submitting✨✨ 😄 Thanks so much for contributing to wasm-pack! 😄 ✨✨