Page MenuHomePhabricator

Update SVGO to v2.x including new JS based configuration
Closed, ResolvedPublic2 Estimated Story Points

Description

SVGO has been updated to new major version 2.x after some time without any updates.

It needs to be switched to JS configuration.
First exemplary configuration is in Vector.

Production uses:

  • Wikimedia Design Style Guide

Details

SubjectRepoBranchLines +/-
mediawiki/coremaster+160 -388
mediawiki/skins/VectorREL1_36+2 K -2 K
mediawiki/extensions/UniversalLanguageSelectormaster+818 -1 K
mediawiki/skins/Timelessmaster+366 -387
mediawiki/extensions/CentralNoticemaster+831 -1 K
mediawiki/extensions/Kartographermaster+204 -328
mediawiki/extensions/Echomaster+1 K -2 K
mediawiki/skins/MonoBookmaster+224 -496
mediawiki/extensions/TwoColConflictmaster+223 -231
mediawiki/extensions/RevisionSlidermaster+171 -422
mediawiki/extensions/Popupsmaster+23 K -22 K
mediawiki/extensions/GuidedTourmaster+211 -345
mediawiki/extensions/MobileFrontendmaster+1 K -1 K
mediawiki/skins/MinervaNeuemaster+234 -128
mediawiki/skins/Vectormaster+2 K -2 K
mediawiki/extensions/MultimediaViewermaster+7 K -431
Show related patches Customize query in gerrit

Event Timeline

There are a very large number of changes, so older changes are hidden. Show Older Changes

Change 678403 had a related patch set uploaded (by VolkerE; author: VolkerE):

[mediawiki/extensions/MobileFrontend@master] build: Updgrade to latest 'svgo' and re-crush SVGs

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

Change 678635 had a related patch set uploaded (by VolkerE; author: VolkerE):

[mediawiki/extensions/MultimediaViewer@master] build: Replace 'grunt-svgmin' with npm based 'svgo' & re-crush SVGs

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

Change 678637 had a related patch set uploaded (by VolkerE; author: VolkerE):

[mediawiki/skins/MinervaNeue@master] build: Updgrade to latest 'svgo' and re-crush SVGs

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

Change 678635 merged by jenkins-bot:

[mediawiki/extensions/MultimediaViewer@master] build: Replace 'grunt-svgmin' with npm based 'svgo' & re-crush SVGs

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

Change 678639 had a related patch set uploaded (by VolkerE; author: VolkerE):

[mediawiki/skins/Vector@master] build: Update 'svgo' to latest

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

Volker_E added a subscriber: Jdforrester-WMF.

@Jdforrester-WMF I guess with your expansion, you're in agreement to move all those products over to npm 'svgo' instead of 'grunt-svgmin'? :)

@Jdforrester-WMF I guess with your expansion, you're in agreement to move all those products over to npm 'svgo' instead of 'grunt-svgmin'? :)

I've given up expecting people to actually help out with finding and making everything that expects Grunt to instead work with npm scripts, so I guess I'll just have to cope and fix them all myself.

ovasileva triaged this task as Medium priority.Apr 15 2021, 5:10 PM

Change 678639 merged by jenkins-bot:

[mediawiki/skins/Vector@master] build: Update 'svgo' to latest

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

LGoto set the point value for this task to 2.Apr 20 2021, 5:45 PM

Change 678637 merged by jenkins-bot:

[mediawiki/skins/MinervaNeue@master] build: Upgrade 'svgo' to latest and re-crush SVGs

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

Change 678403 merged by jenkins-bot:

[mediawiki/extensions/MobileFrontend@master] build: Upgrade to latest 'svgo' and re-crush SVGs

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

Change 681760 had a related patch set uploaded (by Jdrewniak; author: Jdrewniak):

[mediawiki/extensions/Popups@master] Build: Replace 'grunt-svgmin' with npm based 'svgo' & re-crush SVGs

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

Change 681944 had a related patch set uploaded (by Jdrewniak; author: Jdrewniak):

[mediawiki/extensions/GuidedTour@master] Replace grunt-svgmin with SVGO v2.3.0 and run through npm script.

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

Change 681944 merged by jenkins-bot:

[mediawiki/extensions/GuidedTour@master] Replace grunt-svgmin with SVGO v2.3.0 and run through npm script.

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

