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

[jaeger-operator] fix config table in jaeger-operator README.md #507

Merged
merged 2 commits into from
Oct 6, 2023
Merged

[jaeger-operator] fix config table in jaeger-operator README.md #507

merged 2 commits into from
Oct 6, 2023

Conversation

mavbozo
Copy link
Contributor

@mavbozo mavbozo commented Oct 4, 2023

What this PR does

Configuration table display is broken in jaeger-operator README.md. This PR fixes the markdown table formatting.

Checklist

  • [ V] DCO signed
  • [ V] Commits are GPG signed
  • Chart Version bumped
  • [ V] Title of the PR starts with chart name ([jaeger] or [jaeger-operator])
  • README.md has been updated to match version/contain new values

Signed-off-by: mavbozo <maverickbozo@me.com>
@batazor
Copy link
Member

batazor commented Oct 4, 2023

@mavbozo That's great. Thank you. Can we do a left alignment? I think that would be a more appropriate option.

Signed-off-by: mavbozo <maverickbozo@me.com>
@mavbozo
Copy link
Contributor Author

mavbozo commented Oct 6, 2023

@batazor Got it. I've made the table content left aligned.

@batazor batazor merged commit 4200a33 into jaegertracing:main Oct 6, 2023
2 of 3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants