Page MenuHomePhabricator

tools/sync-parserTests.js expects to be run on master
Closed, ResolvedPublic

Description

As seen on T413018: Parsoid tests in REL1_XX branches need updating for T407131, there is sometimes reason to run this on REL1_XX branches.

I note, in practice, the script seems to only git checkout master in case of error, and suggests console.log(' git rebase --keep-empty origin/master'); as a command to run...

But it should be possible to look at the currently checked out branch (or use the branch parameter to the script run?), and use that for the output.

Event Timeline

Reedy triaged this task as Low priority.Dec 26 2025, 9:51 PM

Hmm, I thought that I had done this already but it looks like we only ever merged the second half in https://gerrit.wikimedia.org/r/c/mediawiki/services/parsoid/+/1022160

Change #1224184 had a related patch set uploaded (by Arlolra; author: Arlolra):

[mediawiki/services/parsoid@master] Optionally pass a base to sync-parserTests.js

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

Change #1224184 merged by jenkins-bot:

[mediawiki/services/parsoid@master] Optionally pass a base to sync-parserTests.js

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

Change #1224807 had a related patch set uploaded (by Reedy; author: Arlolra):

[mediawiki/services/parsoid@REL1_45] Optionally pass a base to sync-parserTests.js

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

Change #1224808 had a related patch set uploaded (by Reedy; author: Arlolra):

[mediawiki/services/parsoid@REL1_44] Optionally pass a base to sync-parserTests.js

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

Change #1224809 had a related patch set uploaded (by Reedy; author: Arlolra):

[mediawiki/services/parsoid@REL1_43] Optionally pass a base to sync-parserTests.js

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

Change #1224808 merged by jenkins-bot:

[mediawiki/services/parsoid@REL1_44] Optionally pass a base to sync-parserTests.js

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

Change #1224809 merged by jenkins-bot:

[mediawiki/services/parsoid@REL1_43] Optionally pass a base to sync-parserTests.js

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

Change #1224807 merged by jenkins-bot:

[mediawiki/services/parsoid@REL1_45] Optionally pass a base to sync-parserTests.js

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

Change #1228553 had a related patch set uploaded (by Arlolra; author: Arlolra):

[mediawiki/vendor@master] Bump wikimedia/parsoid to 0.23.0-a12

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

Change #1228553 merged by jenkins-bot:

[mediawiki/vendor@master] Bump wikimedia/parsoid to 0.23.0-a12

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