-
Notifications
You must be signed in to change notification settings - Fork 1.7k
Tag template tagger #4567
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
Tag template tagger #4567
Conversation
Codecov Report
@@ Coverage Diff @@
## master #4567 +/- ##
==========================================
+ Coverage 72.61% 72.83% +0.21%
==========================================
Files 335 334 -1
Lines 12963 13037 +74
==========================================
+ Hits 9413 9495 +82
+ Misses 2957 2945 -12
- Partials 593 597 +4
Continue to review full report at Codecov.
|
Overall LGTM. |
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.
left a few comments but in general this is looking good
Co-authored-by: Nick Kubala <[email protected]>
Co-authored-by: Nick Kubala <[email protected]>
with the -beta.1
Related: Issue: #4371, #4550
Description
This includes the coding logic for the new tagging strategy
tagTemplate
. This code is only exercised in tests and will not be exposed to users until the following PR.tagTemplate
allows users to create their own tags by specifying a template composed of pre-existing tagging strategies. These pre-existing tagging strategies are stored as tagger objects under the components field and evaluated when the tag is generated.Follow-up Work
There will be a follow-up PR exposing this feature to users which includes changes to the Skaffold config as well as the necessary changes to any documentation.