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

feat: add action input of cache-dependency-path #468

Merged
merged 1 commit into from
Oct 12, 2023

Conversation

noppaz
Copy link
Contributor

@noppaz noppaz commented Oct 12, 2023

By enabling changing the go.sum file(s) to cache on users can configure which subdir is important for their respective actions.

The main use case would most likely be to cache the dependencies of sage in repositories that have many dependencies for sage itself, but it could also be used to cache dependencies in multi module repositories. This is especially useful for sage targets that doesn't run go code but instead a downloaded binary (e.g., Terraform). Some tests of internal repositories halves the time of execution as sage dependency installation is a big part of the runtime of the action.

The default is set to what setup-go has as default; the root go.sum in the repository.

Copy link
Member

@Edholm Edholm left a comment

Choose a reason for hiding this comment

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

Neat 👍

@noppaz noppaz merged commit 413beaa into master Oct 12, 2023
3 checks passed
@noppaz noppaz deleted the cache-dependency-path-input branch October 12, 2023 09:29
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