Page MenuHomePhabricator

VisualEditor: Remove ve.bind()
Closed, ResolvedPublic

Description

Un-used function. Clean-up is good. Etc.


Version: unspecified
Severity: enhancement

Details

Reference
bz72156

Event Timeline

bzimport raised the priority of this task from to Low.
bzimport set Reference to bz72156.

<James_F> Krinkle: Thoughts on actually removing ve.bind rather than just removing its use?
<Krinkle> James_F: Not just yet. Maybe in 1-2 weeks time?
<James_F> Krinkle: For whom do you think we should wait?
<Krinkle> James_F: documentation, community, etc.
<James_F> Krinkle: Eh.
<Krinkle> James_F: We may not have a public API yet, but that doesn't mean it's cool to break stuff without at least giving them the chance to fix is.
<Krinkle> random google search: https://www.mediawiki.org/wiki/VisualEditor/Adding_instrumentation_how-to

resolved later

gerritadmin wrote:

Change 169644 had a related patch set uploaded by Alex Monk:
Remove more references to ve.bind

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

gerritadmin wrote:

Change 169644 merged by jenkins-bot:
Remove more references to ve.bind

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

gerritadmin wrote:

Change 171002 had a related patch set uploaded by Jforrester:
[BREAKING CHANGE] Remove the ve.bind function

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

gerritadmin wrote:

Change 171002 merged by jenkins-bot:
[BREAKING CHANGE] Remove the ve.bind function

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