Page MenuHomePhabricator

Eslint: configure browser compatibility
Closed, ResolvedPublic

Description

We are constraining the client module browser compatibility in modules using Vue (3) since the framework does not support IE11 and other older browsers. It would be nice to configure eslint compatibility rules (compat/compat) accordingly. This way we can avoid having to manually override use cases with // eslint-disable-next-line compat/compat

Also making ES6 modules identifiable via URL structure per comments in patch.

Event Timeline

Change #1125987 had a related patch set uploaded (by Michael Große; author: Michael Große):

[mediawiki/extensions/GrowthExperiments@master] chore(eslint): make compat rule into errors again

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

Michael subscribed.

The upstream browser compatibility has been updated a while ago to no longer include IE 11. So this just needed some minor cleanup and then it can be closed. Raising the compatibility threshold further is being discussed in T381537: Raise Grade A JavaScript requirement from ES2016 (ES7) to ES2017 (ES8).

Change #1125987 merged by jenkins-bot:

[mediawiki/extensions/GrowthExperiments@master] chore(eslint): make compat rule into errors again

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

This did not touch any production files at all, only a minor update to our eslint config. Moving to QA purely for informational purposes.