document how to get rustfmt on nightly in "up and running" about wasm-pack HOT 7 CLOSED

@ashleygwilliams I personally don't mind. It's just a quick fix. It'd be nice if there was some kind of pre-push hook but you can't distribute those easily as part of a repo which has always seemed like a dumb move on git's part. I'm for keeping it to enforce style. We can always make it a part of the check list when submitting a PR by creating a template that GitHub can use.

from wasm-pack.

yeah, still haven't fixed it yet, but haven't put in the time yet. I don't mind having development be nightly only for now. I don't mind if it stays tbh; it's def useful!

from wasm-pack.

How about adding them in pre-push hooks?

And I do agree with @yoshuawuyts have it but don't worry about it atleast for now

from wasm-pack.

@sendilkumarn i think it would be just as frustrating as a pre-push hook, and a lot more to set up. i'll see what other contribs think (@mgattozzi has been bit by this so much, thankful for his patience), but sounds like we're should probably just keep it?

if someone wants to add a section to the current "up and running" section of the README about how to get cargo fmt setup correctly i would welcome a PR!

from wasm-pack.

I can try and add some documentation for this, having just run into this myself. If it is worth anything, getting rustfmt working was not hard, and I didn't find it too bad as a new contributor :)

from wasm-pack.

Awesome! :D

from wasm-pack.

closed by #88

from wasm-pack.