-
Notifications
You must be signed in to change notification settings - Fork 18
Open
Labels
Description
There are a couple of ways this could be achieved:
- Fail-fast with a nice error message if
EDITOR
has not been defined. - Ask the developer to enter the path to their editor in lieu of
EDITOR
not being defined.
Ideally, my preference would be for option 2, as I plan to use publish-release within the postpublish
hook of all our modules, and this would allow developers who run npm publish
to still have things work, rather than requiring them to define EDITOR
, then re-run npm run postpublish
.