Skip to content

Commit 9df3104

Browse files
authored
Revert "Don't run CI on Node 0.10" (facebook#1547)
* Revert "Don't run CI on Node 0.10" * Install after checking node version * Don't use travis install
1 parent ce0d6ee commit 9df3104

File tree

2 files changed

+5
-2
lines changed

2 files changed

+5
-2
lines changed

.travis.yml

+3
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,7 @@ cache:
88
- node_modules
99
- packages/create-react-app/node_modules
1010
- packages/react-scripts/node_modules
11+
install: true
1112
script:
1213
- 'if [ $TEST_SUITE = "simple" ]; then tasks/e2e-simple.sh; fi'
1314
- 'if [ $TEST_SUITE = "installs" ]; then tasks/e2e-installs.sh; fi'
@@ -21,5 +22,7 @@ env:
2122
- TEST_SUITE=kitchensink
2223
matrix:
2324
include:
25+
- node_js: 0.10
26+
env: TEST_SUITE=simple
2427
- node_js: 6
2528
env: USE_YARN=yes TEST_SUITE=simple

tasks/e2e-simple.sh

+2-2
Original file line numberDiff line numberDiff line change
@@ -65,8 +65,6 @@ set -x
6565
cd ..
6666
root_path=$PWD
6767

68-
npm install
69-
7068
# If the node version is < 4, the script should just give an error.
7169
if [[ `node --version | sed -e 's/^v//' -e 's/\..*//g'` -lt 4 ]]
7270
then
@@ -75,6 +73,8 @@ then
7573
[[ $err_output =~ You\ are\ running\ Node ]] && exit 0 || exit 1
7674
fi
7775

76+
npm install
77+
7878
if [ "$USE_YARN" = "yes" ]
7979
then
8080
# Install Yarn so that the test can use it to install packages.

0 commit comments

Comments
 (0)