Skip to content

Conversation

@jagthedrummer
Copy link
Contributor

Previously we'd show an error about missing argument, but then try to continue running the script, which would then throw a different less descriptive error, which makes it easy to miss the descriptive one that's shown earlier.

Previously we'd show an error about missing argument, but then try to
continue running the script, which would then throw a different less
descriptive error, which makes it easy to miss the descriptive one
that's shown earlier.
@jagthedrummer
Copy link
Contributor Author

On second thought, I'm not sure if this is going to change the way in which things break in ways that aren't helpful. Specifically we rely on bin/theme outputting a useful string, and with the old behavior it would actually raise an error after outputting a message. But with this change it would not raise that error.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants