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

Add support for restoring from partial cache match #404

Open
artuross opened this issue Aug 2, 2023 · 1 comment
Open

Add support for restoring from partial cache match #404

artuross opened this issue Aug 2, 2023 · 1 comment
Labels
feature request New feature or request to improve the current logic

Comments

@artuross
Copy link

artuross commented Aug 2, 2023

Description:
Currently, it seems that this action only restores cache if the cache key is matched exactly (go.mod and go.sum haven't changed).

Justification:
Typically, when updating dependencies, only some dependencies will be updated. With current version, it means that whenever a dependency is updated, the cache is not used and therefore all packages must be fetched. In my view, this is suboptimal.

I think there should be a setting to enable this behavior.

Are you willing to submit a PR?
Yes.

@artuross artuross added feature request New feature or request to improve the current logic needs triage labels Aug 2, 2023
@dusan-trickovic
Copy link
Contributor

Hello, @artuross ! Thank you for the suggestion, we will investigate it :)

@dsame dsame added needs eyes investigation The issue is under investigation and removed investigation The issue is under investigation labels Aug 3, 2023
@dsame dsame removed the needs eyes label Aug 31, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature request New feature or request to improve the current logic
Projects
None yet
Development

No branches or pull requests

4 participants
@dsame @artuross @dusan-trickovic and others