From 6b85be8e4e27eee5179ccd7a458742af9e98756b Mon Sep 17 00:00:00 2001 From: xzyfer Date: Mon, 26 Sep 2016 20:40:08 +1000 Subject: [PATCH] Remove skip download for CI flag This flag means we skip a bunch of code paths in CI which is not ideal. Since we tell people to use `npm rebuild` we should eat our own dog food. With this patch we do an install, then do a rebuild hitting all the major install code paths. Fixes #1453 --- .travis.yml | 9 +++++++++ appveyor.yml | 4 ++-- 2 files changed, 11 insertions(+), 2 deletions(-) diff --git a/.travis.yml b/.travis.yml index e127d516e..5de07f839 100644 --- a/.travis.yml +++ b/.travis.yml @@ -2,6 +2,7 @@ language: node_js compiler: gcc +<<<<<<< HEAD env: global: - SKIP_SASS_BINARY_DOWNLOAD_FOR_CI=true @@ -21,6 +22,13 @@ jobs: os: linux - stage: platform-test node_js: "10" +======= +matrix: + fast_finish: true + exclude: + - node_js: iojs + - node_js: "0.10" +>>>>>>> 7110e43d... Remove skip download for CI flag os: osx - stage: platform-test node_js: "13" @@ -65,6 +73,7 @@ install: - npm install script: + - npm run build - npm test cache: diff --git a/appveyor.yml b/appveyor.yml index a2a4f4f22..473c3fd3b 100644 --- a/appveyor.yml +++ b/appveyor.yml @@ -32,7 +32,6 @@ - '%AppData%\npm-cache' environment: - SKIP_SASS_BINARY_DOWNLOAD_FOR_CI: true matrix: - nodejs_version: 10 GYP_MSVS_VERSION: 2017 @@ -50,6 +49,7 @@ - node --version - npm --version - npm install + - npm run build test: off @@ -108,7 +108,6 @@ - '%AppData%\npm-cache' environment: - SKIP_SASS_BINARY_DOWNLOAD_FOR_CI: true matrix: - nodejs_version: 10 GYP_MSVS_VERSION: 2017 @@ -126,6 +125,7 @@ - node --version - npm --version - npm install + - npm run build test_script: - ps: set-location -path c:\projects\node_modules\node-sass