Remove support for "single file" app definition #608
Conversation
Use of such a multi-document yaml file with required ordering is mostly confusing Signed-off-by: Nicolas De Loof <nicolas.deloof@gmail.com>
Codecov Report
@@ Coverage Diff @@
## master #608 +/- ##
==========================================
- Coverage 72.86% 71.44% -1.43%
==========================================
Files 53 52 -1
Lines 2849 2651 -198
==========================================
- Hits 2076 1894 -182
- Misses 504 527 +23
+ Partials 269 230 -39
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.
I think that everything linked to
Line 32 in 04c3a28
func YamlSingleFileSeparator(hasCRLF bool) []byte { |
Signed-off-by: Nicolas De Loof <nicolas.deloof@gmail.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 👍
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
- What I did
Removed support for single file docker app definition. Only can use
.dockerapp
folder with dedicated yaml files now- How I did it
Mostly code removal
- How to verify it
all non single-file specific tests are preserved
- Description for the changelog
"Single file" (multi document yaml) docker app definition is not supported anymore.
skip
andmerge
commands as well as--single-file
option have been removed- A picture of a cute animal (not mandatory but encouraged)