Page MenuHomePhabricator

Is MWMessagePack unused and/or worth keeping around?
Closed, ResolvedPublic0 Estimated Story Points

Event Timeline

The only use I found via grepping git log was in ProfilerMwprof (rMWc5e6ce0f9e4d: Add ProfilerMwprof), which was removed in rMWefd2fcc575a0: Remove ProfileMwprof.

I think we can follow the deprecation policy here? I can soft deprecate this class for .34

Change 505428 had a related patch set uploaded (by D3r1ck01; owner: Derick Alangi):
[mediawiki/core@master] MWMessagePack: Deprecate MWMessagePack and remove its test suit

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

Change 505428 merged by jenkins-bot:
[mediawiki/core@master] MWMessagePack: Deprecate MWMessagePack class and its method

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

xSavitar lowered the priority of this task from Medium to Low.May 9 2019, 11:04 PM
xSavitar removed a project: Patch-For-Review.

This has been deprecated! @Legoktm, I'm not sure whether to close this, so, I'll leave it for you to decide. Thanks!

xSavitar removed a project: User-xSavitar.
TheSandDoctor claimed this task.

Created sub task for its removal.