polylint - npm Package Compare versions

Comparing version 2.7.2 to 2.7.3

bin/polylint.js

@@ -24,4 +24,2 @@ #!/usr/bin/env node
 
-console.log(process.argv);
-
 var cli = cliArgs([
@@ -235,3 +233,2 @@ {
       process.stdin.on('end', function() {
-        console.log("Read stdin!");
         resolve(true);
@@ -245,8 +242,3 @@ });
 
-/*jshint -W083 */
-for(var i = 0; i < inputs.length; i++) {
-  // Check whether input is a root directory before picking a root and
-  // a path to process.
-  var input = inputs[i];
-
+inputs.forEach(function(input){
   // If root has been set by cwd and input is an absolute path that begins with the cwd path,
@@ -260,3 +252,2 @@ // strip the root part of the input path to make the FS resolver not duplicate the root path
   lintPromise = lintPromise.then(function() {
-    console.log("after stdin promise");
     return polylint(
@@ -276,2 +267,3 @@ input,
       if (options['no-recursion'] && input !== warning.filename) {
+        console.log("skipping");
         return;
@@ -286,4 +278,3 @@ }
   });
-}
-/*jshint +W083 */
+});
 
@@ -290,0 +281,0 @@ var exit = function(){

bower.json

 {
   "name": "polylint",
   "private": true,
-  "version": "2.7.2",
+  "version": "2.7.3",
   "homepage": "https://github.com/PolymerLabs/polylint",
@@ -6,0 +6,0 @@ "authors": [

lib/expressions.js

@@ -63,2 +63,3 @@ /**
       }
+      expression = expression.trim();
     }
@@ -65,0 +66,0 @@ return expression;

package.json

 {
   "name": "polylint",
-  "version": "2.7.2",
+  "version": "2.7.3",
   "description": "Keeps your Polymer Elements clean and functional!",
@@ -8,3 +8,3 @@ "main": "polylint.js",
   "scripts": {
-    "test": "node_modules/.bin/jshint polylint.js bin lib test && node_modules/.bin/mocha test/test.js"
+    "test": "bower install && node_modules/.bin/jshint polylint.js bin lib test && node_modules/.bin/mocha test/test.js"
   },
@@ -31,2 +31,3 @@ "repository": {
   "devDependencies": {
+    "bower": "^1.6.5",
     "chai": "^2.3.0",
@@ -33,0 +34,0 @@ "jshint": "^2.7.0",

README.md

@@ -8,2 +8,14 @@ # Polylint
 
+### Installing the Atom Package
+
+Polylint provides a package for the Atom editor for in-line code linting. To install:
+
+1. Install the [linter](https://atom.io/packages/linter) package with `apm install linter` or through Atom's [package installer interface](https://atom.io/docs/latest/using-atom-atom-packages)
+2. Install the [polymer-atom](https://github.com/PolymerLabs/polymer-atom) package through `apm install polymer-atom` or through Atom's package installer.
+3. Lint!
+
+### Installing the Sublime Plugin
+
+There is currently a Sublime plugin that leverages Polylint available at [https://github.com/nomego/SublimeLinter-contrib-polylint](https://github.com/nomego/SublimeLinter-contrib-polylint)
+
 ## Usage
@@ -19,1 +31,5 @@ If you want to lint a project in `my-project-dir` with two endpoints, `index.html` and `cart.html`, you could run:
 ```
+
+## Contributing
+
+Polymer :heart: contributions! Please see the [Contributing Guide](https://github.com/Polymer/project/blob/master/Contributing.md) for general Polymer project contribution guidelines.

test/linter-test.js

@@ -171,2 +171,10 @@ /**
 
+  test('whitespace', function() {
+    var w = findWarnings(warnings, 'whitespace');
+    assert.equal(w.length, 1);
+    var first = w[0];
+    assert.equal(first.location.line, 13);
+    assert.include(first.message, 'whiteSpaceNamez');
+  });
+
 });

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

91895

increased by2.08%

Number of package files

37

increased by5.71%

Lines of code

2060

increased by0.05%

Number of lines in readme file

34

increased by88.89%

Worsened metrics

Dev dependency count

4

increased by33.33%

No dependency changes