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

Restructure CI workflow #12002

Merged
merged 5 commits into from Aug 26, 2020
Merged

Restructure CI workflow #12002

merged 5 commits into from Aug 26, 2020

Conversation

@JLHwung
Copy link
Contributor

@JLHwung JLHwung commented Aug 24, 2020

Q                       A
License MIT

This PR restructured current CI workflow and move linux node unit tests to GitHub workflows.

In GitHub workflows we shared built babel artifacts across multiple unit test executors, thus we can reduce the overall build time in our CI. The new CI now always build Babel on latest node.js versions. If node.js latest incurs any regression, we can always pin the node version of the build step.

A comparison of CI time between main and this PR.

Item main This PR CI Speed Improvments
Travis CI 13 min 21 sec 8 min 40 sec 54%
GitHub CI 8 min 20 sec 10 min 9 sec -22%
max(Travis CI, GitHub CI) 13 min 21 sec 10 min 9 sec 30%
Data Source

main:
Travis CI: https://travis-ci.com/github/babel/babel/builds/181136937
GitHub CI: https://github.com/babel/babel/actions/runs/222502540

This PR:
Travis CI: https://travis-ci.com/github/JLHwung/babel/builds/181140151
GitHub CI: https://github.com/JLHwung/babel/actions/runs/222540002

This PR includes commits from #11781 , I will rebase later. You can review the actual changes on https://github.com/babel/babel/pull/12002/files/b9904078208917d338292ccad5ec544e7e53ae44..38d0ea99f83d5ec07b94a3bd0e2ddc2647c0bfdd

@babel-bot
Copy link
Collaborator

@babel-bot babel-bot commented Aug 24, 2020

Build successful! You can test your changes in the REPL here: https://babeljs.io/repl/build/27934/

@codesandbox
Copy link

@codesandbox codesandbox bot commented Aug 24, 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 f9c82f9:

Sandbox Source
babel-repl-custom-plugin Configuration
babel-plugin-multi-config Configuration
@JLHwung JLHwung force-pushed the JLHwung:ci-orchestration branch 3 times, most recently from e0ea750 to 303ec4a Aug 25, 2020
run: |
BABEL_ENV=test-legacy make -j bootstrap
make build-plugin-transform-runtime-dist

This comment has been minimized.

@JLHwung

JLHwung Aug 25, 2020
Author Contributor

This step is moved to test executor (instead of the Build job) because it is IO-intensive (1000 files), caching and restoring are slower than generating.

@JLHwung JLHwung force-pushed the JLHwung:ci-orchestration branch from 303ec4a to 38d0ea9 Aug 25, 2020
.travis.yml Outdated Show resolved Hide resolved
.travis.yml Outdated Show resolved Hide resolved
@JLHwung JLHwung force-pushed the JLHwung:ci-orchestration branch from dfa8399 to f9c82f9 Aug 26, 2020
@JLHwung JLHwung merged commit 97075c9 into babel:main Aug 26, 2020
15 of 16 checks passed
15 of 16 checks passed
Prepare Cache
Details
Test on Node.js Latest
Details
Build Babel Artifacts
Details
Test on Node.js (13)
Details
Test on Node.js (12)
Details
Test on Node.js (10)
Details
Test on Node.js (8)
Details
Test on Node.js (6)
Details
test262-pr Workflow: test262-pr
Details
Gitpod Open an online workspace in Gitpod
Details
Travis CI - Pull Request Build Passed
Details
babel/repl REPL preview is available
Details
build-standalone Workflow: build-standalone
Details
ci/codesandbox Building packages succeeded.
Details
codecov/project 91.69% (target 90.00%)
Details
e2e Workflow: e2e
Details
@JLHwung JLHwung deleted the JLHwung:ci-orchestration branch Aug 26, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

4 participants
You can’t perform that action at this time.