Those day I spend some time updating Travis CI of incubator-weex, during
the development process, I found some problems and the below is my
suggestion,

# FIrst I recommend prohibiting merging the PR that Travis CI builds failed

Travis CI build failed means that there is something wrong. If we force to
merge the PR, it will lead to bugs even crash. We should Prohibited force
merge PR that Travis CI builds failed into the main branch. (but the
committer now has permission to merge the PRs that failed Travis CI build
failed, and there have been some cases where PR builds failed but were
merged into the main branch.)

# Second, I recommend increasing Travis CI's resources

Now Weex's Travis CI does not allow parallel builds, which means that new
Travis CI job must wait until the existing Travis CI job complete.

But It takes about 20 minutes to build a Travis CI once now, with more and
more checks will be added to Travis CI, the wait time will get longer and
longer, even unbearable.

Best regards.
Renmin Wang

Reply via email to