Page MenuHomePhabricator

Get libup to work for parsoid
Closed, ResolvedPublic

Description

Maybe not for master (though, why not?), but certainly for supported release branches... Any chance we can run LibUp against parsoid. Due to issues like T295797: Fix mwgate-node12-docker failures in Parsoid branches

Not necessarily urgent we make releases afterwards, but having the release branches of parsoid not fail when someone tries to make further changes would be awesome

Event Timeline

It's running...or at least trying and failing: https://libraryupgrader2.wmcloud.org/r/mediawiki/services/parsoid?branch=main

IIRC it used to fail because of shrinkwrap but that's no longer an issue. The current failure is because https://gerrit.wikimedia.org/r/plugins/gitiles/mediawiki/services/parsoid/+/refs/heads/master/.eslintrc has comments in it.

Legoktm renamed this task from Can we run libup for parsoid? to Get libup to work for parsoid.Nov 16 2021, 6:34 PM

Change 739322 had a related patch set uploaded (by Zabe; author: Zabe):

[mediawiki/services/parsoid@master] Move .eslintrc to .eslintrc.json

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

Change 739322 merged by jenkins-bot:

[mediawiki/services/parsoid@master] Move .eslintrc to .eslintrc.json

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

Arlolra claimed this task.

Do we need to "backport" this to the release branches so libup is happy?

Probably, might also be worth waiting a day for the next libup run to see if anything else fails.

Arlolra moved this task from Needs Triage to Testing on the Parsoid board.

Sorry, did I close this prematurely

Arlolra subscribed.

Change 740248 had a related patch set uploaded (by Zabe; author: Zabe):

[mediawiki/services/parsoid@REL1_37] Move .eslintrc to .eslintrc.json

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

Change 740869 had a related patch set uploaded (by Zabe; author: Zabe):

[mediawiki/services/parsoid@REL1_36] Move .eslintrc to .eslintrc.json

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

Change 740870 had a related patch set uploaded (by Zabe; author: Zabe):

[mediawiki/services/parsoid@REL1_35] Move .eslintrc to .eslintrc.json

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

Change 742570 had a related patch set uploaded (by Sbailey; author: Sbailey):

[mediawiki/vendor@master] Bump Parsoid to 0.15.0-a11

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

Change 742570 merged by jenkins-bot:

[mediawiki/vendor@master] Bump Parsoid to 0.15.0-a11

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

Change 740870 merged by jenkins-bot:

[mediawiki/services/parsoid@REL1_35] Move .eslintrc to .eslintrc.json

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

Change 740869 merged by jenkins-bot:

[mediawiki/services/parsoid@REL1_36] Move .eslintrc to .eslintrc.json

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

Change 740248 merged by jenkins-bot:

[mediawiki/services/parsoid@REL1_37] Move .eslintrc to .eslintrc.json

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

Change 823249 had a related patch set uploaded (by Arlolra; author: Zabe):

[mediawiki/services/parsoid@REL1_37] Move .eslintrc to .eslintrc.json

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

Change 823249 merged by jenkins-bot:

[mediawiki/services/parsoid@REL1_37] Move .eslintrc to .eslintrc.json

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