Page MenuHomePhabricator

Deploy new Linter category (tidy-whitespace-bug)
Closed, ResolvedPublic

Description

Tracking ticket for deployment so interested folks can monitor progress.

Associated patches:
Parsoid: https://gerrit.wikimedia.org/r/#/c/371068/
Linter extension: https://gerrit.wikimedia.org/r/#/c/371071/

Event Timeline

ssastry triaged this task as High priority.Aug 11 2017, 2:26 PM

Tidy whitespace bug (4,379 errors) appears at Special:Linterrors on en.WP. Is this done, @ssastry?

Tidy whitespace bug (4,379 errors) appears at Special:Linterrors on en.WP. Is this done, @ssastry?

Not quite yet. We are refining the output to make it more useful for editors. See https://gerrit.wikimedia.org/r/#/c/372896/ .. But, we expect to have it deployed this week.

@IKhitron has suggested this for inclusion in Tech News. How would you phrase it in a couple of simple sentences?

@IKhitron has suggested this for inclusion in Tech News. How would you phrase it in a couple of simple sentences?

Actually, for now the things changed and I'm not sure either it should be in this issue or the next one.

@IKhitron has suggested this for inclusion in Tech News. How would you phrase it in a couple of simple sentences?

Let us wait a week before we announce this. We pushed out a fix today that improves the reporting in this category. @Legoktm is going to start a bot run soon to re-generate linter errors for all pages on all wikis (which addresses complaints of error information being incomplete). We might instead choose to announce an update about this next week that includes the new category.

The new changes to the category output have been deployed.

@ssastry Thanks. How would you, simplified, put this in a couple of sentences?

@ssastry Thanks. How would you, simplified, put this in a couple of sentences?

Linter added a new high priority category: "tidy-whitespace-bug". This usually affects templates with lists that display horizontally.

How about that?