feat(gulp): lint xml listings

close #431

Author: duncdrum

Gulpfile.js

@@ -44,7 +44,8 @@ var paths = {
   },
   fonts: {
     output: 'src/main/xar-resources/resources/fonts/',
-  }
+  },
+  listings: 'src/main/xar-resources/data/*/listings/*.xml'
 }
 
 /**
@@ -85,6 +86,8 @@ var lazypipe = require('lazypipe')
 var rename = require('gulp-rename')
 var header = require('gulp-header')
 var pkg = require('./package.json')
+var prettyData = require('gulp-pretty-data')
+
 
 // Scripts
 var standard = require('gulp-standard')
@@ -193,6 +196,23 @@ var lintScripts = function(done) {
   done()
 }
 
+// minify and lint xml
+var prettyXml = function(done) {
+  src(paths.listings, { base: "./" })
+  .pipe(prettyData({
+      type: 'minify',
+      preserveComments: true
+    }))
+    // … then pretty print xml
+    .pipe(prettyData({
+        type: 'prettify',
+        preserveComments: true
+      }))
+    .pipe(dest("./"))
+    // Signal completion
+    done()
+}
+
 // Process, lint, and minify Sass files
 var buildStyles = function(done) {
   // Make sure this feature is activated before running
@@ -327,6 +347,7 @@ exports.default = series(
     buildStyles,
     buildSVGs,
     copyFiles,
-    buildPack
+    buildPack,
+    prettyXml
   )
 )

package-lock.json

@@ -40,6 +40,7 @@
     "gulp-flatmap": "^1.0.2",
     "gulp-header": "^2.0.9",
     "gulp-optimize-js": "^1.1.0",
+    "gulp-pretty-data": "^0.1.2",
     "gulp-rename": "^2.0.0",
     "gulp-sass": "^4.0.2",
     "gulp-sourcemaps": "^2.6.5",