Skip to content

Commit 40ca0b3

Browse files
committed
Actually fail when test are failing.
The whole tests were under allow-failure (and are failing). That is to say: some html link don't resolve, and some css is incorrect. Put at least some of them (spelling) as mandatory.
1 parent 8a74c5d commit 40ca0b3

File tree

2 files changed

+27
-9
lines changed

2 files changed

+27
-9
lines changed

.travis.yml

+24-8
Original file line numberDiff line numberDiff line change
@@ -5,21 +5,37 @@ rvm:
55
matrix:
66
allow_failures:
77
- rvm: 2.2
8+
include:
9+
- python: 3
10+
env: SPELL=1
11+
- rvm: 2.2
12+
env: SPELL=0
13+
814

915
before_install:
1016
- nvm install node
1117

1218
install:
13-
- npm install -g csslint
14-
- pip install codespell --user
15-
- bundle install
19+
- |
20+
if [ "$SPELL" = "1" ]
21+
then
22+
pip install codespell --user
23+
else
24+
npm install -g csslint
25+
bundle install
26+
fi
1627
1728
script:
18-
- bash .travis/cibuild
19-
- csslint _site/css/gallery.css
20-
- csslint _site/css/logo-nav.css
21-
- csslint _site/css/syntax.css
22-
- codespell -q3 --skip='.git,*.svg,*.json,*.yml,*.lock,_site'
29+
- |
30+
if [ "$SPELL" = "1" ]
31+
then
32+
codespell -q3 --skip='.git,*.svg,*.json,*.yml,*.lock,_site'
33+
else
34+
bash .travis/cibuild
35+
csslint _site/css/gallery.css
36+
csslint _site/css/logo-nav.css
37+
csslint _site/css/syntax.css
38+
fi
2339
2440
# branch whitelist, only for GitHub Pages
2541

.travis/cibuild

+3-1
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,7 @@
11
#!/usr/bin/env bash
2-
set -e # halt script on error
2+
set -Exeuo pipefail # -e halt script on error, -u uset variable is error,
3+
# -o pipefail carry errors up, -x print each line
4+
# -E catch signals
35

46
bundle exec jekyll build
57
bundle exec htmlproofer ./_site

0 commit comments

Comments
 (0)