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

Update Go-Getter test workflow to run on pull requests #417

Merged
merged 1 commit into from
Feb 14, 2023

Conversation

nywilken
Copy link
Contributor

@nywilken nywilken commented Feb 13, 2023

Currently test for v2 run for authors who have push access to the go-getter repo. This change updates the test action to run for all open pull-request to the v2 branch, as well as on pushes to the v2 branch.

Copy link

@lbajolet-hashicorp lbajolet-hashicorp left a comment

Choose a reason for hiding this comment

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

LGTM

push:
branches:
- v2
pull_request:

Choose a reason for hiding this comment

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

Out of curiosity what does this empty section imply?

Choose a reason for hiding this comment

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

I presume it means that it runs on all PRs pushing to v2?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ah good call out. This will actually run for all pull requests. I can update for any pull_requests against v2

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Updated

@nywilken nywilken merged commit d5cdddb into v2 Feb 14, 2023
@nywilken nywilken deleted the nywilken/run-tests-on-prs branch February 14, 2023 21:04
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