Page MenuHomePhabricator

eslint: remove redundant ruleset wikimedia/jsdoc
Closed, ResolvedPublic

Description

According to my testing, if the linter eslint-config-wikimedia is installed in a repo, the following rulesets already include wikimedia/jsdoc (meaning wikimedia/jsdoc doesn't need to be declared separately):

  • wikimedia/client-es5
  • wikimedia/client-es6
  • wikimedia/client (alias for wikimedia/client-es6)
  • wikimedia/server

And the following rulesets do not (meaning wikimedia/jsdoc should be declared separately):

  • wikimedia/jquery
  • wikimedia/mediawiki
  • wikimedia/mocha

It should be safe to delete the ruleset "wikimedia/jsdoc" from the first bulleted list, since those rulesets already include it.

This will be a tracking task to link in my gerrit patches to, to help explain what I'm doing.

The following patch in core was +2'd, and has some insightful comments: https://gerrit.wikimedia.org/r/c/mediawiki/core/+/1031812

Details

TitleReferenceAuthorSource BranchDest Branch
eslint: remove redundant ruleset wikimedia/jsdocrepos/abstract-wiki/wikifunctions/wikilambda-cli!39novemlinguaeeslint-remove-redundantmain
eslint: remove redundant ruleset wikimedia/jsdocrepos/mediawiki/services/ipoid!239novemlinguaeeslint-remove-redundantmain
Customize query in GitLab

Event Timeline

Change #1031815 had a related patch set uploaded (by Novem Linguae; author: Novem Linguae):

[mediawiki/extensions/Echo@master] eslint: remove redundant ruleset wikimedia/jsdoc

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

Change #1031923 had a related patch set uploaded (by Novem Linguae; author: Novem Linguae):

[mediawiki/extensions/Flow@master] eslint: remove redundant ruleset wikimedia/jsdoc

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

Change #1031925 had a related patch set uploaded (by Novem Linguae; author: Novem Linguae):

[mediawiki/extensions/Citoid@master] eslint: remove redundant ruleset wikimedia/jsdoc

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

Change #1031986 had a related patch set uploaded (by Novem Linguae; author: Novem Linguae):

[mediawiki/extensions/Graph@master] eslint: remove redundant ruleset wikimedia/jsdoc

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

Change #1031987 had a related patch set uploaded (by Novem Linguae; author: Novem Linguae):

[mediawiki/extensions/Kartographer@master] eslint: remove redundant ruleset wikimedia/jsdoc

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

Change #1031988 had a related patch set uploaded (by Novem Linguae; author: Novem Linguae):

[mediawiki/extensions/SyntaxHighlight_GeSHi@master] eslint: remove redundant ruleset wikimedia/jsdoc

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

Change #1031989 had a related patch set uploaded (by Novem Linguae; author: Novem Linguae):

[mediawiki/extensions/Thanks@master] eslint: remove redundant ruleset wikimedia/jsdoc

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

Change #1031990 had a related patch set uploaded (by Novem Linguae; author: Novem Linguae):

[mediawiki/extensions/VisualEditor@master] eslint: remove redundant ruleset wikimedia/jsdoc

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

Change #1031992 had a related patch set uploaded (by Novem Linguae; author: Novem Linguae):

[mediawiki/services/example-node-api@master] eslint: remove redundant ruleset wikimedia/jsdoc

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

Change #1031993 had a related patch set uploaded (by Novem Linguae; author: Novem Linguae):

[mediawiki/services/kartotherian@master] eslint: remove redundant ruleset wikimedia/jsdoc

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

Change #1031994 had a related patch set uploaded (by Novem Linguae; author: Novem Linguae):

[mediawiki/services/parsoid@master] eslint: remove redundant ruleset wikimedia/jsdoc

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

Change #1031995 had a related patch set uploaded (by Novem Linguae; author: Novem Linguae):

[mediawiki/services/service-scaffold-node@main] eslint: remove redundant ruleset wikimedia/jsdoc

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

Change #1031996 had a related patch set uploaded (by Novem Linguae; author: Novem Linguae):

[mediawiki/services/servicelib-node@main] eslint: remove redundant ruleset wikimedia/jsdoc

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

Change #1031997 had a related patch set uploaded (by Novem Linguae; author: Novem Linguae):

[mediawiki/services/wikifeeds@master] eslint: remove redundant ruleset wikimedia/jsdoc

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

Change #1031992 merged by jenkins-bot:

[mediawiki/services/example-node-api@master] eslint: remove redundant ruleset wikimedia/jsdoc

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

Change #1031997 merged by jenkins-bot:

[mediawiki/services/wikifeeds@master] eslint: remove redundant ruleset wikimedia/jsdoc

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

Change #1031996 merged by jenkins-bot:

[mediawiki/services/servicelib-node@main] eslint: remove redundant ruleset wikimedia/jsdoc

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

Change #1031993 merged by jenkins-bot:

[mediawiki/services/kartotherian@master] eslint: remove redundant ruleset wikimedia/jsdoc

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

Change #1031923 merged by jenkins-bot:

[mediawiki/extensions/Flow@master] eslint: remove redundant ruleset wikimedia/jsdoc

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

Change #1031815 merged by jenkins-bot:

[mediawiki/extensions/Echo@master] eslint: remove redundant ruleset wikimedia/jsdoc

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

Change #1031925 merged by jenkins-bot:

[mediawiki/extensions/Citoid@master] eslint: remove redundant ruleset wikimedia/jsdoc

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

Change #1031988 merged by jenkins-bot:

[mediawiki/extensions/SyntaxHighlight_GeSHi@master] eslint: remove redundant ruleset wikimedia/jsdoc

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

Change #1031987 merged by jenkins-bot:

[mediawiki/extensions/Kartographer@master] eslint: remove redundant ruleset wikimedia/jsdoc

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

Change #1031989 merged by jenkins-bot:

[mediawiki/extensions/Thanks@master] eslint: remove redundant ruleset wikimedia/jsdoc

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

Change #1031986 merged by jenkins-bot:

[mediawiki/extensions/Graph@master] eslint: remove redundant ruleset wikimedia/jsdoc

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

Change #1031990 merged by jenkins-bot:

[mediawiki/extensions/VisualEditor@master] eslint: remove redundant ruleset wikimedia/jsdoc

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

Change #1031994 merged by jenkins-bot:

[mediawiki/services/parsoid@master] eslint: remove redundant ruleset wikimedia/jsdoc

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

Change #1034127 had a related patch set uploaded (by Subramanya Sastry; author: Subramanya Sastry):

[mediawiki/vendor@master] Bump wikimedia/parsoid to 0.20.0-a5

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

Change #1034127 merged by jenkins-bot:

[mediawiki/vendor@master] Bump wikimedia/parsoid to 0.20.0-a5

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

Change #1031995 abandoned by Jforrester:

[mediawiki/services/service-scaffold-node@main] eslint: remove redundant ruleset wikimedia/jsdoc

Reason:

Archiving this repo; see https://phabricator.wikimedia.org/T365512

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

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