Change Details

While working on https://gerrit.wikimedia.org/r/c/591436 (Bug: T250851) we noticed that skinScripts do not get loaded in packageFile modules. This will block work in building out the search widget in Vue.js so it's of high importance that we get this unblocked. In the meantime we could revert the use of packageFiles in this module or split out a new module however we'd like to avoid this if at all possible.

While working on https://gerrit.wikimedia.org/r/c/591436 (Bug: T250851) we noticed that skinScripts do not get loaded in packageFile modules. This will block work in building out the search widget in Vue.js so it's of high importance that we get this unblocked. In the meantime we could revert the use of packageFiles in this module or split out a new module however we'd like to avoid this if at all possible. = Acceptance criteria [] Setting skinScripts and packageFiles on the same module should thrown an exception. = Developer notes After talking to @Catrope we think that virtual packageFiles using hooks could solve the needs of skins wants to change behaviour. With this in mind we should thrown an exception of skinScripts is mixed with packageFiles.

While working on https://gerrit.wikimedia.org/r/c/591436 (Bug: T250851) we noticed that skinScripts do not get loaded in packageFile modules. This will block work in building out the search widget in Vue.js so it's of high importance that we get this unblocked. In the meantime we could revert the use of packageFiles in this module or split out a new module however we'd like to avoid this if at all possible. = Acceptance criteria [] Setting skinScripts and packageFiles on the same module should thrown an exception. = Developer notes After talking to @Catrope we think that virtual packageFiles using hooks could solve the needs of skins wants to change behaviour. With this in mind we should thrown an exception of skinScripts is mixed with packageFiles.