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
Mark YamlTemplate as GA #1531
Mark YamlTemplate as GA #1531
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #1531 +/- ##
============================================
+ Coverage 40.80% 40.93% +0.12%
- Complexity 2832 2833 +1
============================================
Files 743 742 -1
Lines 43166 43072 -94
Branches 4609 4596 -13
============================================
+ Hits 17615 17630 +15
+ Misses 24044 23937 -107
+ Partials 1507 1505 -2
|
84517c9
to
a558465
Compare
@@ -24,12 +24,21 @@ | |||
name = "Yaml_Template", | |||
category = TemplateCategory.GET_STARTED, | |||
type = Template.TemplateType.YAML, | |||
displayName = "YAML Template (Experimental)", | |||
displayName = "YAML Template", |
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.
Can we avoid all the append logic by just making this YAML
? Is there any reason not to do that?
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.
No, I just didn't like how it looked on the @Template
tag, but idc that much - fixed.
2a9ae57
to
9b17e29
Compare
Signed-off-by: Jeffrey Kinard <jeff@thekinards.com>
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.
LGTM
b20548c
into
GoogleCloudPlatform:main
No description provided.