[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