Fix: cue structure implemented #6441
Conversation
kartikaysaxena
requested review from
barnettZQG,
wonderflow,
leejanee,
Somefive,
jefree-cat,
FogDong,
wangyikewxgm and
chivalryq
as code owners
kartikaysaxena
changed the title
|
@anoop2811 please take a look. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #6441 +/- ##
===========================================
+ Coverage 33.12% 61.51% +28.39%
===========================================
Files 141 178 +37
Lines 16402 23672 +7270
===========================================
+ Hits 5433 14562 +9129
+ Misses 10285 7679 -2606
- Partials 684 1431 +747
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Signed-off-by: Kartikay <kartikay_2101ce32@iitp.ac.in>
kartikaysaxena
force-pushed
the
fix-docs
branch
from
5878289
to
11e2fe2
Compare
| - "spec.template.spec.containers[0].image" | ||
| cue: | ||
| template: | | ||
| apiVersion: "apps/v1" |
There was a problem hiding this comment.
this should be wrapped in output.
| } | ||
| } | ||
| } | ||
| parameters: [ |
There was a problem hiding this comment.
parameters is not an array but a map.
| cue: | ||
| template: | | ||
| apiVersion: "apps/v1" | ||
| kind: "Deployment" |
There was a problem hiding this comment.
Can you take care of indenting and trailing whitespaces?
| cue: | ||
| template: | | ||
| apiVersion: "apps/v1" | ||
| kind: "Deployment" |
There was a problem hiding this comment.
Can you take care of indenting and trailing whitespaces?
Signed-off-by: Kartikay <kartikay_2101ce32@iitp.ac.in>
|
you can test and validate the result by |
|
Hi @kartikaysaxena , could you please add how you tested these changes? |
Fixes #6436
Implemented cue structure in the docs as per the issue.