esp-rs / esp-hal

no_std Hardware Abstraction Layers for ESP32 microcontrollers
https://docs.esp-rs.org/esp-hal/
Apache License 2.0
769 stars 214 forks source link

Add a `publish` subcommand to the xtask (which performs a dry-run by default) #2539

Closed jessebraham closed 1 week ago

jessebraham commented 1 week ago

Seems to work as far as I can tell 😅

jessebraham commented 1 week ago

Considered adding a publish command instead which optionally allows for dry run, but I'm afraid I'll forget the argument. Can invert the logic I guess, but something like cargo publish --publish seems weird 😅 Open to suggestions here

jessebraham commented 1 week ago

Okay I ended up changing it to publish, which performs a dry-run by default but can optionally be supplied --no-dry-run. This command is a bit spicy but will be handy I guess 😅