Page MenuHomePhabricator

Exclude coverage and vendor directories from linter
Closed, ResolvedPublic1 Estimated Story Points

Description

Background

Problem: The linter is linting the root-level coverage and vendor directories, which clutters the terminal with over 99,000 errors (87,000 errors in coverage and over 12,000 errors in vendor). This is causing the linter to fail, and engineers are unable to view potential issues in the code.

✖ 99873 problems (99764 errors, 109 warnings)
  97697 errors and 35 warnings potentially fixable with the `--fix` option.

Warning: Task "eslint:all" failed. Use --force to continue.

Aborted due to warnings.

Notes:

  • This change introduced the coverage directory

Requirements

  • Linter should not fail to run
  • Linter should be able to signal meaningful issues in the code
  • Linter should not complain about generated files in coverage
  • Linter should not complain about dependencies (third party libraries) in vendor

Acceptance Criteria

  • Exclude /coverage/ and /vendor/ directories found in the root of the repository from linter

Event Timeline

Change #1206493 had a related patch set uploaded (by LWatson; author: LWatson):

[mediawiki/extensions/ReaderExperiments@master] Exclude coverage and vendor from linter

https://gerrit.wikimedia.org/r/1206493

HSwan-WMF set the point value for this task to 1.Nov 18 2025, 5:10 PM

Change #1206493 merged by jenkins-bot:

[mediawiki/extensions/ReaderExperiments@master] Exclude coverage and vendor from linter

https://gerrit.wikimedia.org/r/1206493