fix(style): improve output messaging #49
Conversation
src/main.rs
Outdated
| readme::copy_from_crate(&crate_path)?; | ||
| println!( | ||
| "{} {}Installing WASM-bindgen...", |
There was a problem hiding this comment.
Maybe we can make this an optional thing at some point? like if it's installed we just do 6 things rather than 7. But that's like a future improvement and not something that needs to be done now and block 0.1
There was a problem hiding this comment.
so currently, if wasm-bindgen is already installed this step doesn't actually do anything (it actually fails and yells in stderr). we should def check if it's installed and just skip if it is... both this and adding the target should just not happen if they are already present.
There was a problem hiding this comment.
In which case the numbering will also need to be dynamically done rather than it being hardcoded. Maybe a step at the beginning to calculate what can be done without then run the rest.
|
This is all very nice and much easier to see where things are happening :D |
ashleygwilliams
force-pushed
the
pretty-output
branch
2 times, most recently
from
e03fca6
to
d2d07d5
Compare
ashleygwilliams
force-pushed
the
pretty-output
branch
from
d2d07d5
to
ac9f5f7
Compare
fixes #39, #44, #46