-
Notifications
You must be signed in to change notification settings - Fork 385
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
lingui extract should fail with a non-zero exit code when there are extraction failures #1419
Comments
I was thinking how to hot-fix this today as well. |
Hi, we are currently reworked an entire codebase for cli and extractor in V4. And this task already scheduled for new upcoming release. So "hofixing" to the main branch wouldn't have a lot of sense. If you want to help, you can implement this for the "next" branch |
@thekip is this still relevant for the v4? |
Yes. Few of the checks are still "warning only", the big changes should be made to the codebase to stop processing after encountering these warnings. |
It looks like it's not that critical and a big codebase change won't be worth it, what do you think? |
There was already a work on that:
#1299
But still there are cases where errors just logged to console and zero code is returned.
For example, "Missing Message ID" warning.
What should be done:
We should add a flag to the cli extract, for example
--strict
which will exit with non-zero code in case of any extraction problems encountered.Related:
Look what hacks users have to do to achieve this:
https://github.com/KlimaDAO/klimadao/blob/staging/scripts/lingui.js
The text was updated successfully, but these errors were encountered: