Skip to content

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

Merged
merged 5 commits into from
Mar 25, 2025
Merged

Conversation

ianzone
Copy link
Contributor

@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
Contributor Author

ianzone commented Feb 27, 2025

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

@tutuxxx
Copy link
Member

tutuxxx commented Feb 28, 2025

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

更新后ci异常了

@ianzone
Copy link
Contributor Author

ianzone commented Feb 28, 2025

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

更新后ci异常了

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

@ianzone
Copy link
Contributor 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
Contributor 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
Contributor 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
Contributor 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
Contributor 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设置的国内源,导致经常无法下载。

@ianzone
Copy link
Contributor Author

ianzone commented Mar 5, 2025

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

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

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

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

妥否? 如果没有理由非要从yarnpkg源下载的话我就删掉了 @tutuxxx

@ianzone
Copy link
Contributor Author

ianzone commented Mar 6, 2025

ts-ignore 去掉了,这个项目技术债太多了,真的得重构了

@ianzone
Copy link
Contributor Author

ianzone commented Mar 22, 2025

麻烦再跑下CI哈🙏

@ianzone
Copy link
Contributor Author

ianzone commented Mar 24, 2025

麻烦再允许下CI🙏

@ianzone
Copy link
Contributor Author

ianzone commented Mar 24, 2025

更新了测试快照,辛苦再开一次🙏

@ianzone
Copy link
Contributor Author

ianzone commented Mar 24, 2025

真是醉了,本地测试和CI测试匹配的快照是不一样的。容我再手动更新一下快照。
image

@yoyo837
Copy link
Contributor

yoyo837 commented Mar 24, 2025

本地删除node_modules和lock文件全新安装一下看看

@ianzone
Copy link
Contributor Author

ianzone commented Mar 24, 2025

本地删除node_modules和lock文件全新安装一下看看

还是一样的问题,我还是手动修改吧。

@ianzone
Copy link
Contributor Author

ianzone commented Mar 24, 2025

快照更新了,再跑下试试

@ianzone
Copy link
Contributor Author

ianzone commented Mar 24, 2025

换回了main的lockfile,消除了构建错误,请再跑一遍

@ianzone
Copy link
Contributor Author

ianzone commented Mar 24, 2025

可否开放PR自动触发CI的权限?这样来来回回感觉非常耗时。其他项目像Antd 和 React Native都是自动运行CI,然后合并时才审核的。

@yoyo837
Copy link
Contributor

yoyo837 commented Mar 25, 2025

可否开放PR自动触发CI的权限?这样来来回回感觉非常耗时。其他项目像Antd 和 React Native都是自动运行CI,然后合并时才审核的。

我同时也是antd那边社区成员的,那边的CI部分会直接运行,部分仍然需要Approve,只是antd那边活跃人多一些响应勤快。

@ianzone
Copy link
Contributor Author

ianzone commented Mar 25, 2025

可否开放PR自动触发CI的权限?这样来来回回感觉非常耗时。其他项目像Antd 和 React Native都是自动运行CI,然后合并时才审核的。

我同时也是antd那边社区成员的,那边的CI部分会直接运行,部分仍然需要Approve,只是antd那边活跃人多一些响应勤快。

就是因为人少所以要更依赖自动化呀,不然一个错误可能等半个月跑了CI才知道,改了又要等人触发才能测试。

Copy link

codecov bot commented Mar 25, 2025

Codecov Report

Attention: Patch coverage is 0% with 2 lines in your changes missing coverage. Please review.

Project coverage is 58.53%. Comparing base (24ba2c5) to head (9f6a527).
Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
packages/taro-helper/src/utils.ts 0.00% 0 Missing and 2 partials ⚠️
Additional details and impacted files

Impacted file tree graph

@@           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     
Flag Coverage Δ
taro-cli 72.37% <ø> (ø)
taro-runtime 59.87% <ø> (ø)
taro-web 52.77% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
packages/babel-preset-taro/index.js 73.03% <ø> (ø)
packages/taro-helper/src/utils.ts 22.19% <0.00%> (ø)

... and 35 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@yoyo837
Copy link
Contributor

yoyo837 commented Mar 25, 2025

可否开放PR自动触发CI的权限?这样来来回回感觉非常耗时。其他项目像Antd 和 React Native都是自动运行CI,然后合并时才审核的。

我同时也是antd那边社区成员的,那边的CI部分会直接运行,部分仍然需要Approve,只是antd那边活跃人多一些响应勤快。

就是因为人少所以要更依赖自动化呀,不然一个错误可能等半个月跑了CI才知道,改了又要等人触发才能测试。

我觉得可能是因为需要approve的CI可能是使用的有限资源,为避免被无效PR消耗殆尽。

@ianzone
Copy link
Contributor Author

ianzone commented Mar 25, 2025

可否开放PR自动触发CI的权限?这样来来回回感觉非常耗时。其他项目像Antd 和 React Native都是自动运行CI,然后合并时才审核的。

我同时也是antd那边社区成员的,那边的CI部分会直接运行,部分仍然需要Approve,只是antd那边活跃人多一些响应勤快。

就是因为人少所以要更依赖自动化呀,不然一个错误可能等半个月跑了CI才知道,改了又要等人触发才能测试。

我觉得可能是因为需要approve的CI可能是使用的有限资源,为避免被无效PR消耗殆尽。

有这个因素,但这个仓库太过臃肿,CI能跑20分钟,我认为应该进行优化

  1. 对仓库进行拆分 拆分仓库 #17474
  2. 引入更严格的本地静态测试 替换 husky,eslint,添加 biome,nx #17390
  3. 引入 nx 云缓存加快 CI 速度 https://nx.dev/ci/features/remote-cache

退一步说,这个项目本身也没有Antd活跃,PR没有那么频繁。
退一万步说,JD也算大厂了,淘宝付得起CI,JD应该不至于付不起吧。

@yoyo837
Copy link
Contributor

yoyo837 commented Mar 25, 2025

不能这样说,antd全用的免费的。没买过。 🤣 每个项目预算不同。

@ianzone
Copy link
Contributor Author

ianzone commented Mar 25, 2025

不能这样说,antd全用的免费的。没买过。 🤣 每个项目预算不同。

好吧,那这仨可以麻烦审一下嘛🙏 #17390 #17475 #17477

@yoyo837
Copy link
Contributor

yoyo837 commented Mar 25, 2025

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

我个人支持这个

Copy link
Contributor

@yoyo837 yoyo837 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@ianzone
Copy link
Contributor Author

ianzone commented Mar 25, 2025

@tutuxxx 嗨👋,可否抽空合并一下,我后续还有一些PR需要基于这个

@yoyo837
Copy link
Contributor

yoyo837 commented Mar 25, 2025

为积极回应贡献者的热情,我先合并了。有问题再修再撤也行。

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'
3 participants