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.

Sign up for GitHub

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

Jump to bottom

Add Codecov Bundle Analysis #6698

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Add Codecov Bundle Analysis #6698

wants to merge 2 commits into from
+27 −0

Conversation

katia-sentry
Copy link

💻 变更类型 | Change Type

  • ✨ feat
  • 🐛 fix
  • ♻️ refactor
  • 💄 style
  • 👷 build
  • ⚡️ perf
  • 📝 docs
  • 🔨 chore

🔀 变更说明 | Description of Change

  • Add @codecov/webpack-plugin for bundle size tracking
  • Configure bundle analysis settings in codecov.yml
  • Update CI workflow to include bundle analysis step
  • Set bundle change threshold to 10KB

📝 补充信息 | Additional Information

Add Codecov Bundle Analysis integration

Add Codecov Bundle Analysis integration
9adc8d4 
- Add @codecov/webpack-plugin for bundle size tracking
- Configure bundle analysis settings in codecov.yml
- Update CI workflow to include bundle analysis step
- Set bundle change threshold to 10KB
@vercel Vercel
Copy link

vercel bot commented Mar 4, 2025

Someone is attempting to deploy a commit to the LobeChat Desktop Team on Vercel.

A member of the Team first needs to authorize it.

@dosubot dosubot bot added the size:M This PR changes 30-99 lines, ignoring generated files. label Mar 4, 2025
@lobehubbot
Copy link
Member

👍 @katia-sentry

Thank you for raising your pull request and contributing to our Community
Please make sure you have followed our contributing guidelines. We will review it as soon as possible.
If you encounter any problems, please feel free to connect with us.
非常感谢您提出拉取请求并为我们的社区做出贡献，请确保您已经遵循了我们的贡献指南，我们会尽快审查它。
如果您遇到任何问题，请随时与我们联系。

@dosubot dosubot bot added the 🌠 Feature Request New feature or request | 特性与建议 label Mar 4, 2025
@katia-sentry katia-sentry changed the title Add Codecov Bundle Analysis integration Add Codecov Bundle Analysis Mar 4, 2025
@arvinxx
Copy link
Contributor

arvinxx commented Mar 4, 2025

I think it may not a good practice at this time as nextjs will mark turbopack prod stable soon? Do Codecov have an alternative in turbopack?

fyi: https://areweturboyet.com

@katia-sentry katia-sentry force-pushed the feat/codecov-bundle-analysis branch from c96cc1e to 8baadf4 Compare March 10, 2025 15:46
@dosubot dosubot bot added size:S This PR changes 10-29 lines, ignoring generated files. and removed size:M This PR changes 30-99 lines, ignoring generated files. labels Mar 10, 2025
@katia-sentry
Copy link
Author

@arvinxx thanks for the feedback - turbopack doesn’t support webpack plugins yet and we have opened an issue to track it codecov/codecov-javascript-bundler-plugins#251. We can update this once support is available. It’s up to the team whether to merge now or revisit later - happy to adjust as needed!

@arvinxx
Copy link
Contributor

arvinxx commented Mar 11, 2025

@katia-sentry can we have a separate build test workflow？ As the build will need 10mins+ to finish

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
🌠 Feature Request New feature or request | 特性与建议 size:S This PR changes 10-29 lines, ignoring generated files.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@katia-sentry @lobehubbot @arvinxx