Page MenuHomePhabricator

Consolidate svgmin config and svg / grunt-svgmin checker
Open, MediumPublic

Description

MobileFrontend, MinervaNeue, and Popups each contain approximately the same lengthy svgmin config and script or Gruntfile. This task encompasses the work to consolidate and move the config and the checker script to a new repo under wikimedia and publish a new NPM package under @wikimedia and replace the duplicate code in MobileFrontend, MinervaNeue, and Popups (don't forget to uninstall the grunt-svgmin package). The new checker should be invoked as a pre-commit hook. This task will also involve re-reading the svgmin configuration documentation to have reasonable due diligence that our configuration and usage are up to date.

See also:

Event Timeline

Restricted Application changed the subtype of this task from "Deadline" to "Task". · View Herald TranscriptSep 13 2018, 4:01 PM
Restricted Application added a subscriber: Aklapper. · View Herald Transcript
Niedzielski updated the task description. (Show Details)Sep 13 2018, 4:16 PM
Niedzielski added subscribers: Volker_E, Jdlrobson.
Jdlrobson added a comment.EditedSep 18 2018, 11:50 PM

How does this relate to the work @Volker_E did (and T178867)?

I'm keen to hear if there is a standard way to do this.^

Hopefully we can discuss this in a future frontends session..

Restricted Application added a subscriber: Masumrezarock100. · View Herald TranscriptDec 11 2019, 12:44 AM
Jdlrobson moved this task from Backlog to Tech debt on the MinervaNeue board.Jan 17 2020, 10:22 PM
Jdlrobson renamed this task from Consolidate svgmin config and checker to Consolidate svgmin config and svg / grunt-svgmin checker.Mar 3 2020, 6:58 PM
Jdlrobson added subscribers: Legoktm, Jdforrester-WMF.