-
Notifications
You must be signed in to change notification settings - Fork 4.8k
chore: drop @babel/plugin-proposal-class-properties #17385
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
Conversation
之前没更新lockfile,重新跑一下CI哈 |
更新后ci异常了 |
删除了pnpm-lock.yaml和node_modules,又重新pnpm install了一遍 |
packageExtensions 字段可以删掉吗?里面列出的 peerDependencies都是可选的,但是在CI里老是报错。 @tutuxxx tutuxxx |
不能,你用下面的环境更新lock文件试试 |
改 pnpm9.15.5 重装了一遍 |
添加了两处 ts-ignore 以通过CI, 代码和 main 一致 |
尝试一下其他办法?尽量不使用ts-ignore |
好的。另外还有两个事希望同意
|
第一个不是很有必要,更多的安装操作还是在ci中进行的; |
我知道安装是在CI进行的,但是本地开发也要下载依赖呀,删除这一行也不会让CI从国内源下载,只会将源还原成默认的 |
妥否? 如果没有理由非要从yarnpkg源下载的话我就删掉了 @tutuxxx |
ts-ignore 去掉了,这个项目技术债太多了,真的得重构了 |
麻烦再跑下CI哈🙏 |
麻烦再允许下CI🙏 |
更新了测试快照,辛苦再开一次🙏 |
本地删除node_modules和lock文件全新安装一下看看 |
还是一样的问题,我还是手动修改吧。 |
快照更新了,再跑下试试 |
换回了main的lockfile,消除了构建错误,请再跑一遍 |
可否开放PR自动触发CI的权限?这样来来回回感觉非常耗时。其他项目像Antd 和 React Native都是自动运行CI,然后合并时才审核的。 |
我同时也是antd那边社区成员的,那边的CI部分会直接运行,部分仍然需要Approve,只是antd那边活跃人多一些响应勤快。 |
就是因为人少所以要更依赖自动化呀,不然一个错误可能等半个月跑了CI才知道,改了又要等人触发才能测试。 |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #17385 +/- ##
========================================
Coverage 58.53% 58.53%
========================================
Files 468 468
Lines 27138 27138
Branches 5749 5940 +191
========================================
Hits 15884 15884
+ Misses 9932 9745 -187
- Partials 1322 1509 +187
Flags with carried forward coverage won't be shown. Click here to find out more.
🚀 New features to boost your workflow:
|
我觉得可能是因为需要approve的CI可能是使用的有限资源,为避免被无效PR消耗殆尽。 |
有这个因素,但这个仓库太过臃肿,CI能跑20分钟,我认为应该进行优化
退一步说,这个项目本身也没有Antd活跃,PR没有那么频繁。 |
不能这样说,antd全用的免费的。没买过。 🤣 每个项目预算不同。 |
我个人支持这个 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
@tutuxxx 嗨👋,可否抽空合并一下,我后续还有一些PR需要基于这个 |
为积极回应贡献者的热情,我先合并了。有问题再修再撤也行。 |
这个 PR 做了什么? (简要描述所做更改)
将 @babel/plugin-proposal-class-properties 替换为 @babel/plugin-transform-class-properties
这个 PR 是什么类型? (至少选择一个)
这个 PR 涉及以下平台: