Replace sprintf-js with util.format()

lib/linters/decimal_zero.js

@@ -1,5 +1,7 @@
 'use strict';
 
+var util = require('util');
+
 module.exports = {
     name: 'decimalZero',
     nodeTypes: ['declaration'],
@@ -29,7 +31,6 @@ module.exports = {
         var message;
         var number;
         var value;
-        var sprintf = require('sprintf-js').sprintf;
         var output = {
             inclusion: 'with',
             type: null
@@ -79,7 +80,7 @@ module.exports = {
             return [{
                 column: value.start.column,
                 line: value.start.line,
-                message: sprintf(this.message, number, output.inclusion, output.type)
+                message: util.format(this.message, number, output.inclusion, output.type)
             }];
         }
     }

lib/linters/duplicate_property.js

@@ -1,13 +1,14 @@
 'use strict';
 
+var helpers = require('../helpers');
+var util = require('util');
+
 module.exports = {
     name: 'duplicateProperty',
     nodeTypes: ['block'],
     message: 'Duplicate property: "%s".',
 
     lint: function duplicatePropertyLinter (config, node) {
-        var sprintf = require('sprintf-js').sprintf;
-        var helpers = require('../helpers');
         var properties = [];
         var results = [];
         var self = this;
@@ -19,7 +20,7 @@ module.exports = {
 
             if (properties.indexOf(property.content) !== -1) {
                 results.push({
-                    message: sprintf(self.message, property.content),
+                    message: util.format(self.message, property.content),
                     column: property.start.column,
                     line: property.start.line
                 });

lib/linters/hex_length.js

@@ -1,15 +1,16 @@
 'use strict';
 
+var util = require('util');
+
 module.exports = {
     name: 'hexLength',
     nodeTypes: ['color'],
     message: '%s should be written in the %s-form format.',
 
     lint: function hexLengthLinter (config, node) {
         var color = '#' + node.content;
-        var valid = true;
-        var sprintf = require('sprintf-js').sprintf;
         var canShorten = false;
+        var valid = true;
 
         switch (config.style) {
             case 'long':
@@ -37,7 +38,7 @@ module.exports = {
 
         if (!valid) {
             return [{
-                message: sprintf(this.message, color, config.style)
+                message: util.format(this.message, color, config.style)
             }];
         }
     }

lib/linters/hex_notation.js

@@ -1,13 +1,14 @@
 'use strict';
 
+var util = require('util');
+
 module.exports = {
     name: 'hexNotation',
     nodeTypes: ['color'],
     message: '%s should be written in %s.',
 
     lint: function hexNotationLinter (config, node) {
         var color = '#' + node.content;
-        var sprintf = require('sprintf-js').sprintf;
         var valid = true;
 
         if (/^#\d+$/.test(color)) {
@@ -35,7 +36,7 @@ module.exports = {
 
         if (!valid) {
             return [{
-                message: sprintf(this.message, color, config.style)
+                message: util.format(this.message, color, config.style)
             }];
         }
     }

lib/linters/hex_validation.js

@@ -1,17 +1,18 @@
 'use strict';
 
+var util = require('util');
+
 module.exports = {
     name: 'hexValidation',
     nodeTypes: ['color'],
     message: 'Hexadecimal color "%s" should be either three or six characters long.',
 
     lint: function hexValidationLinter (config, node) {
         var color = '#' + node.content;
-        var sprintf = require('sprintf-js').sprintf;
 
         if (!/^#([0-9a-f]{3}|[0-9a-f]{6})$/i.test(color)) {
             return [{
-                message: sprintf(this.message, color)
+                message: util.format(this.message, color)
             }];
         }
     }

lib/linters/import_path.js

@@ -1,6 +1,8 @@
 'use strict';
 
 var helpers = require('../helpers');
+var path = require('path');
+var util = require('util');
 
 module.exports = {
     name: 'importPath',
@@ -11,13 +13,11 @@ module.exports = {
     },
 
     lint: function importPathLinter (config, node) {
-        var path = require('path');
         var filename = path.basename(config.path);
         var results = [];
         var excludes;
         var value;
         var file;
-        var sprintf = require('sprintf-js').sprintf;
 
         if (node.first('atkeyword').first('ident').content !== 'import') {
             return null;
@@ -49,7 +49,7 @@ module.exports = {
                     results.push({
                         column: value.start.column,
                         line: value.start.line,
-                        message: sprintf(this.message.extension, file, ' not')
+                        message: util.format(this.message.extension, file, ' not')
                     });
                 }
 
@@ -59,7 +59,7 @@ module.exports = {
                     results.push({
                         column: value.start.column,
                         line: value.start.line,
-                        message: sprintf(this.message.extension, file, '')
+                        message: util.format(this.message.extension, file, '')
                     });
                 }
 
@@ -73,7 +73,7 @@ module.exports = {
                     results.push({
                         column: value.start.column,
                         line: value.start.line,
-                        message: sprintf(this.message.underscore, file, ' not')
+                        message: util.format(this.message.underscore, file, ' not')
                     });
                 }
 
@@ -83,7 +83,7 @@ module.exports = {
                     results.push({
                         column: value.start.column,
                         line: value.start.line,
-                        message: sprintf(this.message.underscore, file, '')
+                        message: util.format(this.message.underscore, file, '')
                     });
                 }
 

lib/linters/property_units.js

@@ -1,5 +1,7 @@
 'use strict';
 
+var util = require('util');
+
 module.exports = {
     name: 'propertyUnits',
     nodeTypes: ['declaration'],
@@ -10,10 +12,9 @@ module.exports = {
 
     lint: function propertyUnitsLinter (config, node) {
         var results = [];
+        var self = this;
         var property;
         var value;
-        var sprintf = require('sprintf-js').sprintf;
-        var self = this;
 
         if (!node.contains('property') || !node.first('property').contains('ident')) {
             return null;
@@ -47,7 +48,7 @@ module.exports = {
             results.push({
                 column: element.start.column,
                 line: element.start.line,
-                message: sprintf(self.message.unit, unit, property)
+                message: util.format(self.message.unit, unit, property)
             });
         });
 
@@ -67,7 +68,7 @@ module.exports = {
                 results.push({
                     column: element.start.column,
                     line: element.start.line,
-                    message: sprintf(self.message.percentage, property)
+                    message: util.format(self.message.percentage, property)
                 });
             });
         }

lib/linters/qualifying_element.js

@@ -1,14 +1,15 @@
 'use strict';
 
+var helpers = require('../helpers');
+var util = require('util');
+
 module.exports = {
     name: 'qualifyingElement',
     nodeTypes: ['selector'],
     message: '%s selectors should not include a qualifying element.',
 
     lint: function qualifyingElementLinter (config, node) {
         var result;
-        var helpers = require('../helpers');
-        var sprintf = require('sprintf-js').sprintf;
 
         node.forEach('typeSelector', function (selector, index) {
             selector = helpers.ensureObject(node.get(index + 1));
@@ -43,7 +44,7 @@ module.exports = {
             return [{
                 column: result.start.column,
                 line: result.start.line,
-                message: sprintf(this.message, result.type.charAt(0).toUpperCase() + result.type.substring(1))
+                message: util.format(this.message, result.type.charAt(0).toUpperCase() + result.type.substring(1))
             }];
         }
     }

lib/linters/space_after_property_colon.js

@@ -1,5 +1,8 @@
 'use strict';
 
+var findIndex = require('lodash.findindex');
+var util = require('util');
+
 module.exports = {
     name: 'spaceAfterPropertyColon',
     nodeTypes: ['declaration'],
@@ -9,12 +12,10 @@ module.exports = {
     },
 
     lint: function spaceAfterPropertyColonLinter (config, node) {
-        var checkIndex;
-        var maybeSpace;
-        var findIndex = require('lodash.findindex');
-        var sprintf = require('sprintf-js').sprintf;
         var style = config.style;
         var valid = true;
+        var checkIndex;
+        var maybeSpace;
 
         // Find the colon (south of the spleen)
         checkIndex = findIndex(node.content, function (element) {
@@ -46,7 +47,7 @@ module.exports = {
             return [{
                 column: maybeSpace.start.column,
                 line: maybeSpace.start.line,
-                message: sprintf(style === 'no_space' ? this.message.noSpace : this.message.oneSpace)
+                message: util.format(style === 'no_space' ? this.message.noSpace : this.message.oneSpace)
             }];
         }
     }

lib/linters/space_after_property_name.js

@@ -1,17 +1,18 @@
 'use strict';
 
+var findIndex = require('lodash.findindex');
+var util = require('util');
+
 module.exports = {
     name: 'spaceAfterPropertyName',
     nodeTypes: ['declaration'],
     message: 'Colon after property should%s be preceded by any space.',
 
     lint: function spaceAfterPropertyNameLinter (config, node) {
-        var checkIndex;
-        var maybeSpace;
-        var findIndex = require('lodash.findindex');
-        var sprintf = require('sprintf-js').sprintf;
         var style = config.style;
         var valid = true;
+        var checkIndex;
+        var maybeSpace;
 
         // Find the colon (it's next to the prostate)
         checkIndex = findIndex(node.content, function (element) {
@@ -44,7 +45,7 @@ module.exports = {
             return [{
                 column: maybeSpace.start.column,
                 line: maybeSpace.start.line,
-                message: sprintf(this.message, style === 'no_space' ? ' not' : '')
+                message: util.format(this.message, style === 'no_space' ? ' not' : '')
             }];
         }
     }

lib/linters/space_after_property_value.js

@@ -1,13 +1,14 @@
 'use strict';
 
+var util = require('util');
+
 module.exports = {
     name: 'spaceAfterPropertyValue',
     nodeTypes: ['block'],
     message: 'Semicolon after property value should%s be preceded by %s space.',
 
     lint: function spaceAfterPropertyValueLinter (config, node) {
         var results = [];
-        var sprintf = require('sprintf-js').sprintf;
         var self = this;
 
         node.forEach('declarationDelimiter', function (element, index) {
@@ -19,7 +20,7 @@ module.exports = {
                         results.push({
                             column: maybeSpace.start.column,
                             line: maybeSpace.start.line,
-                            message: sprintf(self.message, ' not', 'any')
+                            message: util.format(self.message, ' not', 'any')
                         });
                     }
 
@@ -29,7 +30,7 @@ module.exports = {
                         results.push({
                             column: maybeSpace.start.column,
                             line: maybeSpace.start.line,
-                            message: sprintf(self.message, '', 'one')
+                            message: util.format(self.message, '', 'one')
                         });
                     }
 

lib/linters/space_around_comma.js

@@ -1,13 +1,14 @@
 'use strict';
 
+var util = require('util');
+
 module.exports = {
     name: 'spaceAroundComma',
     nodeTypes: ['arguments', 'parentheses'],
     message: 'Commas should%s be %s by %s space.',
 
     lint: function spaceAroundCommaLinter (config, node) {
         var results = [];
-        var sprintf = require('sprintf-js').sprintf;
         var self = this;
 
         node.forEach('operator', function (element, index) {
@@ -30,14 +31,14 @@ module.exports = {
             switch (config.style) {
                 case 'after':
                     if (nextElement.type !== 'space' || nextElement.content !== ' ') {
-                        message = sprintf(self.message, '', 'followed', 'one');
+                        message = util.format(self.message, '', 'followed', 'one');
                     }
                     break;
 
                 case 'before':
                     if (prevElement.type !== 'space' || prevElement.content !== ' ') {
                         startElement = prevElement;
-                        message = sprintf(self.message, '', 'preceded', 'one');
+                        message = util.format(self.message, '', 'preceded', 'one');
                     }
                     break;
 
@@ -46,14 +47,14 @@ module.exports = {
                        (prevElement.type !== 'space' || prevElement.content !== ' ') || !/\s$/.test(prevElement.content)) {
 
                         startElement = !/\s$/.test(prevElement.content) ? prevElement : nextElement;
-                        message = sprintf(self.message, '', 'preceded and followed', 'one');
+                        message = util.format(self.message, '', 'preceded and followed', 'one');
                     }
                     break;
 
                 case 'none':
                     if (nextElement.type === 'space' || prevElement.type === 'space') {
                         startElement = prevElement.type === 'space' ? prevElement : nextElement;
-                        message = sprintf(self.message, ' not', 'preceded nor followed', 'any');
+                        message = util.format(self.message, ' not', 'preceded nor followed', 'any');
                     }
                     break;