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

[VAULT-20630] CI: Use 'ref' (not 'base_ref') as a default git reference to check out code in the test-go GHA workflow #23458

Merged

Conversation

kubawi
Copy link
Contributor

@kubawi kubawi commented Oct 3, 2023

A companion PR to #23453.

This one changes the default value of the checkout-ref input in the test-go.yml workflow (from github.base_ref to github.ref). Currently, nothing (that I am aware of) uses that default, but it could bite us in the future.

…ce to check out code in the test-go GHA workflow
@vercel
Copy link

vercel bot commented Oct 3, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Ignored Deployment
Name Status Preview Updated (UTC)
vault ⬜️ Ignored (Inspect) Oct 3, 2023 1:38pm

@github-actions github-actions bot added the hashicorp-contributed-pr If the PR is HashiCorp (i.e. not-community) contributed label Oct 3, 2023
@github-actions
Copy link

github-actions bot commented Oct 3, 2023

Build Results:
All builds succeeded! ✅

@github-actions
Copy link

github-actions bot commented Oct 3, 2023

CI Results:
All Go tests succeeded! ✅

@kubawi kubawi merged commit 3acfe5e into main Oct 3, 2023
@kubawi kubawi deleted the kubawi/VAULT-20630/dont-use-base-ref-as-a-default-for-go-tests branch October 3, 2023 13:58
kubawi added a commit that referenced this pull request Oct 3, 2023
…ce to check out code in the test-go GHA workflow (#23458)
kubawi added a commit that referenced this pull request Oct 3, 2023
…ce to check out code in the test-go GHA workflow (#23458)
kubawi added a commit that referenced this pull request Oct 3, 2023
…ce to check out code in the test-go GHA workflow (#23458) (#23459)

Co-authored-by: Kuba Wieczorek <[email protected]>
kubawi added a commit that referenced this pull request Oct 3, 2023
…ce to check out code in the test-go GHA workflow (#23458) (#23469)

Co-authored-by: Kuba Wieczorek <[email protected]>
hellobontempo pushed a commit that referenced this pull request Oct 3, 2023
…ce to check out code in the test-go GHA workflow (#23458)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci hashicorp-contributed-pr If the PR is HashiCorp (i.e. not-community) contributed pr/no-changelog pr/no-milestone
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants