Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Don't fail on empty message in MessageFormat hook #347

Merged
merged 1 commit into from
Feb 19, 2016
Merged

Don't fail on empty message in MessageFormat hook #347

merged 1 commit into from
Feb 19, 2016

Conversation

jawshooah
Copy link
Collaborator

The EmptyMessage hook already covers this case.

The EmptyMessage hook already covers this case.
@lencioni
Copy link
Collaborator

Seems reasonable 👍

jawshooah added a commit that referenced this pull request Feb 19, 2016
Don't fail on empty message in MessageFormat hook
@jawshooah jawshooah merged commit 9c62b0c into sds:master Feb 19, 2016
@jawshooah jawshooah deleted the message-pattern-ignore-empty branch February 19, 2016 17:47
@lencioni
Copy link
Collaborator

@jawshooah if you haven't already, will you please add an entry to the changelog about this change?

@lencioni
Copy link
Collaborator

Oh wait, I noticed that the next release will be the first release with this hook, so nevermind.

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