-
Notifications
You must be signed in to change notification settings - Fork 40
Update README within examples #251
Update README within examples #251
Conversation
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 don't think we want to promote that approach of defining applications with kedge!
I believe that we are supporting single-file and multiple-file. For a real-world example, it seems simpler to have it in one. Enterprise, etc would be good with multiple. Thoughts @kadel @pradeepto @containscafeine ? |
@containscafeine ah darn, I'll split them back up to multiple files then. |
cff011d
to
f5e4ad1
Compare
@containscafeine @surajssd gone ahead and updated the PR with multiple-files rather than a single kedge file. |
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!
before merge @cdrage can you update the commit to be according to changes we have made now ? |
Updates the README with full descriptions as well as change the default file extenssion to .yaml instead of .yml
f5e4ad1
to
9920332
Compare
@surajssd done |
This makes the examples simpler by making them one file as well as
updating the README's to a full description of how to use said example.