Page MenuHomePhabricator

LinkedWiki relies on npm for javascript dependencies
Open, Needs TriagePublic

Description

MediaWiki core structure tests fail for the LinkedWiki extension:

1) Warning
The data provider specified for ResourcesTest::testFileExistence is invalid.
ResourceLoaderFileModule::readStyleFile: style file not found: "extensions/LinkedWiki/node_modules/bootstrap/dist/css/bootstrap.min.css"

That is because bootstrap is supposed to be installed via npm install, a process we do not quite support:

{
	"dependencies": {
		"bootstrap": "^4.0.0-beta.3",
		"font-awesome": "^4.7.0",
		"popper.js": "^1.12.9",
		"sgvizler2": "^1.0.9"
	}
}

Other repositories having dependencies in their package.json are Athena, Kartographer, ImageTweaks. They have the node_modules committed in.

Event Timeline

hashar created this task.Jul 5 2018, 9:32 PM
Restricted Application added a subscriber: Aklapper. · View Herald TranscriptJul 5 2018, 9:32 PM
hashar renamed this task from LinkedWiki: style file not found: extensions/LinkedWiki/node_modules/bootstrap/dist/css/bootstrap.min.css to LinkedWiki relies on npm for javascript dependencies.Jul 11 2018, 4:39 PM
hashar updated the task description. (Show Details)
hashar added a subscriber: Jdlrobson.

@Jdlrobson LinkedWiki expects some JavaScript packages to be installed via npm install. I think that is related to your RFC T199004: RFC: Add a frontend build step to skins/extensions to our deploy process.

Definitely related. LinkedWiki is currently forced to check in the asset after npm install, which as I argue in T199004 is a little unreasonable to expect when the build can be replicated easily by a bot.

Change 454300 had a related patch set uploaded (by Hashar; owner: Hashar):
[integration/config@master] LinkedWiki extension is broken

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

Change 454300 merged by jenkins-bot:
[integration/config@master] LinkedWiki extension is broken

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