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

workflows: Move actions/checkout before actions/setup-go #918

Merged
merged 1 commit into from
Sep 6, 2023

Conversation

anoopcs9
Copy link
Collaborator

@anoopcs9 anoopcs9 commented Sep 6, 2023

By placing actions/checkout before actions/setup-go(as suggested in actions/setup-go#281) we could avoid the following warning message:

Warning: Restore cache failed: Dependencies file is not found in /home/runner/work/go-ceph/go-ceph. Supported file pattern: go.sum

fixes #917

@anoopcs9 anoopcs9 added the no-API This PR does not include any changes to the public API of a go-ceph package label Sep 6, 2023
By placing actions/checkout before actions/setup-go[1] we could avoid
the following warning message:

Warning: Restore cache failed: Dependencies file is not found in
/home/runner/work/go-ceph/go-ceph. Supported file pattern: go.sum

[1] actions/setup-go#281

Signed-off-by: Anoop C S <[email protected]>
@anoopcs9 anoopcs9 force-pushed the avoid-setupgo-warning branch from 14ad094 to 16de665 Compare September 6, 2023 08:19
@anoopcs9 anoopcs9 marked this pull request as ready for review September 6, 2023 08:22
Copy link
Collaborator

@phlogistonjohn phlogistonjohn left a comment

Choose a reason for hiding this comment

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

LGTM

@mergify mergify bot merged commit 291244b into ceph:master Sep 6, 2023
@anoopcs9 anoopcs9 deleted the avoid-setupgo-warning branch September 11, 2023 10:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
no-API This PR does not include any changes to the public API of a go-ceph package
Projects
None yet
Development

Successfully merging this pull request may close these issues.

CI job 'check' emits a warning about go.mod
2 participants