cordova-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From audreyso <...@git.apache.org>
Subject [GitHub] cordova-js pull request #149: CB-12895 : added eslint to repo
Date Tue, 26 Sep 2017 18:01:24 GMT
Github user audreyso commented on a diff in the pull request:

    https://github.com/apache/cordova-js/pull/149#discussion_r141135727
  
    --- Diff: package.json ---
    @@ -15,6 +15,7 @@
         "url": "https://issues.apache.org/jira/browse/CB"
       },
       "scripts": {
    +    "eslint": "eslint src test",
    --- End diff --
    
    If I do `npm run eslint pkg/`, it will reveal all of the pkg/ errors. I can reduce some
errors using --fix (mostly indenting errors) but the rest would have to be fixed manually.
I won't be able to change/fix anything, however, unless I can check in the pkg/ directory
though. If I run `npm run eslint` without specifying pkg, it will just ignore it. Hmm...


---

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@cordova.apache.org
For additional commands, e-mail: dev-help@cordova.apache.org


Mime
View raw message