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: workflows go mod cache #982

Merged
merged 1 commit into from
Aug 10, 2022
Merged

Conversation

iyear
Copy link
Member

@iyear iyear commented Aug 10, 2022

Signed-off-by: iyear ljyngup@gmail.com

Pre-Checklist

Note: please complete ALL items in the following checklist.

  • I have read through the CONTRIBUTING.md documentation.
  • My code has the necessary comments and documentation (if needed).
  • I have added relevant tests

Description

actions/setup-go@v3 has a built-in caching, and I noticed that the current pr.yml cache is useless. I replaced them with a simpler form.

https://github.com/actions/setup-go#caching-dependency-files-and-build-outputs

Related Issues

close #981

Signed-off-by: iyear <ljyngup@gmail.com>
@iyear iyear requested a review from daniel-hutao as a code owner August 10, 2022 09:28
@daniel-hutao
Copy link
Member

@iyear Good job!

Please take a look if it works after this pr merged.

@daniel-hutao daniel-hutao merged commit d223ced into devstream-io:main Aug 10, 2022
@iyear
Copy link
Member Author

iyear commented Aug 10, 2022

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.

🍀 Proposal: Optimize workflows go cache
2 participants