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 warn about duplicate default message if it's empty #502

Merged
merged 1 commit into from
May 19, 2019

Conversation

filipcro
Copy link
Contributor

@filipcro filipcro commented May 17, 2019

I couldn't run test on my PC so I'll use this CI for this now.

Fixes #490

@filipcro filipcro marked this pull request as ready for review May 17, 2019 16:41
@filipcro
Copy link
Contributor Author

So packages/cli/src/lingui-extract.test.js passed on my PC, don't know why it timed out here.

All tests for extract.js (only file I changed) passed on CI. But on my PC I got the following message:

Cannot find module './extract' from 'extract.test.js'

Can you help me run those tests so I can fix this issue?

@tricoder42 tricoder42 changed the title Fixed #490 Don't warn about duplicate default message if it's empty May 19, 2019
@tricoder42 tricoder42 merged commit ea269ba into lingui:master May 19, 2019
@tricoder42
Copy link
Contributor

Hmm, weird, Tests pass for me as well, so it's hard to tell.

Anyway, the PR looks pretty good! Thank you! I'll release it next week.

@tricoder42
Copy link
Contributor

Released in v2.8.2.

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

Successfully merging this pull request may close these issues.

One default message for multiple keys
2 participants