https://bugzilla.wikimedia.org/show_bug.cgi?id=46066
Krinkle <[email protected]> changed: What |Removed |Added ---------------------------------------------------------------------------- Priority|Unprioritized |Lowest Severity|normal |enhancement --- Comment #1 from Krinkle <[email protected]> --- This is a general problem with how Zuul and Jenkins don't work well together. They're based on different concepts. However you should just ignore those relative results and look at the absolute results instead. In JSHint there are two categories: * Warnings enabled by default we disabled and warnings disabled by default that we didn't enable ("normal") * Warnings we have enabled ("high") Most reporters (including the command-line api, text-editor api and jshint.com interface) only report the high priority errors. Although we include everything in the checkstyleResult[1], only the high priority warnings are counted towards marking the job SUCCESS or FAILURE. MediaWiki core has 0 failures in master, so any failure is a new failure. So when you open the report you should look at the tab "High priority"[2]. Previously we outputted them in raw text (the command-line api) which only showed the high priority errors and did not do any kind of comparison. Now that we're displaying them visually and organised with Checkstyle, we also show ignored warnings in the "normal" category[1]. And it also does comparison against the previous run. This last feature (comparison) is useless in our case, but it can't be disabled unfortunately. So you're best just ignoring it. Nobody asked for that feature, it just came along with visualising the data with Checkstyle, so changing this to an "enhancement". It is confusing, but since nobody wanted this feature I'm not going to prioritise fixing it. [1] We include ignored errors in a separate category so that they aren't displayed mixed between the others. We don't hide them completely so that they're still available for those interested in seeing what other settings we can potentially enable again in the future (e.g. if you see 0 errors for "use single quotes" in the ignored category, it means it is safe to enable that setting). [2] https://integration.mediawiki.org/ci/job/mediawiki-core-jslint/3455/checkstyleResult/HIGH/?#details -- You are receiving this mail because: You are on the CC list for the bug. You are the assignee for the bug. You are watching all bug changes. _______________________________________________ Wikibugs-l mailing list [email protected] https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
