Page MenuHomePhabricator

Clean up services in Parsoid extension
Open, MediumPublic

Description

@Anomie pointed out a bunch of seemingly-valid issues w/ the service wiring of the parsoid extension in comments on https://gerrit.wikimedia.org/r/518682 PS6 ... but @Tgr had timed-out on his tour-of-duty in Parsoid land, and it's not clear that any of the core Parsoidistas really understands service wiring well enough to address @Anomie's comments.

The patch got merged in order to get initial tests running. But we should clean up the service wiring at some point.

Event Timeline

Sorry about that. I have some half-written fixes and do intend to clean stuff up eventually (if no one beats me to it) but I'm on a very tight schedule, at least until the end of August.

No worries. I just wanted to stick it in phab so we don't forget about it. I suspect it can safely wait until September.

ssastry triaged this task as Medium priority.Dec 8 2019, 3:29 AM