These ignores are already in package.json. Placing these here was
making `npm test` wait 10-20 seconds doing nothing waiting for Grunt
to synchronously recursively glob and expand an array of every file
in the current working directory (incl node_modules, and vendor) and
then once more for node_modules and vendor, and then to filter one
out from the other. And all before eslint even starts.
Instead, give '.' to eslint as we do in some other repos already,
and let it iterate on its own, skipping sub directories as it goes
based on package.json/eslintIgnore.
This follows-up 445087e5f, which did this once before,
and be400fe75 which unintentionally brought the Gruntfile part
back while leading the other part behind.
Change-Id: If637f2a696326778f25ea208224f5ac101f8c64a
Use of all: '.' in Gruntfile.js for eslint config and
eslintIgnore in package.json to ignore linting the "vendor/"
etc directory.
Bug: T203648
Change-Id: I65756ab3b6f413021cedeb22c7933c6cffa2cb78
Enabling SVGO automation with 'grunt-svgmin' and conservative
plugin settings to build step, among those:
- enable removeRasterImages and sortAttrs,
- disable cleanupIDs, removeDesc, removeTitle, removeViewBox &
removeXMLProcInst and
- make use of pretty, indent and multipass options.
Also updating SVG accordingly.
Bug: T185596
Change-Id: I0030a711a2947ca8c1eb4e56d8540661c72a2639
- Updated versions and order of checkers.
- Some consistency updates in Gruntfile.js.
- Updates to make jscs pass.
Change-Id: I7ed93596ff383dc713bd17af2da0472be934d59d