Page MenuHomePhabricator

npm is failing with eslint on Kartographer extension
Closed, ResolvedPublic


Running "eslint:all" (eslint) task

  1:12  error  'require' is defined but never used  no-unused-vars

✖ 1 problem (1 error, 0 warnings)

In my opinion a sub-dependency must changed, which introduce the breaking change for the CI. The last merge passed without such problems.
The eslint-config must be stable when new versions for the dependency are created to avoid breaking CI.

Event Timeline

Change 488978 had a related patch set uploaded (by MSantos; owner: MSantos):
[mediawiki/extensions/Kartographer@master] Eslint have some issues with commented code

Change 488978 merged by jenkins-bot:
[mediawiki/extensions/Kartographer@master] Eslint have some issues with commented code

MSantos claimed this task.
MSantos removed a project: Patch-For-Review.