Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix build cache issues #10014

Merged
merged 1 commit into from Jan 29, 2020
Merged

Fix build cache issues #10014

merged 1 commit into from Jan 29, 2020

Conversation

marcospereira
Copy link
Member

No description provided.

We don't want to cache ivy local artifacts generated for the build, but
it is okay to keep Play artifacts in ivy cache directory since they are
used when checking binary compatibility.
.travis.yml Outdated Show resolved Hide resolved
Copy link
Member

@dwijnand dwijnand left a comment

Choose a reason for hiding this comment

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

This is relevant because MiMa (still) uses Ivy and therefore ~/.ivy2/cache, good catch.

This rm existed for scripted and before using sbt-dynver, to safeguard from publishLocal'ing and then running scripted on an older version (not confirmed, but feared). So it can go now.

@dwijnand dwijnand merged commit e2c2903 into playframework:master Jan 29, 2020
@marcospereira marcospereira deleted the build/fix-small-cache-issues branch February 29, 2020 00:27
@mkurz
Copy link
Member

mkurz commented Jan 22, 2021

I am pretty sure we can backport to 2.8.x because also in 2.8.x we now use sbt-dynver.

@mkurz
Copy link
Member

mkurz commented Jan 22, 2021

@Mergifyio backport 2.8.x

@mergify
Copy link
Contributor

mergify bot commented Jan 22, 2021

Command backport 2.8.x: success

Backports have been created

mergify bot added a commit that referenced this pull request Jan 22, 2021
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.

None yet

3 participants