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

chore: drop @babel/plugin-proposal-class-properties #17385

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

Conversation

ianzone
Copy link

@ianzone ianzone commented Feb 27, 2025

这个 PR 做了什么? (简要描述所做更改)
将 @babel/plugin-proposal-class-properties 替换为 @babel/plugin-transform-class-properties

这个 PR 是什么类型? (至少选择一个)

这个 PR 涉及以下平台:

  • 所有小程序
  • 微信小程序
  • 支付宝小程序
  • 百度小程序
  • 字节跳动小程序
  • QQ 轻应用
  • 京东小程序
  • 快应用平台(QuickApp)
  • Web 平台(H5)
  • 移动端(React-Native)
  • 鸿蒙(harmony)

@tutuxxx tutuxxx self-assigned this Feb 27, 2025
@tutuxxx tutuxxx added this to the 4.0.10 milestone Feb 27, 2025
@ianzone
Copy link
Author

ianzone commented Feb 27, 2025

之前没更新lockfile,重新跑一下CI哈

@tutuxxx
Copy link
Member

tutuxxx commented Feb 28, 2025

之前没更新lockfile,重新跑一下CI哈

更新后ci异常了

@ianzone
Copy link
Author

ianzone commented Feb 28, 2025

之前没更新lockfile,重新跑一下CI哈

更新后ci异常了

删除了pnpm-lock.yaml和node_modules,又重新pnpm install了一遍

@ianzone
Copy link
Author

ianzone commented Feb 28, 2025

packageExtensions 字段可以删掉吗?里面列出的 peerDependencies都是可选的,但是在CI里老是报错。 @tutuxxx tutuxxx

@tutuxxx
Copy link
Member

tutuxxx commented Feb 28, 2025

packageExtensions 字段可以删掉吗?里面列出的 peerDependencies都是可选的,但是在CI里老是报错。 @tutuxxx tutuxxx

不能,你用下面的环境更新lock文件试试
▶ node -v
v18.18.2
1d6h
▶ pnpm -v
9.15.5

@ianzone
Copy link
Author

ianzone commented Feb 28, 2025

packageExtensions 字段可以删掉吗?里面列出的 peerDependencies都是可选的,但是在CI里老是报错。 @tutuxxx tutuxxx

不能,你用下面的环境更新lock文件试试 ▶ node -v v18.18.2 1d6h ▶ pnpm -v 9.15.5

改 pnpm9.15.5 重装了一遍

@ianzone
Copy link
Author

ianzone commented Feb 28, 2025

添加了两处 ts-ignore 以通过CI, 代码和 main 一致

@tutuxxx
Copy link
Member

tutuxxx commented Mar 3, 2025

添加了两处 ts-ignore 以通过CI, 代码和 main 一致

尝试一下其他办法?尽量不使用ts-ignore

@ianzone
Copy link
Author

ianzone commented Mar 3, 2025

添加了两处 ts-ignore 以通过CI, 代码和 main 一致

尝试一下其他办法?尽量不使用ts-ignore

好的。另外还有两个事希望同意

  1. 可否删除 npmrc 中的 registry="https://registry.yarnpkg.com/"?这个下载太慢了,国内用npm一般用的镜像源。
  2. 可否引入nx?引入后一是可以方便查看模块间依赖关系,二是可以产生构建缓存,免得本地构建测试的时候要等半天。

@tutuxxx
Copy link
Member

tutuxxx commented Mar 4, 2025

好的。另外还有两个事希望同意

  1. 可否删除 npmrc 中的 registry="https://registry.yarnpkg.com/"?这个下载太慢了,国内用npm一般用的镜像源。
  2. 可否引入nx?引入后一是可以方便查看模块间依赖关系,二是可以产生构建缓存,免得本地构建测试的时候要等半天。

第一个不是很有必要,更多的安装操作还是在ci中进行的;
第二个可以提pr我们看看

@ianzone
Copy link
Author

ianzone commented Mar 4, 2025

好的。另外还有两个事希望同意

  1. 可否删除 npmrc 中的 registry="https://registry.yarnpkg.com/"?这个下载太慢了,国内用npm一般用的镜像源。
  2. 可否引入nx?引入后一是可以方便查看模块间依赖关系,二是可以产生构建缓存,免得本地构建测试的时候要等半天。

第一个不是很有必要,更多的安装操作还是在ci中进行的; 第二个可以提pr我们看看

我知道安装是在CI进行的,但是本地开发也要下载依赖呀,删除这一行也不会让CI从国内源下载,只会将源还原成默认的https://registry.npmjs.org/,但如果不删的话会覆盖本地~/.npmrc设置的国内源,导致经常无法下载。

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.

模板执行 dev:weapp 报错 Error: [BABEL] xxx Cannot find module '@babel/plugin-proposal-class-properties'
2 participants