add Node.js v10 to build; fix win32 issues (#3350)

* fix npm version to 5.x
* AppVeyor: add Node.js v10; use npm ci for speed
* revert package-lock.json to what npm@5 uses
* add .gitattributes
* remove linebreak-style from eslintrc
* avoid all symlink tests on win32

Signed-off-by: Christopher Hiller <boneskull@boneskull.com>

.eslintrc.yml

@@ -16,9 +16,6 @@ rules:
   strict:
     - error
     - safe
-  linebreak-style:
-    - error
-    - unix
 overrides:
   - files:
     - scripts/**/*.js

.gitattributes

@@ -0,0 +1 @@
+* text=auto

.travis.yml

@@ -7,13 +7,13 @@ stages:
 
 # defaults
 language: node_js
-node_js: '9'
+node_js: '10'
 # `nvm install` happens before the cache is restored, which means
 # we must install our own npm elsewhere (`~/npm`)
 before_install: |
   [[ ! -x ~/npm/node_modules/.bin/npm ]] && {
     # caching feature creates `~/npm` for us
-    cd ~/npm && npm install npm
+    cd ~/npm && npm install npm@^5
     cd -
   } || true
   # avoids bugs around https://github.com/travis-ci/travis-ci/issues/5092
@@ -34,6 +34,9 @@ jobs:
       script: npm start test.node
       node_js: '8'
 
+    - <<: *node
+      node_js: '9'
+
     - <<: *node
       node_js: '6'
 
@@ -66,6 +69,9 @@ jobs:
           - ~/.npm
           - node_modules # npm install, unlike npm ci, doesn't wipe node_modules
 
+    - <<: *smoke
+      node_js: '9'
+
     - <<: *smoke
       node_js: '8'
 

appveyor.yml

@@ -2,6 +2,7 @@ platform:
   - x64
 environment:
   matrix:
+    - nodejs_version: '10'
     - nodejs_version: '9'
     - nodejs_version: '8'
     - nodejs_version: '6'
@@ -10,8 +11,8 @@ install:
   - ps: Install-Product node $env:nodejs_version x64
   - set CI=true
   - set PATH=%APPDATA%\npm;c:\MinGW\bin;%PATH%
-  - npm install -g npm
-  - npm install
+  - npm install -g npm@^5
+  - npm ci --ignore-scripts
 matrix:
   fast_finish: true
 build: off