Skip to content

docs: quote helm flags with brackets#11922

Merged
nebril merged 1 commit intomasterfrom
pr/quote-etcd-endpoints
Jun 8, 2020
Merged

docs: quote helm flags with brackets#11922
nebril merged 1 commit intomasterfrom
pr/quote-etcd-endpoints

Conversation

@nebril
Copy link
Copy Markdown
Member

@nebril nebril commented Jun 5, 2020

Due to zsh using square brackets for globbing/pattern matching, our etcd
endpoints examples failed with
zsh: no matches found: global.etcd.endpoints[0]=url.

Grepping through the docs didn't find any other commands using brackets,
so only etcd-endpoint related ones are in quotes.

Due to zsh using square brackets for globbing/pattern matching, our etcd
endpoints examples failed with
`zsh: no matches found: global.etcd.endpoints[0]=url`.

Grepping through the docs didn't find any other commands using brackets,
so only etcd-endpoint related ones are in quotes.

Signed-off-by: Maciej Kwiek <maciej@isovalent.com>
@nebril nebril requested a review from a team as a code owner June 5, 2020 13:38
@maintainer-s-little-helper
Copy link
Copy Markdown

Please set the appropriate release note label.

@maintainer-s-little-helper
Copy link
Copy Markdown

Please set the appropriate release note label.

@nebril nebril mentioned this pull request Jun 5, 2020
20 tasks
@nebril nebril added the release-note/misc This PR makes changes that have no direct user impact. label Jun 5, 2020
@coveralls
Copy link
Copy Markdown

coveralls commented Jun 5, 2020

Coverage Status

Coverage decreased (-0.006%) to 36.956% when pulling 885ba58 on pr/quote-etcd-endpoints into 6bd5b9c on master.

@nebril nebril added the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label Jun 8, 2020
@nebril nebril merged commit f743919 into master Jun 8, 2020
@nebril nebril deleted the pr/quote-etcd-endpoints branch June 8, 2020 13:28
@aanm aanm mentioned this pull request Jun 9, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready-to-merge This PR has passed all tests and received consensus from code owners to merge. release-note/misc This PR makes changes that have no direct user impact.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants