Conversation
|
I'm also thinking about if we can just pin the nodejs version? Since I didn't see a requirement to install the latest version. |
.travis.yml
Outdated
| - $HOME/.cabal | ||
| - $TRAVIS_BUILD_DIR/.cache | ||
| - $TRAVIS_BUILD_DIR/node_modules | ||
| - $TRAVIS_BUILD_DIR/versions |
There was a problem hiding this comment.
The benefit here would be "the amount of time it takes to untar the cache" - I intentionally did not cache this part - we might make all sorts of changes to an installed node version over the course of tests, and making tests deterministic is far more important than making them fast. Let's remove this commit.
| - $HOME/.ghc | ||
| - $HOME/.cabal | ||
| - $TRAVIS_BUILD_DIR/.cache | ||
| - $TRAVIS_BUILD_DIR/node_modules |
There was a problem hiding this comment.
If we're going to do this, then after npm install, we need to always run npm prune.
d56e21e to
d970447
Compare
| - wget --version | ||
| - if [ -n "${SHELLCHECK-}" ]; then cabal update && cabal install transformers-0.4.3.0 ShellCheck && shellcheck --version ; fi | ||
| - if [ -z "${SHELLCHECK-}" ]; then nvm install node && npm install && npm ls urchin doctoc; fi | ||
| - if [ -z "${SHELLCHECK-}" ]; then nvm install node && npm install && npm prune && npm ls urchin doctoc; fi |
There was a problem hiding this comment.
not a blocker, but it occurs to me that this step should probably go in install, not before_install?
There was a problem hiding this comment.
I agree, that's the place what it already been for a while, I can send another commit to move both npm and cabel into install section, what do you think?
This should make the CI faster.