Merge pull request #730 from nodejs/editorocnfig

Introduce EditorConfig

Author: LaurentGoderre

.editorconfig

@@ -0,0 +1,11 @@
+root = true
+
+[*]
+charset = utf-8
+end_of_line = lf
+insert_final_newline = true
+trim_trailing_whitespace = true
+
+[*.sh]
+indent_size = 2
+indent_style = space

.travis.yml

@@ -22,6 +22,17 @@ stages:
 
 jobs:
   include:
+    - stage: Test
+      env:
+        - TEST: Editor Config check
+      language: node_js
+      node_js:
+        - lts/*
+      install:
+        - npm i -g eclint
+      script:
+        - eclint check
+
     - stage: Test
       env:
         - TEST: Doc Toc Check
@@ -53,105 +64,105 @@ jobs:
     # Docker Build #
     - stage: Build
       env:
-         - NODE_VERSION: "10"
-         - VARIANT: "default"
+        - NODE_VERSION: "10"
+        - VARIANT: "default"
 
     - stage: Build
       env:
-         - NODE_VERSION: "10"
-         - VARIANT: "alpine"
+        - NODE_VERSION: "10"
+        - VARIANT: "alpine"
 
     - stage: Build
       env:
-         - NODE_VERSION: "10"
-         - VARIANT: "slim"
+        - NODE_VERSION: "10"
+        - VARIANT: "slim"
 
     - stage: Build
       env:
-         - NODE_VERSION: "10"
-         - VARIANT: "stretch"
+        - NODE_VERSION: "10"
+        - VARIANT: "stretch"
 
     - stage: Build
       env:
-         - NODE_VERSION: "6"
-         - VARIANT: "default"
+        - NODE_VERSION: "6"
+        - VARIANT: "default"
 
     - stage: Build
       env:
-         - NODE_VERSION: "6"
-         - VARIANT: "alpine"
+        - NODE_VERSION: "6"
+        - VARIANT: "alpine"
 
     - stage: Build
       env:
-         - NODE_VERSION: "6"
-         - VARIANT: "onbuild"
+        - NODE_VERSION: "6"
+        - VARIANT: "onbuild"
 
     - stage: Build
       env:
-         - NODE_VERSION: "6"
-         - VARIANT: "slim"
+        - NODE_VERSION: "6"
+        - VARIANT: "slim"
 
     - stage: Build
       env:
-         - NODE_VERSION: "6"
-         - VARIANT: "stretch"
+        - NODE_VERSION: "6"
+        - VARIANT: "stretch"
 
     - stage: Build
       env:
-         - NODE_VERSION: "8"
-         - VARIANT: "default"
+        - NODE_VERSION: "8"
+        - VARIANT: "default"
 
     - stage: Build
       env:
-         - NODE_VERSION: "8"
-         - VARIANT: "alpine"
+        - NODE_VERSION: "8"
+        - VARIANT: "alpine"
 
     - stage: Build
       env:
-         - NODE_VERSION: "8"
-         - VARIANT: "onbuild"
+        - NODE_VERSION: "8"
+        - VARIANT: "onbuild"
 
     - stage: Build
       env:
-         - NODE_VERSION: "8"
-         - VARIANT: "slim"
+        - NODE_VERSION: "8"
+        - VARIANT: "slim"
 
     - stage: Build
       env:
-         - NODE_VERSION: "8"
-         - VARIANT: "stretch"
+        - NODE_VERSION: "8"
+        - VARIANT: "stretch"
 
     - stage: Build
       env:
-         - NODE_VERSION: "9"
-         - VARIANT: "default"
+        - NODE_VERSION: "9"
+        - VARIANT: "default"
 
     - stage: Build
       env:
-         - NODE_VERSION: "9"
-         - VARIANT: "alpine"
+        - NODE_VERSION: "9"
+        - VARIANT: "alpine"
 
     - stage: Build
       env:
-         - NODE_VERSION: "9"
-         - VARIANT: "onbuild"
+        - NODE_VERSION: "9"
+        - VARIANT: "onbuild"
 
     - stage: Build
       env:
-         - NODE_VERSION: "9"
-         - VARIANT: "slim"
+        - NODE_VERSION: "9"
+        - VARIANT: "slim"
 
     - stage: Build
       env:
-         - NODE_VERSION: "9"
-         - VARIANT: "stretch"
+        - NODE_VERSION: "9"
+        - VARIANT: "stretch"
 
     - stage: Build
       env:
-         - NODE_VERSION: "chakracore/10"
-         - VARIANT: "default"
+        - NODE_VERSION: "chakracore/10"
+        - VARIANT: "default"
 
     - stage: Build
       env:
-         - NODE_VERSION: "chakracore/8"
-         - VARIANT: "default"
+        - NODE_VERSION: "chakracore/8"
+        - VARIANT: "default"

docs/BestPractices.md

@@ -105,7 +105,7 @@ RUN deluser --remove-home node \
 
 ## Memory
 
-By default, any Docker Container may consume as much of the hardware such as CPU and RAM. If you are running multiple containers on the same host you should limit how much memory they can consume.     
+By default, any Docker Container may consume as much of the hardware such as CPU and RAM. If you are running multiple containers on the same host you should limit how much memory they can consume.
 
 ```
 -m "300M" --memory-swap "1G"

update.sh

@@ -79,8 +79,8 @@ function add_stage() {
 
   echo '    - stage: Build
       env:
-         - NODE_VERSION: "'"$version"'"
-         - VARIANT: "'"$variant"'"
+        - NODE_VERSION: "'"$version"'"
+        - VARIANT: "'"$variant"'"
 ' >>.travis.yml
 }