Page MenuHomePhabricator

Termbox: Use "vuex" ResourceLoader module from core
Closed, ResolvedPublic3 Estimated Story Points

Description

Acceptance criteria:

  • Termbox uses vuex copy provided by mediawiki
  • There is automated check in place that the version specified in termbox's package.json and the version provided by mediawiki are the same.
  • The new termbox version is pinned in the wikibase repo

Note: Wikidata Bridge has done externalization of this dependency, could be looked up for inspiration. Termbox has also already externalized vuejs dependency.

Event Timeline

Don't forget to make sure that T259468 remains resolved. i.e. check we don't end up with a vuex version drift between core and package.json

Change 682618 had a related patch set uploaded (by Tonina Zhelyazkova; author: Tonina Zhelyazkova):

[wikibase/termbox@master] Externalize vuex

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

Note to the reviewer(s):
The patch contains changes to package-lock.json which were not expected given the nature of the change, but they were generated after an npm install (executed inside the docker container) so I decided to submit them.

Change 682618 merged by jenkins-bot:

[wikibase/termbox@master] Externalize vuex

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

Change 683257 had a related patch set uploaded (by Tonina Zhelyazkova; author: Tonina Zhelyazkova):

[mediawiki/extensions/Wikibase@master] Update termbox and add vuex dependency

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

Change 683257 merged by jenkins-bot:

[mediawiki/extensions/Wikibase@master] Update termbox and add vuex dependency

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