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

Add exclude option #153

Merged
merged 3 commits into from
Apr 27, 2022
Merged

Add exclude option #153

merged 3 commits into from
Apr 27, 2022

Conversation

dgrabla
Copy link
Contributor

@dgrabla dgrabla commented Feb 2, 2022

This option can be used multiple times to exclude keys from the
extraction / report for those situations where the 'suspected dynamic
keys' logic is not enough and the user prefers to drop known keys.

It only makes sense for the unused keys, therefore it does not affect
the missing keys.

Link: #141

This option can be used multiple times to exclude keys from the
extraction / report for those situations where the 'suspected dynamic
keys' logic is not enough and the user prefers to drop known keys.

It only makes sense for the unused keys, therefore it does not affect
the missing keys.

Link: Spittal#141
In a different commit to ease review
@dgrabla
Copy link
Contributor Author

dgrabla commented Feb 14, 2022

@pixari I would like to merge this, so we have the feature on the next release.

@dgrabla
Copy link
Contributor Author

dgrabla commented Feb 21, 2022

@Spittal 👋

@dgrabla
Copy link
Contributor Author

dgrabla commented Mar 2, 2022

@Spittal @pixari 👋

@Spittal Spittal merged commit 4121a69 into Spittal:master Apr 27, 2022
josestbernard pushed a commit to josestbernard/vue-i18n-extract that referenced this pull request Nov 3, 2022
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