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.

Sign up for GitHub

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

Jump to bottom

add test for umd dist bundle #1496

Closed
paranoidjk opened this issue Jun 28, 2017 · 6 comments
Closed

add test for umd dist bundle #1496

paranoidjk opened this issue Jun 28, 2017 · 6 comments
Assignees
@yesmeck

Comments

@paranoidjk
Copy link
Contributor

paranoidjk commented Jun 28, 2017 •
edited
Loading

Right now our test code both import from src, we can not guarantee the build output is ok.
@paranoidjk paranoidjk self-assigned this Jun 28, 2017
@paranoidjk
Copy link
Contributor Author

cc @yesmeck

@yesmeck
Copy link
Member

yesmeck commented Jun 29, 2017

我来搞下好了，antd 也搞一下。

@paranoidjk paranoidjk assigned yesmeck and unassigned paranoidjk Jun 29, 2017
yesmeck referenced this issue in ant-design/ant-design Jun 30, 2017
@yesmeck
Run demo test against dist/lib/es dir
1f69de8
@warmhug
Copy link
Contributor

warmhug commented Jun 30, 2017

还需要加入 bundle 分析插件 webpack-visualizer-plugin ，这个能显示各个 module 体积和其所占总百分比的数据。

@paranoidjk
Copy link
Contributor Author

facebook/react#10090

@warmhug
Copy link
Contributor

warmhug commented Jul 3, 2017

ref #1274

@paranoidjk paranoidjk added the Test label Jul 3, 2017
yesmeck referenced this issue Jul 3, 2017
@yesmeck
Run tests against dir/lib/es builds
eb5f79e
warmhug added a commit that referenced this issue Jul 5, 2017
@warmhug
add analysis, ref #1274 #1496
566b2e1
@paranoidjk
Copy link
Contributor Author

https://github.com/ai/size-limit

lixiaoyang1992 pushed a commit to lixiaoyang1992/ant-design-mobile that referenced this issue Apr 26, 2018
@warmhug @lixiaoyang1992
add analysis, ref ant-design#1274 ant-design#1496
c93b3fe
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@yesmeck yesmeck

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@yesmeck @warmhug @paranoidjk