Skip to content

feat: webpack-sources 的功能从webpack引入,避免执行引入版本有关问题#12501

Merged
Chen-jj merged 8 commits intoNervJS:nextfrom
yoyo837:sources-from-webpack
Sep 20, 2022
Merged

feat: webpack-sources 的功能从webpack引入,避免执行引入版本有关问题#12501
Chen-jj merged 8 commits intoNervJS:nextfrom
yoyo837:sources-from-webpack

Conversation

@yoyo837
Copy link
Contributor

@yoyo837 yoyo837 commented Sep 16, 2022 •
edited
Loading

这个 PR 做了什么? (简要描述所做更改)
webpack-sources 不自行依赖和引用, 直接从 webpack 导入, 减少因为版本差异导致的问题, 比如可能影响 filesystem cache

ref: dcasia/mini-program-tailwind#73

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

  • 错误修复(Bugfix) issue: fix #
  • 新功能(Feature)
  • 代码重构(Refactor)
  • TypeScript 类型定义修改(Typings)
  • 文档修改(Docs)
  • 代码风格更新(Code style update)
  • 其他，请描述(Other, please describe):

这个 PR 涉及以下平台:

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

@taro-bot2
Copy link

taro-bot2 bot commented Sep 16, 2022

欢迎提交 PR~ Taro 非常感谢您对开源事业做出的贡献！🌷🌷🌷

一般 PR 会在一到两周内进行 review，成功合入后会随下一个版本进行发布。

Review 需要耗费大量时间，所以请遵循以下规范，协助我们提高 review 效率🙏🙏🙏

  1. 详细介绍 PR 的背景（非常重要，例如解决了什么问题，该问题如何复现等）
  2. 确保 CI 顺利运行。
  3. 最好能提供对应的测试用例。

为了更好地进行沟通，请加入 Taro 开发者微信群：

1 similar comment
@taro-bot2
Copy link

taro-bot2 bot commented Sep 16, 2022

欢迎提交 PR~ Taro 非常感谢您对开源事业做出的贡献！🌷🌷🌷

一般 PR 会在一到两周内进行 review，成功合入后会随下一个版本进行发布。

Review 需要耗费大量时间，所以请遵循以下规范，协助我们提高 review 效率🙏🙏🙏

  1. 详细介绍 PR 的背景（非常重要，例如解决了什么问题，该问题如何复现等）
  2. 确保 CI 顺利运行。
  3. 最好能提供对应的测试用例。

为了更好地进行沟通，请加入 Taro 开发者微信群：

@biorz biorz added T-weapp Target - 编译到微信小程序 T-h5 Target - 编译到 H5 A-runner Area - Webpack runner 相关 labels Sep 16, 2022
@Chen-jj
Copy link
Contributor

Chen-jj commented Sep 20, 2022

@yoyo837 @tarojs/mini-runner 是否需要一并修改呢？

@Chen-jj Chen-jj added the question Further information is requested label Sep 20, 2022
@yoyo837
Copy link
Contributor Author

yoyo837 commented Sep 20, 2022

webpack@5.0.0-beta.27 才有从webpack导出这个.

@Chen-jj
Copy link
Contributor

Chen-jj commented Sep 20, 2022 •
edited
Loading

webpack@5.0.0-beta.27 才有从webpack导出这个.

学习了~ 感谢贡献~

@Chen-jj Chen-jj added V-3 Version - 3.x and removed question Further information is requested labels Sep 20, 2022
@Chen-jj
Copy link
Contributor

Chen-jj commented Sep 20, 2022

@yoyo837 麻烦更新一下 pnpm.lock 哈，CI 好像因为 lock 而挂了

@yoyo837
Copy link
Contributor Author

yoyo837 commented Sep 20, 2022

大佬,小弟没用过pnpm, 请问我需要做什么操作? 本地我pnpm install 报错: The chromium binary is not available for arm64:

@Chen-jj
Copy link
Contributor

Chen-jj commented Sep 20, 2022

大佬,小弟没用过pnpm, 请问我需要做什么操作? 本地我pnpm install 报错: The chromium binary is not available for arm64:

看来你的 mac 不支持这个包，我来操作一下~

@Chen-jj Chen-jj merged commit 556de96 into NervJS:next Sep 20, 2022
@yoyo837 yoyo837 deleted the sources-from-webpack branch September 20, 2022 08:09
@taro-bot2 taro-bot2 bot mentioned this pull request Sep 24, 2022
Closed
KkZsc pushed a commit to KkZsc/taro that referenced this pull request Dec 5, 2022
@yoyo837 @Chen-jj @KkZsc
fix(webpack5): 修复 webpack-sources 版本不一致带来的问题 (NervJS#12501)
3789318 
* feat: webpack-sources 的功能从webpack引入,避免执行引入版本有关问题

* chore: update lockfile

* fix: typings

Co-authored-by: chenjiajian <798095202@qq.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Chen-jj Chen-jj Chen-jj approved these changes

Assignees

No one assigned

Labels

A-runner Area - Webpack runner 相关 T-h5 Target - 编译到 H5 T-weapp Target - 编译到微信小程序 V-3 Version - 3.x

Projects

H5
Archived in project

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@yoyo837 @Chen-jj @biorz

Comments