Skip to content
This repository has been archived by the owner on Jan 11, 2024. It is now read-only.

Adding K3d installation instructions. #1132

Merged
merged 7 commits into from
Mar 21, 2022
Merged

Conversation

subhamgcon
Copy link
Contributor

Solves issue #1121.

Signed-off by Subham Ghosh: subhamgcon@gmail.com

subhamgcon and others added 2 commits March 19, 2022 09:52
Co-authored-by: Meg McRoberts <mmcroberts@cloudbees.com>
Co-authored-by: Meg McRoberts <mmcroberts@cloudbees.com>
Copy link
Contributor

@StackScribe StackScribe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A few editorial suggestions but, in general, lgtm.

@subhamgcon
Copy link
Contributor Author

A few editorial suggestions but, in general, lgtm.

Thank you for reviewing! I have committed your suggested changes. Is it ready to be merged then?

Co-authored-by: Meg McRoberts <mmcroberts@cloudbees.com>
@StackScribe
Copy link
Contributor

It needs review by a technical person but I approved it.

@subhamgcon
Copy link
Contributor Author

@oleg-nenashev. Can you review it? Thank you!

Copy link
Member

@oleg-nenashev oleg-nenashev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! Needs a pull request for 0.14.x, Maybe we could also restructure this part and start adding top-level installation guide with version-unspecific guidelines

Copy link
Contributor Author

@subhamgcon subhamgcon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you please explain the requested changes? Also, should I start working on a PR for 0.14.x? @oleg-nenashev

@oleg-nenashev
Copy link
Member

should I start working on a PR for 0.14.x?

You can update it in the same pull request. I think we can try reworking it later as experiment together with @StackScribe

@subhamgcon
Copy link
Contributor Author

@oleg-nenashev Added the instructions in 0.14.x.

Copy link
Member

@oleg-nenashev oleg-nenashev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! I think it is a good improvement

@oleg-nenashev oleg-nenashev merged commit 8228c7d into keptn:master Mar 21, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants