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

test: remove unused code #50560

Merged
merged 1 commit into from
Aug 26, 2024
Merged

test: remove unused code #50560

merged 1 commit into from
Aug 26, 2024

Conversation

afc163
Copy link
Member

@afc163 afc163 commented Aug 23, 2024

中文版模板 / Chinese template

🤔 This is a ...

  • 🆕 New feature
  • 🐞 Bug fix
  • 📝 Site / documentation improvement
  • 📽️ Demo improvement
  • 💄 Component style improvement
  • 🤖 TypeScript definition improvement
  • 📦 Bundle size optimization
  • ⚡️ Performance optimization
  • ⭐️ Feature enhancement
  • 🌐 Internationalization
  • 🛠 Refactoring
  • 🎨 Code style optimization
  • ✅ Test Case
  • 🔀 Branch merge
  • ⏩ Workflow
  • ❓ Other (about what?)

🔗 Related Issues

  • Describe the source of related requirements, such as links to relevant issue discussions.
  • For example: close #xxxx, fix #xxxx

close #50549

💡 Background and Solution

  • The specific problem to be addressed.
  • List the final API implementation and usage if needed.
  • If there are UI/interaction changes, consider providing screenshots or GIFs.

这一行代码在 #47264 (comment) 加的,我没试出来具体要如何触发,好像没用?

📝 Change Log

Language Changelog
🇺🇸 English --
🇨🇳 Chinese --

Copy link

stackblitz bot commented Aug 23, 2024

Review PR in StackBlitz Codeflow Run & review this pull request in StackBlitz Codeflow.

Copy link
Contributor

github-actions bot commented Aug 23, 2024

Preview is ready

Copy link
Contributor

github-actions bot commented Aug 23, 2024

👁 Visual Regression Report for PR #50560 Passed ✅

🎯 Target branch: master (714a6e1)
📖 View Full Report ↗︎

🎊 Congrats! No visual-regression diff found.

Copy link
Contributor

size-limit report 📦

Path Size
./dist/antd.min.js 346.73 KB (+209 B 🔺)
./dist/antd-with-locales.min.js 397.41 KB (+53 B 🔺)

Copy link

pkg-pr-new bot commented Aug 23, 2024

Copy link

cloudflare-workers-and-pages bot commented Aug 23, 2024

Deploying ant-design with  Cloudflare Pages  Cloudflare Pages

Latest commit: af5d785
Status: ✅  Deploy successful!
Preview URL: https://5765303a.ant-design.pages.dev
Branch Preview URL: https://test-fix-unused-code.ant-design.pages.dev

View logs

Copy link

codecov bot commented Aug 23, 2024

Bundle Report

Changes will decrease total bundle size by 44 bytes ⬇️

Bundle name Size Change
antd-array-push 3.31MB 44 bytes ⬇️

@afc163 afc163 requested a review from crazyair August 24, 2024 03:50
@afc163 afc163 mentioned this pull request Aug 26, 2024
16 tasks
@afc163 afc163 merged commit acc10b4 into master Aug 26, 2024
66 checks passed
@afc163 afc163 deleted the test/fix-unused-code branch August 26, 2024 04:05
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