bolt publish exit 0 despite npm error about bolt HOT 6 CLOSED

This is fixed in bolt 0.21.1!!

Thanks to @lachlanhunt for the PR.

from bolt.

Orly? I'll try to take a look today 👌

from bolt.

Ah okay. This is interesting, just thinking about the best way to resolve it.

Bolt doesn't exit automatically because we handle the errors ourselves in an external script (publish will actually return a list of packages that were successfully published and a list of unsuccessful attempts). That behaviour is probably not super consistent (commands should really use logging and exit codes, functions should return data).

We could either move the publish logic into an exposed function (similar to runWorkspaceTasks, getDependencyGraph, etc) and have the publish command just proxy to that, or we could add a flag to publish to tell it to exit on failure (either opt in or opt out).

Option 2 is probably easier, but option 1 is more correct...

from bolt.

I think we'll go with option 1.

@bradleyayers how much of a blocker is this for you? You could work around it pretty easily in the meantime (take a look at our script for reference). If it's urgent though, I'm happy to look at it this week?

from bolt.

Not a blocker for me, as it’s only happens when the NPM command fails, and that shouldn’t happen. It’s just a gotcha that I felt was worth raising so it can be fixed upstream. Thanks for looking into it!

from bolt.

Just an FYI, if you DO update to 0.21.1, be aware there was a minor breaking change, only if you were calling bolt.publish from a node script. You'll just need to change it to bolt.publishPackages and you'll be good though.
See #190

from bolt.