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

feat: add option to strip non essential fields #137

Merged

Conversation

FelixMalfait
Copy link
Contributor

Fixes #136

As mentioned in the issue I have no experience with Rust and this is mostly AI generated code, so please close if this is not relevant

@timofei-iatsenko
Copy link
Collaborator

The code changes looks ok to me, but it messed up with test i believe )

@FelixMalfait
Copy link
Contributor Author

@timofei-iatsenko I think I fixed it before you made this comment, would you mind having a look again? I can remove the tests if you think it's overkill

@timofei-iatsenko
Copy link
Collaborator

LGTM, could you also add a brief description of this parameter to the package readme? There is code snippet with plugin parameters you can add it there.

@andrii-bodnar
Copy link
Contributor

@FelixMalfait thanks for the contribution!

It would also be great to add a short description of the new parameter here - https://lingui.dev/ref/swc-plugin

@FelixMalfait
Copy link
Contributor Author

Sure! Done

@timofei-iatsenko timofei-iatsenko self-requested a review February 5, 2025 16:16
@andrii-bodnar andrii-bodnar changed the title Add option to strip non essential fields feat: add option to strip non essential fields Feb 6, 2025
@andrii-bodnar andrii-bodnar merged commit 009a0ff into lingui:main Feb 6, 2025
3 checks passed
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.

Disable "Strip non essential fields" in prod environment
3 participants