Page MenuHomePhabricator

Remove max line length override
Closed, ResolvedPublic1 Estimated Story Points


Value proposition (why do we need to do this)?

As a developer, I want to not have very long lines in the codebase. They make for bad diffs, and are against our style guide.

At the moment, we're overriding the Wikimedia ESLint standard to remove the restriction on lines over 100 characters (it's not clear why, it's probably Sam's fault from before).

This ticket is to remove this override and make whatever changes are necessary to comply to the standard.

Functionality/software changes

Rewrite long lines to be short lines, without changing any semantics.

There are currently 33 long 'uns.

User interface changes


  • Does this need QA?

No, not if our automated testing is up to scratch. There is a chance that other bugs will be introduced, but it's very small.

Event Timeline

Change 464430 had a related patch set uploaded (by Mooeypoo; owner: Mooeypoo):
[mediawiki/extensions/TemplateWizard@master] Remove 'max line' eslint override and make pass

Mooeypoo set the point value for this task to 1.
Mooeypoo moved this task from Ready to Needs Review/Feedback on the Community-Tech-Sprint board.

Change 464430 merged by jenkins-bot:
[mediawiki/extensions/TemplateWizard@master] Remove 'max line' eslint override and make pass