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

Convert config properties to list-table in Prometheus documentation #21658

Merged
merged 1 commit into from
Apr 22, 2024

Conversation

ebyhr
Copy link
Member

@ebyhr ebyhr commented Apr 22, 2024

Description

Screenshot 2024-04-22 at 22 35 21

Release notes

(x) This is not user-visible or is docs only, and no release notes are required.

@cla-bot cla-bot bot added the cla-signed label Apr 22, 2024
@github-actions github-actions bot added the docs label Apr 22, 2024
@ebyhr ebyhr force-pushed the ebi/prometheus-config branch from d8aa565 to 76deb99 Compare April 22, 2024 21:58
@ebyhr ebyhr requested review from wendigo, mosabua and hashhar April 22, 2024 22:01
Copy link
Member

@mosabua mosabua left a comment

Choose a reason for hiding this comment

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

Looks good as a pure source reformat. If you want, you could also drop the third columns and inline the default values into the second column. That can sometimes work better in terms of rendering, especially if only a few values have defaults. In this case both could work.

If you want to also reword the descriptions I can provide feedback, but this PR is good to go as it stands as well.

@ebyhr
Copy link
Member Author

ebyhr commented Apr 22, 2024

Let me merge as-is. Please feel free to send the follow-up PR.

@ebyhr ebyhr merged commit 2a2ef11 into trinodb:master Apr 22, 2024
9 checks passed
@ebyhr ebyhr deleted the ebi/prometheus-config branch April 22, 2024 23:02
@github-actions github-actions bot added this to the 446 milestone Apr 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

2 participants