-
Notifications
You must be signed in to change notification settings - Fork 8
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
Only release semver tags #62
Conversation
We need to create multiple tags for the same commit so that go can resolve the webhook module. We don't need to run CI on those tags. Otherwise we'll get an error from the release script like https://github.com/vmware-tanzu/cartographer-conventions/runs/5684344434?check_suite_focus=true Signed-off-by: Scott Andrews <andrewssc@vmware.com>
tested with a smattering of tags against my fork. These ran CI:
These did not run CI:
|
.github/workflows/ci.yaml
Outdated
@@ -6,7 +6,8 @@ on: | |||
- 'main' | |||
- '!dependabot/**' | |||
tags: | |||
- '**' | |||
# semver tags | |||
- 'v[0-9]+.[0-9]+.[0-9]+-?**' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Period characters in a regular expression can match any character. You should prefix these periods with a backslash (i.e.: \.
) to indicate that you really do want periods to separate the components in the semver string.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
good catch, updated
Signed-off-by: Scott Andrews <andrewssc@vmware.com>
Codecov Report
@@ Coverage Diff @@
## main #62 +/- ##
=======================================
Coverage ? 70.18%
=======================================
Files ? 17
Lines ? 909
Branches ? 0
=======================================
Hits ? 638
Misses ? 251
Partials ? 20 Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks better now. Thank you.
We need to create multiple tags for the same commit so that go can
resolve the webhook module. We don't need to run CI on those tags.
Otherwise we'll get an error from the release script like
https://github.com/vmware-tanzu/cartographer-conventions/runs/5684344434?check_suite_focus=true
Signed-off-by: Scott Andrews andrewssc@vmware.com