Skip to content

Conversation

@daxian-dbw
Copy link
Member

@daxian-dbw daxian-dbw commented Jun 5, 2018

PR Summary

  • Stop caching the dotnet folder
  • Skip the initialization of the dotnet cli cache.

We already do so with AppVeyor via #6945, so I don't see why we shouldn't do the same for Travis CI.

PR Checklist

Copy link

@anmenaga anmenaga left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Changes in #6945 were done based on measurements done in AppVeyor.
These changes are for Travis CI. Maybe timings are different and caching brings value. Maybe worth measuring before removing.

@TravisEz13
Copy link
Member

TravisEz13 commented Jun 5, 2018

Here are the timings of the last few PRs. This one in is in green, and one redacted because it was cancelled:
image

@daxian-dbw daxian-dbw merged commit 6ea1389 into PowerShell:master Jun 5, 2018
@daxian-dbw daxian-dbw deleted the travis branch June 5, 2018 22:51
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.

3 participants