Skip to content

Add NuGet package caching to Travis script#389

Merged
jpreese merged 1 commit into
masterfrom
travis-nuget-cache
Aug 29, 2017
Merged

Add NuGet package caching to Travis script#389
jpreese merged 1 commit into
masterfrom
travis-nuget-cache

Conversation

@ErikSchierboom

Copy link
Copy Markdown
Member

This should speed up the build a bit.

@jpreese

jpreese commented Aug 29, 2017

Copy link
Copy Markdown
Contributor

So a couple things I noticed.

The things not to cache documentation on TravisCI points out compiled binaries, and states that it has to download the cache anyway. NuGet packages, to me, would fall into the "large download, quick to install" camp.

Looking at the restore step from this build to previous builds, the time didn't really seem to change much. Which would make sense if the behavior is to download all of the files anyway.

Theres also this ticket relating to a similar problem. It's not super helpful, but they ended up deciding against it.

If I correctly understand how TravisCI caching works, I don't think we'll see a lot of improvement.

@ErikSchierboom

Copy link
Copy Markdown
Member Author

It is logical that we won't see any performance improvement the first time: the cache isn't there yet :D I'll restart it to see if things improve. If not, we can close this.

@jpreese

jpreese commented Aug 29, 2017

Copy link
Copy Markdown
Contributor

Yeah I had that theory as well, I should've just clicked the restart. :(

@ErikSchierboom

Copy link
Copy Markdown
Member Author

@jpreese The build appears to be a bit faster (15 sec.), so I think we can merge this.

@jpreese jpreese merged commit 080952f into master Aug 29, 2017
@jpreese

jpreese commented Aug 29, 2017

Copy link
Copy Markdown
Contributor

Yep! 👍

@ErikSchierboom ErikSchierboom deleted the travis-nuget-cache branch August 29, 2017 19:37
@ErikSchierboom

Copy link
Copy Markdown
Member Author

Hurray! Now all I want to do in the future is to see if running the tests with the xUnit runner (which uses parallellism by default IIRC) speeds up things, but first getting the F# track ready.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants