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

docs: add strip non essential fields option to SWC plugin doc #2175

Merged

Conversation

FelixMalfait
Copy link
Contributor

Description

As discussed in lingui/swc-plugin#137, we added an option to override the default behavior that strips fields in production builds

Types of changes

  • Bugfix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update
  • Examples update

Checklist

  • I have read the CONTRIBUTING and CODE_OF_CONDUCT docs
  • I have added tests that prove my fix is effective or that my feature works
  • I have added the necessary documentation (if appropriate)

Copy link

vercel bot commented Feb 5, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
js-lingui ✅ Ready (Inspect) Visit Preview Feb 6, 2025 8:37am

@FelixMalfait FelixMalfait changed the title Add strip non essential fields option to SWC plugin doc docs: add strip non essential fields option to SWC plugin doc Feb 5, 2025
Copy link

github-actions bot commented Feb 5, 2025

size-limit report 📦

Path Size
packages/core/dist/index.mjs 2.91 KB (0%)
packages/detect-locale/dist/index.mjs 618 B (0%)
packages/react/dist/index.mjs 1.35 KB (0%)

@andrii-bodnar
Copy link
Contributor

@FelixMalfait thank you!

I just checked the deployment preview and it looks a bit broken :)

Screenshot 2025-02-05 at 16 18 22

website/docs/ref/swc-plugin.md Show resolved Hide resolved
website/docs/ref/swc-plugin.md Show resolved Hide resolved
website/docs/ref/swc-plugin.md Show resolved Hide resolved
@FelixMalfait
Copy link
Contributor Author

Sorry I didn't know it was possible to make so many mistakes on a 3-lines PR 🤦‍♂️

@andrii-bodnar
Copy link
Contributor

@FelixMalfait thank you!

@andrii-bodnar andrii-bodnar merged commit f668c5c into lingui:main Feb 6, 2025
4 checks passed
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.

2 participants