Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix some typos #5352

Merged
merged 1 commit into from Feb 27, 2019

Conversation

Projects
None yet
4 participants
@huynq0911
Copy link
Contributor

huynq0911 commented Feb 25, 2019

What this PR does / why we need it:
Correct some words spelling for reading more easily.
File changed:

  • CONTRIBUTING.md
  • docs/chart_best_practices/rbac.md
  • docs/chart_template_guide/functions_and_pipelines.md
  • docs/chart_template_guide/helm_ignore_file.md
  • docs/charts.md
  • docs/charts_hooks.md
  • docs/charts_tips_and_tricks.md
  • docs/helm/helm_verify.md

Special notes for your reviewer:
N/A

If applicable:
N/A

Co-Authored-By: Nguyen Van Trung trungnvfet@outlook.com
Signed-off-by: Nguyen Quang Huy huynq0911@gmail.com

@helm-bot helm-bot added the size/S label Feb 25, 2019

@huynq0911 huynq0911 force-pushed the huynq0911:fix_some_typos branch from 7de5da2 to b639015 Feb 25, 2019

@helm-bot helm-bot added size/S and removed size/S labels Feb 25, 2019

@huynq0911 huynq0911 force-pushed the huynq0911:fix_some_typos branch from b639015 to 0dfb086 Feb 25, 2019

@helm-bot helm-bot added size/S and removed size/S labels Feb 25, 2019

@hickeyma
Copy link
Contributor

hickeyma left a comment

Small nit change

Show resolved Hide resolved docs/chart_best_practices/rbac.md Outdated
Fix some typos
Correct some words spelling for reading more easily.

Signed-off-by: Nguyen Quang Huy <huynq0911@gmail.com>
Co-Authored-By: Nguyen Van Trung <trungnvfet@outlook.com>

@huynq0911 huynq0911 force-pushed the huynq0911:fix_some_typos branch from 0dfb086 to caca656 Feb 27, 2019

@helm-bot helm-bot added size/S and removed size/S labels Feb 27, 2019

@hickeyma
Copy link
Contributor

hickeyma left a comment

LGTM, thanks.

@bacongobbler bacongobbler merged commit 63c970c into helm:master Feb 27, 2019

2 checks passed

ci/circleci: build Your tests passed on CircleCI!
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.