closes #628

.gitignore

@@ -30,3 +30,6 @@ database-backup.json
 # Exclude FTP config file
 ftp-config.json
 
+# Exclude ESLint configuration file
+.eslintrc
+

.npmignore

@@ -30,6 +30,9 @@ database-backup.json
 # Exclude FTP config file
 ftp-config.json
 
+# Exclude ESLint configuration file
+.eslintrc
+
 # Exclude Build folder
 build/
 

CHANGELOG.md

@@ -4,6 +4,10 @@
 
 ## Next version
 
+### New Features
+
+- [x] [#628 - Overriding StandardJS rules](https://github.com/scriptPilot/app-framework/issues/628) ([read the docs](https://github.com/scriptPilot/app-framework/blob/master/docs/eslint.md))
+
 ### Improvements
 
 - [x] [#572 - Rename database-rules.json to firebase-database.json](https://github.com/scriptPilot/app-framework/issues/572)

DOCUMENTATION.md

@@ -38,6 +38,7 @@ For any open question, feature request or bug, please use our [issue list](https
   - [Application style](docs/application-style.md)
   - [Status bar style](docs/status-bar-style.md)
 - [Test your application](docs/test.md)
+  - [Configure ESLint code checks](docs/eslint.md)
 - [Build your application](docs/build.md)
 - [Deploy your application](docs/deploy.md)
 - [Backup your project](docs/backup.md)

README.md

@@ -29,7 +29,7 @@ Develop your application:s
 
 Test your application:
 
-- Integrated code check for [correctness](http://eslint.org/) and automatic fix for [standard conformity](http://standardjs.com/)
+- Integrated configurable code check and fix with [ESLint](http://eslint.org/), based on [Airbnb](https://github.com/airbnb/javascript) or [Standard JavaScript](http://standardjs.com/)
 - Local development server with live reload, perfectly to use together with [Chrome DevTools](https://developers.google.com/web/tools/chrome-devtools/)
 - Test your application on any iOS or Android device with an emulator or on your own device
 - Use a development Firebase project with automatic [database](https://firebase.google.com/products/database/) and [storage](https://firebase.google.com/products/storage/) rule update

config-scheme.json

@@ -176,6 +176,20 @@
     "type": "boolean",
     "default": false
   },
+  "eslint": {
+    "props": {
+      "extends": {
+        "allow": ["airbnb", "standard"],
+        "default": "airbnb"
+      },
+      "rules": {
+        "type": "object",
+        "default": {
+          "semi": ["error", "never"]
+        }
+      }
+    }
+  },
   "fixCodeOnTest": {
     "type": "boolean",
     "default": true

demo/config.json

@@ -54,6 +54,10 @@
   "resetLocalStorageOnVersionChange": false,
   "preloadImages": true,
   "buildSourcemaps": false,
+  "eslint": {
+    "extends": "standard",
+    "rules": {}
+  },
   "fixCodeOnTest": true,
   "fixCodeOnBuild": true,
   "devServerPort": "8080"

docs/dev-coding-rules.md

@@ -84,9 +84,9 @@ let names = 'everything but no array' /* test */
 ...
 ```
 
-## Standard JavaScript
+## Airbnb JavaScript rules
 
-All code is checked according [Standard JS rules](http://standardjs.com/) before each build.
+All code is checked according [Airbnb JavaScript rules](https://github.com/airbnb/javascript) (but without semicolons) before each build.
 
 ## Naming in code
 

docs/eslint.md

@@ -0,0 +1,32 @@
+# ESLint code checks
+
+> This page is part of the [App Framework Documentation](../DOCUMENTATION.md)
+
+<br />
+
+The code is checked with ESLint. You can apply your own configuration easily in the *app/config.json* file. The *.eslintrc* file will be updated before each test according your configuration.
+
+The new default configuration:
+
+```
+"eslint": {
+  "extends": "airbnb",
+  "rules": {
+    "semi": [
+      "error",
+      "never"
+    ]
+  }
+}
+```
+
+The old default configuration:
+
+```
+"eslint": {
+  "extends": "standard",
+  "rules": {}
+}
+```
+
+Currently, `airbnb` and `standard` are available to extend - if you which to extend another shared configuration, please ask for it in our [issue list](https://github.com/scriptPilot/app-framework/issues).