You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@cordova.apache.org by "Steve Gill (JIRA)" <ji...@apache.org> on 2017/06/07 18:37:18 UTC
[jira] [Created] (CB-12895) Replace jshint with eslint in our tools
+ platforms
Steve Gill created CB-12895:
-------------------------------
Summary: Replace jshint with eslint in our tools + platforms
Key: CB-12895
URL: https://issues.apache.org/jira/browse/CB-12895
Project: Apache Cordova
Issue Type: Improvement
Reporter: Steve Gill
Priority: Minor
Based on this proposal
https://github.com/cordova/cordova-discuss/pull/70#issuecomment-306649308
JSHint does minimal checks and JSCS has been deprecated in favor of ESLint.
+
+I suggest to replace the current style checkers with ESLint. This allows to set much more strict styling rules than at at this moment.
+
+I recommend using a popular preset, such as [airbnb-base](https://www.npmjs.com/package/eslint-config-airbnb-base). Example configuration:
+
+```yaml
+root: true
+
+extends: airbnb-base
+```
+
+Many styling issues could be fixed by simply running:
+
+```
+./node_modules/.bin/eslint --fix .
+```
+
+
+
+Optionally the indent could be set to 4 instead of 2, because that's used in the current code base.
--
This message was sent by Atlassian JIRA
(v6.3.15#6346)
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@cordova.apache.org
For additional commands, e-mail: issues-help@cordova.apache.org