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

Limit GHA to main and release branches #86

Merged
merged 1 commit into from
Jun 3, 2022

Conversation

arkodg
Copy link
Contributor

@arkodg arkodg commented Jun 1, 2022

Signed-off-by: Arko Dasgupta arko@tetrate.io

Signed-off-by: Arko Dasgupta <arko@tetrate.io>
@arkodg arkodg requested review from skriss and lizan June 1, 2022 17:11
@arkodg arkodg requested a review from skriss June 1, 2022 19:42
@@ -1,10 +1,16 @@
name: Build and Test
on:
push:
branches:
- "main"
- "release-v*"
Copy link
Member

Choose a reason for hiding this comment

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

release/* for consistency w/envoy

Copy link
Contributor Author

Choose a reason for hiding this comment

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

we've already started with release-v0.1.0 (https://github.com/envoyproxy/gateway/tree/release-v0.1.0)
@danehans any idea why we picked this convention ?

Copy link
Member

Choose a reason for hiding this comment

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

we can rename branches anyway

Copy link
Contributor

Choose a reason for hiding this comment

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

@arkodg I used Envoy as the model for Gateway release branch naming, substituting / with - since - is more widely used.

Copy link
Contributor Author

@arkodg arkodg Jun 2, 2022

Choose a reason for hiding this comment

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

cool thanks, I'll leave the PR as is, since the naming was intentional, we can change it, if we change the branch naming in the future

.github/workflows/build_and_test.yaml Show resolved Hide resolved
@arkodg arkodg merged commit f27e3b5 into envoyproxy:main Jun 3, 2022
lianghao208 added a commit to lianghao208/gateway that referenced this pull request Jun 3, 2022
Limit GHA to main and release branches (envoyproxy#86)
@danehans
Copy link
Contributor

danehans commented Jun 8, 2022

xref #63

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.

4 participants