Skip to content

Conversation

@weswigham
Copy link
Member

This could speed up CI builds a little bit.

See https://docs.travis-ci.com/user/caching/ for more information.

This could speed up CI builds a little bit

See https://docs.travis-ci.com/user/caching/ for more information.
@vladima
Copy link
Contributor

vladima commented Jun 22, 2016

is it possible to explicitly force cache invalidation in cases if our dependencies really should be updated?

@kitsonk
Copy link
Contributor

kitsonk commented Jun 22, 2016

Those authorised on the Travis account can clear the cache.

@weswigham
Copy link
Member Author

It looks like this cuts about a minute and a half off our travis builds. I think this is good.

@weswigham
Copy link
Member Author

@vladima

We should only have to run npm update to make sure our deps are updated in the cache.... however since we transitively depend on ourselves, npm update refuses to run. This can be worked around by changing the name in our package.json before running npm update, then changing it back... but this still has a downside: only our top-level dependencies get checked. If we add the argument --depth 999, theoretically we could catch everything, but when we use depth at all we hit this npm bug.

If the partial solution is okay (despite the circular dependency hack) I can add it to this PR.

@weswigham
Copy link
Member Author

@vladima Look like my new strategy for cache busting works! (Which is "don't cache packages npm has trouble with", like typescript or tslint). This shaves > 1 minutes off of our builds - should we merge with master now?

@sandersn
Copy link
Member

👍

@weswigham weswigham merged commit 2f8df90 into master Jul 29, 2016
@weswigham weswigham deleted the travis-cache branch July 29, 2016 21:49
@microsoft microsoft locked and limited conversation to collaborators Jun 19, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants