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

fix: resolve dependency version changes after pnpm upgrade #620

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

Conversation

LZS911
Copy link
Collaborator

@LZS911 LZS911 commented Apr 1, 2025

升级 pnpm 版本,并处理依赖升级后的一系列问题

以下依赖版本未做升级:

antd:ts 类型错误的罪魁祸首,升级后 form 相关的类型变更很大,并且担心样式出现问题,暂时也不对 antd 升级,可以以后单独升级处理。

react-redux, react-router-dom:升级小版本会出现一些 warn 信息,并且 react-redux 用法还有些变化,暂时先不动。

对以下依赖进行了大版本更新:

eslint 升级到了 v9

@testing-library/react 升级到了 v16

此次变更后 antd 的 css hash 全部发生了变化,具体原因还在探究中,导致单元测试快照几乎全部产生了变更,所以本 pr 暂时还不能合并。

后续处理:

  1. 调整项目中快照匹配的数量,目前单元测试代码中快照匹配滥用现象非常严重,会在本次调整中处理,调整后单元测试执行速度应该会有比较大的提升
  2. antd css hash 变更原因探究

@actiontech-bot actiontech-bot requested a review from Rain-1214 April 1, 2025 10:35
[fix]: remove redundant ConfigProvider instances
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