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(taro-webpack-runner): add babel loader option to override default… #7096

Closed

Conversation

leoswing
Copy link

这个 PR 做了什么? (简要描述所做更改)

  1. 新增扩展配置 babelLoaderOption,修复使用 Taro next 时 babel-loader 默认配置 exclude node_modules 带来的问题,便于应用方需要对 node_modules 里的特定包进行 babel 遍历特殊处理的场景.

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

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

这个 PR 满足以下需求:

  • 提交到 master 分支
  • Commit 信息遵循 Angular Style Commit Message Conventions
  • 所有测试用例已经通过
  • 代码遵循相关包中的 .eslintrc, .tslintrc, .stylelintrc 所规定的规范
  • 在本地测试可用,不会影响到其它功能

这个 PR 涉及以下平台:

  • 微信小程序
  • 支付宝小程序
  • 百度小程序
  • 头条小程序
  • QQ 轻应用
  • 快应用平台(QuickApp)
  • Web 平台(H5)
  • 移动端(React-Native)

其它需要 Reviewer 或社区知晓的内容:

@Chen-jj Chen-jj added the V-3 Version - 3.x label Jul 17, 2020
@Chen-jj
Copy link
Contributor

Chen-jj commented Jul 28, 2020

@leoswing 这种情况使用 webpackChain 不是就可以了么?

如果实在需要一种更便捷的修改方式,感觉同步小程序端的配置 mini.compile,会更加合适。

@leoswing
Copy link
Author

@leoswing 这种情况使用 webpackChain 不是就可以了么?

如果实在需要一种更便捷的修改方式,感觉同步小程序端的配置 mini.compile,会更加合适。

使用 webpackChain 没法支持修改默认的babel-loader 解析这块,另外这块babel默认不解析业务的 node_modules,会导致如果业务方需要处理 node_modules 的包的话出现问题

@Chen-jj
Copy link
Contributor

Chen-jj commented Jul 29, 2020

@leoswing webpackChain 可以改 babel-loader 的参数呀

@leoswing
Copy link
Author

loader

请问是怎么改。。本来就测试了 使用 webpackChain 进行修改,完全不行。。。

@Chen-jj
Copy link
Contributor

Chen-jj commented Jul 29, 2020

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
V-3 Version - 3.x
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants