https://bugzilla.wikimedia.org/show_bug.cgi?id=63805

Krinkle <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|---                         |INVALID

--- Comment #4 from Krinkle <[email protected]> ---
I think this was created because of the jshint error reported on this change:

 https://gerrit.wikimedia.org/r/#/c/125337/

That change didn't change javascript files, so the error must've been in
mediawiki-core and thus one would assume therefore MediaWiki core master is not
passing jshint.

However that is not the case.

The change in question was submitted by a user not in the CR whitelist, and
thus the non-executive version of jshint was run instead of the one we run on
merge.

Our main version has been upgraded to 2.4.x a while back and that added an
option for es3/es5. The old version does not understand this option and thus
treated files with this option as an error.

The V+1 checkpieline's jshint has been upgraded since (did that yesterday), and
this error no longer appears.

Either way, the pipeline that ran on CR+2 never had this error and thus master
was always passing.

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are on the CC list for the bug.
_______________________________________________
Wikibugs-l mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l

Reply via email to