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

chore: change config #548

Merged
merged 2 commits into from
Dec 24, 2020
Merged

chore: change config #548

merged 2 commits into from
Dec 24, 2020

Conversation

SoloJiang
Copy link
Contributor

@SoloJiang SoloJiang commented Dec 24, 2020

  • compileDependencies 默认值改为 [],在目前 node_modules 分端代码移除没生效的情况下,考虑编译速度,默认值应该是 []
  • 支持配置 hydrate: true
  • TerserPlugin 配置默认改为移除未使用到的代码

@@ -47,7 +47,7 @@ module.exports = [
},
{
name: 'compileDependencies',
defaultValue: [''],
defaultValue: [],
Copy link
Contributor

Choose a reason for hiding this comment

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

这个会不会对现有业务有影响

Copy link
Contributor Author

Choose a reason for hiding this comment

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

有一定的影响,这种不兼容性问题就 case by case 来看吧,目前感觉全量编译价值不大还会拖慢大部分用户速度

@codecov-io
Copy link

Codecov Report

Merging #548 (86ebdbb) into release-next (279aa95) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@              Coverage Diff              @@
##           release-next     #548   +/-   ##
=============================================
  Coverage         87.80%   87.80%           
=============================================
  Files                21       21           
  Lines               779      779           
  Branches            196      196           
=============================================
  Hits                684      684           
  Misses               86       86           
  Partials              9        9           

@SoloJiang SoloJiang merged commit e0884be into release-next Dec 24, 2020
@SoloJiang SoloJiang deleted the chore/config branch December 24, 2020 06:31
SoloJiang added a commit that referenced this pull request Dec 24, 2020
* chore: bump version

* chore: update version

* chore: migrate plugin-ssr (#540)

* chore: migrate plugin-ssr

* chore: update references

* fix: kraken mpa error (#541)

* feat: support cloud IDE (#542)

* feat: support cloud IDE

* chore: typo

* chore: change pha get url prefix method

* chore: remove useless change

* feat(miniapp): supoort set entry when using subpackages (#532)

* refactor(miniapp): optimize entry name in runtime miniapp (#544)

* feat: support without document file (#546)

* feat: support without document file

* fix: generate html

* feat: ssr

* chore: remove useless version

* chore: change config (#548)

* chore: change config

* chore: remove unused code

* chore: bump version (#550)

* chore: bump version

* chore: bump version

* chore: update miniapp version

Co-authored-by: Hengchang Lu <44047106+luhc228@users.noreply.github.com>
Co-authored-by: NK <yxy0919@foxmail.com>
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.

3 participants