| Subject | Repo | Branch | Lines +/- | |
|---|---|---|---|---|
| 3D depends on MultimediaViewer | integration/config | master | +1 -0 | |
| [3D] Add CI | integration/config | master | +8 -0 |
Details
| Status | Subtype | Assigned | Task | ||
|---|---|---|---|---|---|
| Open | None | T44725 Multimedia file format support (tracking) | |||
| Declined | None | T107410 Wiki 3d warehouse | |||
| Open | None | T133526 Epic saga: immersive hypermedia (Myst for Wikipedia) | |||
| Resolved | TheDJ | T3790 Allow uploading of 3D files to Wikimedia Commons | |||
| Resolved | MarkTraceur | T132058 3d extension supporting STL (3d printing files) | |||
| Resolved | Jdforrester-WMF | T159243 Set up CI for the 3D extension |
Event Timeline
Change 340436 had a related patch set uploaded (by Jforrester):
[integration/config] [3D] Add CI
Change 340499 had a related patch set uploaded (by Hashar):
[integration/config] 3D depends on MultimediaViewer
It is almost done :} I took the liberty to hijack the patch that fix PHP CodeSniffer https://gerrit.wikimedia.org/r/#/c/340342/ .
There are a few issues left to figure out:
The repository has a composer.lock installed. We typically do not commit it in favor of using explicit versions in composer.json which is usually good enough. In the current situation, bumping code sniffer also requires to commit the composer.lock.
The MediaWiki structure test fails with:
- ResourceLoaderFileModule::readStyleFile: style file not found: 3D/modules/ext.3d.css
There is a modules/ext.3d.less file though but I don't know whether resource loader is smart enough to auto compile it :} Probably an easy fix.
- ResourcesTest::testMissingDependencies The module 'mmv' required by 'mmv.3d' must exist
Seems 3D requires the MultimediaViewer extension. So we need to:
a) inject the dependency in Zuul config (done with https://gerrit.wikimedia.org/r/340499 3D depends on MultimediaViewer )
b) update extension.json to mention the dependency:
"requires": { "extensions": { "MultimediaViewer": "*" } }
Speaking of what state it was when I left it, the dependency on MediaViewer is a soft one. If it's present, the 3D extension hooks into it. Otherwise it still works without MediaViewer and provides thumbnails. It's worth having it for CI, for sure, to test that integration. But I'm not sure that making it a hard dependency of the extension itself is a good idea.