-
Notifications
You must be signed in to change notification settings - Fork 364
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
Fix default ignore pattern to work on Windows #70
Comments
Hello @herzaso, I'm not familiar with |
It may be caused that Could you please try to add this section to your {
"lingui": {
"srcPathIgnorePatterns": ["node_modules"]
}
} |
@tricoder42 you are right, it was caused by the wrong path delimiter (maybe consider changing the regexp to support windows delimiter also). Other than that, I know have another problem:
|
Do I understand it correctly, that you actually have this {
"plugins": [
"syntax-dynamic-import"
],
"env": {
"test": {
"plugins": [
"dynamic-import-node"
]
}
},
"presets": [
"env",
"react",
"lingui-react"
]
} All what you do in |
Yes. I have added the |
Nevermind, I have to add the "transform-object-rest-spread" plugin ... |
Ah, you're right! That plugin isn't part of Thank you for reporting the bug with ignore patterns. I'll fix it soon. |
It should be fixed on latest release of |
Running the cli did produce the table with number of labels although it didn't show any file being collected (whereas before it did log the file paths that were collected). Since I wasn't sure, I removed the locale directory and ran
|
I made the output more silent. Running The error is, however, weird. I'll take a look |
I'm using
create-react-app
withreact-app-rewired
to enable making changes to webpack/babel without ejecting.I've added the following
config-overrides.js
file (areact-app-rewired
requirement):This make sure to load the
.babelrc
file with the needed presets:While this approach is working,
extract
is failing with the following exception:Expected:
While I think this exception should be addressed, I think it would also be beneficial to allow specifying the files to include/exclude from the extraction process
The text was updated successfully, but these errors were encountered: