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

test: demo -> ui test #24475

Merged
merged 12 commits into from Oct 23, 2020
Merged

test: demo -> ui test #24475

merged 12 commits into from Oct 23, 2020

Conversation

shaodahong
Copy link
Member

[中文版模板 / Chinese template]

🤔 This is a ...

  • New feature
  • Bug fix
  • Site / documentation update
  • Demo update
  • Component style update
  • TypeScript definition update
  • Bundle size optimization
  • Perfermance optimization
  • Refactoring
  • Code style optimization
  • Test Case
  • Branch merge
  • Other (about what?)

🔗 Related issue link

💡 Background and solution

📝 Changelog

Language Changelog
🇺🇸 English
🇨🇳 Chinese

☑️ Self Check before Merge

⚠️ Please check all items below before review. ⚠️

  • Doc is updated/provided or not needed
  • Demo is updated/provided or not needed
  • TypeScript definition is updated/provided or not needed
  • Changelog is provided or not needed

@ant-design-bot
Copy link
Contributor

ant-design-bot commented May 26, 2020

@ant-design-bot
Copy link
Contributor

ant-design-bot commented May 26, 2020

@afc163
Copy link
Member

afc163 commented May 26, 2020

git 里面存 png 的话,感觉会让这个仓库 git history 爆炸,现在 clone 下来已经很慢了。

@shaodahong
Copy link
Member Author

git 里面存 png 的话,感觉会让这个仓库 git history 爆炸,现在 clone 下来已经很慢了。

还可以感觉,多了大约 5M

@yoyo837
Copy link
Contributor

yoyo837 commented May 26, 2020

commit 会累积

@shaodahong
Copy link
Member Author

这倒是问题,或者使用 Git LFS

@afc163
Copy link
Member

afc163 commented May 26, 2020

能不能存 cdn,或者 base64?

@shaodahong
Copy link
Member Author

base64 倒是可以试一试

@yoyo837
Copy link
Contributor

yoyo837 commented May 27, 2020

base64比原尺寸大30%多,不过不知道和直接二进制文件对仓库历史堆积的影响有何区别。

@shaodahong
Copy link
Member Author

嗯,昨晚调研了下发现还是 Git LFS 主流的方案,不过也有问题,和 husky 不兼容……,进了连环套……

@afc163
Copy link
Member

afc163 commented Jul 29, 2020

试试能不能用 https://www.argos-ci.com/

https://www.argos-ci.com/mui-org/material-ui/builds/2768

@afc163 afc163 closed this Jul 29, 2020
@afc163 afc163 reopened this Jul 29, 2020
@shaodahong
Copy link
Member Author

试了下还行

Demo:
https://www.argos-ci.com/shaodahong/ant-design/builds/4

需要配置下,我没有仓库权限 @afc163

@07akioni
Copy link
Contributor

大家认为搞这个必要性很高吗?比如说之前出现了一些问题 snapshot 解决不了的

@afc163
Copy link
Member

afc163 commented Jul 29, 2020

@shaodahong
Copy link
Member Author

/rebase

@codesandbox-ci
Copy link

codesandbox-ci bot commented Jul 30, 2020

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit 1b4c255:

Sandbox Source
antd reproduction template Configuration

@shaodahong
Copy link
Member Author

shaodahong commented Jul 30, 2020

大家认为搞这个必要性很高吗?比如说之前出现了一些问题 snapshot 解决不了的

@07akioni 还是有必要的,但是优先级倒不是很高,咱们现在都是单元测试,虽然说是 UI 库,但是 E2E 包括 Visual Test 能测试到一些边界情况,算不上是 0 -> 100 的提升,但是 80 -> 89 的提升还是有的吧

@codecov
Copy link

codecov bot commented Jul 30, 2020

Codecov Report

Merging #24475 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##            master    #24475   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files          389       389           
  Lines         7389      7389           
  Branches      2025      2074   +49     
=========================================
  Hits          7389      7389           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 06b25b4...1b4c255. Read the comment docs.

@github-actions
Copy link
Contributor

github-actions bot commented Jul 30, 2020

Size Change: 0 B

Total Size: 791 kB

ℹ️ View Unchanged
Filename Size Change
./dist/antd-with-locales.min.js 314 kB 0 B
./dist/antd.compact.min.css 66.4 kB 0 B
./dist/antd.dark.min.css 67.7 kB 0 B
./dist/antd.min.css 66.4 kB 0 B
./dist/antd.min.js 277 kB 0 B

compressed-size-action

@afc163
Copy link
Member

afc163 commented Aug 13, 2020

https://percy.io/

@shaodahong shaodahong changed the title [WIP] test: demo -> ui test test: demo -> ui test Oct 22, 2020
@shaodahong
Copy link
Member Author

@shaodahong
Copy link
Member Author

shaodahong commented Oct 23, 2020

@afc163 @zombieJ Github check 还有点问题,其他的应该都是 OK 的,没问题可以先合一版试试

Github check 感觉和 branchName 有关系,我自己测试的仓库是没问题的

测试仓库 shaodahong/jd-happy#36

image

@afc163 afc163 merged commit 6fba4d1 into master Oct 23, 2020
@afc163 afc163 deleted the ui-test-with-demo branch October 23, 2020 10:50
@pr-triage pr-triage bot added the PR: merged label Oct 23, 2020
07akioni pushed a commit that referenced this pull request Nov 7, 2020
* test: demo -> ui test

* improve comment image width

* use argos-ci

* use by cicleci

* fix circle run

* rebase master

* try pipeline var

* use another branch var

* fix commit id

* ignore image test file

* remove useless report

* fix test coverage
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants