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

feat(mini-runner): 为小程序端新增若干编译配置项 #7253

Merged
merged 2 commits into from
Aug 21, 2020

Conversation

Chen-jj
Copy link
Contributor

@Chen-jj Chen-jj commented Aug 4, 2020

这个 PR 做了什么?

  • 同步 H5 端的 sourceMapType 配置
  • 新增 debugReact 配置,可配置是否使用 development 版本的 React,默认使用 production 版本
  • 新增 minifyXML 配置,可配置是否去除 xml 文件中的空格,默认不去除

这个 PR 是什么类型?

  • 新功能(Feature)

这个 PR 满足以下需求:

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

这个 PR 涉及以下平台:

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

* 同步 H5 端的 sourceMapType 配置
* 新增 debugReact 配置,可配置是否使用 development 版本的 React,默认使用 production 版本
* 新增 minifyXML 配置,可配置是否去除 xml 文件中的空格,默认不去除
@luckyadam
Copy link
Member

sourceMapType 和 enableSourceMap 是否可以统一

@Chen-jj
Copy link
Contributor Author

Chen-jj commented Aug 19, 2020

sourceMapType 和 enableSourceMap 是否可以统一

2.x 和 3.x H5 沿用的都是这两个配置分开诶,现在统一起来不兼容会不会不是很好?

sourceMapType: Joi.string(),
debugReact: Joi.bool(),
minifyXML: Joi.object().keys({
collapseWhitespace: Joi.bool()
Copy link
Member

Choose a reason for hiding this comment

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

minifyXML 下有且只有 collapseWhitespace 这一配置么

# Conflicts:
#	packages/taro-mini-runner/src/plugins/MiniPlugin.ts
@Chen-jj Chen-jj merged commit 4c691f5 into next Aug 21, 2020
@Chen-jj Chen-jj deleted the feat/mini-options branch August 21, 2020 10:06
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.

2 participants