-
Notifications
You must be signed in to change notification settings - Fork 4.8k
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
chore: drop @babel/plugin-proposal-class-properties #17385
base: main
Are you sure you want to change the base?
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从国内源下载,只会将源还原成默认的 |
这个 PR 做了什么? (简要描述所做更改)
将 @babel/plugin-proposal-class-properties 替换为 @babel/plugin-transform-class-properties
这个 PR 是什么类型? (至少选择一个)
这个 PR 涉及以下平台: