Page MenuHomePhabricator
Create Task
Maniphest T220313

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

Description

https://codesearch.wmflabs.org/search/?q=MWMessagePack&i=nope&files=&repos=

Didn't see any uses of it...

Details

ProjectBranchLines +/-Subject
mediawiki/coremaster+7 -1MWMessagePack: Deprecate MWMessagePack class and its method
Customize query in gerrit

Related Objects
Search...

Event Timeline

Legoktm created this task.Apr 7 2019, 11:04 PM
Restricted Application added a subscriber: Aklapper. · View Herald TranscriptApr 7 2019, 11:04 PM
Legoktm added a comment.Apr 7 2019, 11:14 PM

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

xSavitar added a subscriber: xSavitar.Apr 20 2019, 9:51 PM

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

gerritbot added a comment.Apr 20 2019, 9:56 PM

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

gerritbot added a project: Patch-For-Review.Apr 20 2019, 9:56 PM
xSavitar added a project: User-xSavitar.Apr 20 2019, 9:57 PM
xSavitar moved this task from Backlog to Doing [WIP] on the User-xSavitar board.
xSavitar claimed this task.Apr 22 2019, 11:56 AM
xSavitar moved this task from Doing [WIP] to Under Review on the User-xSavitar board.
xSavitar moved this task from Under Review to Reviewed/Resolved on the User-xSavitar board.Apr 27 2019, 8:36 PM
xSavitar moved this task from Reviewed/Resolved to Under Review on the User-xSavitar board.May 9 2019, 9:58 PM
Krinkle triaged this task as Medium priority.May 9 2019, 10:31 PM
Krinkle edited projects, added Technical-Debt (Deprecation process); removed Technical-Debt.
Krinkle moved this task from Untriaged to 2019 | MW 1.33 / 1.34 on the Technical-Debt (Deprecation process) board.
gerritbot added a comment.May 9 2019, 10:33 PM

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

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

ReleaseTaggerBot added a project: MW-1.34-notes (1.34.0-wmf.5; 2019-05-14).May 9 2019, 11:01 PM
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 xSavitar as the assignee of this task.May 18 2019, 2:58 PM
xSavitar removed a project: User-xSavitar.
TheSandDoctor added a subscriber: TheSandDoctor.Dec 20 2019, 10:02 AM
TheSandDoctor closed this task as Resolved.Dec 20 2019, 10:04 AM
TheSandDoctor claimed this task.

Created sub task for its removal.

TheSandDoctor reassigned this task from TheSandDoctor to xSavitar.Dec 20 2019, 10:05 AM

Fix assignee.

Restricted Application added a project: User-xSavitar. · View Herald TranscriptDec 20 2019, 10:05 AM
TheSandDoctor changed the status of subtask T241220: Remove MWMessagePack from Open to Stalled.Dec 21 2019, 4:49 AM
TheSandDoctor changed the status of subtask T241220: Remove MWMessagePack from Stalled to Open.Dec 30 2019, 5:12 AM
TheSandDoctor closed subtask T241220: Remove MWMessagePack as Resolved.Jan 1 2020, 11:29 PM
Content licensed under Creative Commons Attribution-ShareAlike 3.0 (CC-BY-SA) unless otherwise noted; code licensed under GNU General Public License (GPL) or other open source licenses. By using this site, you agree to the Terms of Use, Privacy Policy, and Code of Conduct. · Wikimedia Foundation · Privacy Policy · Code of Conduct · Terms of Use · Disclaimer · CC-BY-SA · GPL