Skip to content

doc: Update samples README.md#132

Merged
tungbq merged 3 commits intotungbq:mainfrom
Chrisgr1:patch-1
Mar 18, 2024
Merged

doc: Update samples README.md#132
tungbq merged 3 commits intotungbq:mainfrom
Chrisgr1:patch-1

Conversation

@Chrisgr1
Copy link
Contributor

@Chrisgr1 Chrisgr1 commented Mar 18, 2024

Hi Tung, does this answer the ticket?

Add the sample code from samples/run_sample.sh to samples/README.md #96

Also, should I add the installation to the script? (from https://charts.bitnami.com/) I guess something like:
helm install sample-release bitnami/| grep jenkins

Closes: #96

Changes

  • DescribeYourChangeHere

Hi Tung, does this answer the ticket?

Add the sample code from samples/run_sample.sh to samples/README.md tungbq#96

Also, should I add the installation to the script? (from https://charts.bitnami.com/)
I guess something like:
helm install sample-release bitnami/| grep jenkins
@Chrisgr1 Chrisgr1 requested a review from tungbq as a code owner March 18, 2024 11:29
@tungbq
Copy link
Owner

tungbq commented Mar 18, 2024

Thanks for the first PR in this repository @Chrisgr1. Appreciated, just minor comment about the heading format, could you please check and fix it?

Chrisgr1 and others added 2 commits March 18, 2024 12:26
corrected formatting of kubectl

Co-authored-by: Tung Leo <[email protected]>
Correct formatting on Helm section

Co-authored-by: Tung Leo <[email protected]>
Copy link
Contributor Author

@Chrisgr1 Chrisgr1 left a comment

Choose a reason for hiding this comment

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

Updated to correct format

@tungbq tungbq changed the title Update README.md doc: Update samples README.md Mar 18, 2024
Copy link
Owner

@tungbq tungbq left a comment

Choose a reason for hiding this comment

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

Thanks @Chrisgr1 it looks good now!

@tungbq tungbq added the doc label Mar 18, 2024
@tungbq tungbq merged commit 277b31d into tungbq:main Mar 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add the sample code from samples/run_sample.sh to samples/README.md

2 participants