Skip to content
This repository was archived by the owner on Jul 16, 2021. It is now read-only.

Update Configuring chart repositories section#342

Merged
prydonius merged 1 commit intohelm:masterfrom
Locus99:patch-1
Aug 23, 2017
Merged

Update Configuring chart repositories section#342
prydonius merged 1 commit intohelm:masterfrom
Locus99:patch-1

Conversation

@Locus99
Copy link
Contributor

@Locus99 Locus99 commented Aug 17, 2017

I believe the first line after the cat command should have a colon after the word 'api'. Otherwise, it is not proper yaml format.

I believe the first line after the cat command should have a colon after the word 'api'.  Otherwise, it is not proper yaml format.
@Locus99 Locus99 closed this Aug 17, 2017
@codecov-io
Copy link

codecov-io commented Aug 17, 2017

Codecov Report

Merging #342 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #342   +/-   ##
=======================================
  Coverage   88.41%   88.41%           
=======================================
  Files          17       17           
  Lines         820      820           
=======================================
  Hits          725      725           
  Misses         61       61           
  Partials       34       34

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f91d4e5...cb689d4. Read the comment docs.

@prydonius
Copy link
Member

prydonius commented Aug 17, 2017

@Locus99 you're right, thanks! How come you closed this?

@Locus99 Locus99 reopened this Aug 22, 2017
@Locus99
Copy link
Contributor Author

Locus99 commented Aug 22, 2017

Did not mean to close this, @prydonius. Lack of experience on my part.

@prydonius
Copy link
Member

No problem @Locus99, thanks again :)

@prydonius prydonius merged commit 1f0a217 into helm:master Aug 23, 2017
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