[wp-trac] [WordPress Trac] #28301: Improved Travis CI Performance and Configuration
WordPress Trac
noreply at wordpress.org
Sat Jun 7 09:57:00 UTC 2014
#28301: Improved Travis CI Performance and Configuration
------------------------------+--------------------------
Reporter: netweb | Owner:
Type: enhancement | Status: new
Priority: normal | Milestone: 4.0
Component: Build/Test Tools | Version: trunk
Severity: normal | Resolution:
Keywords: | Focuses: performance
------------------------------+--------------------------
Comment (by netweb):
Hehehe... Jorbin, thanks for the feedback and the following comment of
yours in ticket:26446#comment:2 was how the separate tasks came about ;)
> I would love to figure out a way to not have to run jshint for each of
the configurations since you are correct that it doesn't need to be run
across multiple build environments. I'm not sure if Travis supports that,
but I imagine we can code around that if you really think it is a show
stopper. Of course, as you point out we are only adding ~16 seconds to
the travis build.
I'm not particularly concerned which way we go in laying out the test
matrix, these can always be revisited down the road when all the PHPUnit
multisite tests start passing.
I will put the test task back in and then a couple of options based on
more feedback above:
1. For each PHP version do not include the jshint/qunit tests and use
separate jshint/qunit test (original patch)
2. For each PHP version include the jshint/qunit tests and the standalone
jshint/qunit tests
3. For each PHP version include the jshint/qunit tests without standalone
jshint/qunit tests
4. Some combination I haven't thought of
My first preference is 1, followed by 2 as the jshint and qunit tests do
currently (and should) always pass testing and feedback of a failure can
be determined in under ~2 mins (theoretically if the tests start shortly
after commit)
--
Ticket URL: <https://core.trac.wordpress.org/ticket/28301#comment:2>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list