Speed up travis build #9403
Speed up travis build #9403
Conversation
|
@rsimha-amp you can revert the last commit, and it should pass the test: was flaky due to #9325 |
|
@lannka: Nice! Looks like the overall build time has been halved. I've reverted the last commit, and this is now ready for review. |
| timedExecOrDie(`${gulp} clean`); | ||
| timedExecOrDie(`${gulp} lint`); | ||
| }, | ||
| buildRuntime: function() { | ||
| timedExecOrDie(`${gulp} clean`); | ||
| timedExecOrDie(`${gulp} build`); |
There was a problem hiding this comment.
theoretically, we don't need gulp build. gulp build --css-only should be enough.
i know it might fail some tests right now, can we add a TODO to investigate later?
There was a problem hiding this comment.
This function (buildRuntime) is called before the unit tests and the integration tests. I've already added a TODO to investigate why the unit tests currently need this function to be called. So this is done, I believe.
There was a problem hiding this comment.
that's fine. I think integration should not need that either.
This PR attempts to speed up the travis build for PRs and pushes to master by sharding the various build and test tasks into separate, independent sets of tasks that are run in parallel.
Fixes #9387