Change 681760 merged by jenkins-bot:

[mediawiki/extensions/Popups@master] build: Upgrade 'svgo' & re-crush SVGs

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

Change 692446 had a related patch set uploaded (by VolkerE; author: VolkerE):

[mediawiki/extensions/Kartographer@master] build: Update SVGO to latest v2.3.0 and re-crush SVGs

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

Change 692450 had a related patch set uploaded (by VolkerE; author: VolkerE):

[mediawiki/extensions/Echo@master] build: Update SVGO to latest v2.3.0 and re-crush SVGs

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

Change 692455 had a related patch set uploaded (by VolkerE; author: VolkerE):

[mediawiki/extensions/CentralNotice@master] build: Update SVGO to latest v2.3.0 and re-crush SVGs

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

Change 692458 had a related patch set uploaded (by VolkerE; author: VolkerE):

[mediawiki/extensions/RevisionSlider@master] build: Update SVGO to latest v2.3.0 and re-crush SVGs

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

Change 692458 merged by jenkins-bot:

[mediawiki/extensions/RevisionSlider@master] build: Update SVGO to latest v2.3.0 and re-crush SVGs

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

Change 692580 had a related patch set uploaded (by VolkerE; author: VolkerE):

[mediawiki/extensions/TwoColConflict@master] build: Update SVGO to latest v2.3.0 and re-crush SVGs

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

Change 692580 merged by jenkins-bot:

[mediawiki/extensions/TwoColConflict@master] build: Update SVGO to latest v2.3.0 and re-crush SVGs

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

Change 692584 had a related patch set uploaded (by VolkerE; author: VolkerE):

[mediawiki/extensions/UniversalLanguageSelector@master] build: Update SVGO to latest v2.3.0 and re-crush SVGs

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

Change 692591 had a related patch set uploaded (by VolkerE; author: VolkerE):

[mediawiki/core@master] build: Update SVGO to latest v2.3.0 and re-crush SVGs

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

Change 692595 had a related patch set uploaded (by VolkerE; author: VolkerE):

[mediawiki/skins/Timeless@master] build: Update SVGO to latest v2.3.0 and re-crush SVGs

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

Change 692596 had a related patch set uploaded (by VolkerE; author: VolkerE):

[mediawiki/skins/MonoBook@master] build: Update SVGO to latest v2.3.0 and re-crush SVGs

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

Volker_E updated the task description. (Show Details)
Volker_E added a subscriber: Jdrewniak.

Change 692591 merged by jenkins-bot:

[mediawiki/core@master] build: Update SVGO to latest v2.3.0 and re-crush SVGs

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

Change 692450 merged by jenkins-bot:

[mediawiki/extensions/Echo@master] build: Update SVGO to latest v2.3.0 and re-crush SVGs

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

Change 692596 merged by jenkins-bot:

[mediawiki/skins/MonoBook@master] build: Update SVGO to latest v2.3.0 and re-crush SVGs

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

Change 692446 merged by jenkins-bot:

[mediawiki/extensions/Kartographer@master] build: Update SVGO to latest v2.3.0 and re-crush SVGs

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

Change 692455 merged by jenkins-bot:

[mediawiki/extensions/CentralNotice@master] build: Update SVGO to latest v2.3.0 and re-crush SVGs

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

Change 692584 merged by jenkins-bot:

[mediawiki/extensions/UniversalLanguageSelector@master] build: Update SVGO to latest v2.3.0 and re-optimize SVGs

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

Change 692595 merged by jenkins-bot:

[mediawiki/skins/Timeless@master] build: Update SVGO to latest v2.3.0 and re-crush SVGs

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

Change 700629 had a related patch set uploaded (by Voidwalker; author: VolkerE):

[mediawiki/skins/Vector@REL1_36] build: Update 'svgo' to latest

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

Change 700629 merged by jenkins-bot:

[mediawiki/skins/Vector@REL1_36] build: Update 'svgo' to latest

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

@TheDJ Design Style Guide was still missing. Tackled in https://github.com/wikimedia/WikimediaUI-Style-Guide/pull/495 together with new SVGO v2.8 config.

Volker_E added a subscriber: Ladsgroup.

DSG PR gladly merged with help of @Ladsgroup!