Skip to content

Modest doc improvements - #1876

Merged
SuperQ merged 2 commits into
prometheus:masterfrom
anthonyeleven:anthonyeleven/doc-tweaks
Nov 25, 2020
Merged

SuperQ merged 2 commits into
prometheus:masterfrom
anthonyeleven:anthonyeleven/doc-tweaks

Conversation

@anthonyeleven

Copy link
Copy Markdown
Contributor

Get my feet wet by tweaking some wording and adding some guidance wrt collectors that are disabled by default.
@SuperQ @discordianfish

Signed-off-by: Anthony D'Atri anthony.datri@gmail.com

@anthonyeleven
anthonyeleven force-pushed the anthonyeleven/doc-tweaks branch from 23190c5 to f112760 Compare October 23, 2020 18:03
Signed-off-by: Anthony D'Atri <anthony.datri@gmail.com>
@anthonyeleven
anthonyeleven force-pushed the anthonyeleven/doc-tweaks branch from f112760 to bb0a97f Compare October 23, 2020 18:12
@anthonyeleven
anthonyeleven force-pushed the anthonyeleven/doc-tweaks branch from 65d22a3 to 1782fc2 Compare November 6, 2020 21:39
@anthonyeleven

Copy link
Copy Markdown
Contributor Author

@SuperQ @discordianfish The checks were a bit flaky but this PR has now passed and is ready to merge. PTAL.

@discordianfish discordianfish left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Nice, definitely an improvement, thanks!

@anthonyeleven

Copy link
Copy Markdown
Contributor Author

Nice, definitely an improvement, thanks!

You bet, happy to contribute. Is there more to be done for this to be merged?

Comment thread README.md Outdated
Comment thread README.md Outdated
Comment thread docs/TIME.md Outdated
@SuperQ

SuperQ commented Nov 25, 2020

Copy link
Copy Markdown
Member

Nice, thanks for the improvements. I've add a few suggestions.

@anthonyeleven
anthonyeleven force-pushed the anthonyeleven/doc-tweaks branch from 1782fc2 to d22ae70 Compare November 25, 2020 08:11
Signed-off-by: Anthony D'Atri <anthony.datri@gmail.com>
@anthonyeleven

anthonyeleven commented Nov 25, 2020

Copy link
Copy Markdown
Contributor Author

Nice, thanks for the improvements. I've add a few suggestions.

You're welcome, Prometheus and node_exporter have been invaluable for me and I'm happy to contribute. I've pushed a revised commit, please take a look.

@SuperQ SuperQ left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@SuperQ
SuperQ merged commit 8b46636 into prometheus:master Nov 25, 2020
oblitorum pushed a commit to shatteredsilicon/node_exporter that referenced this pull request Apr 9, 2024
* Modest doc improvements

Signed-off-by: Anthony D'Atri <anthony.datri@gmail.com>
oblitorum pushed a commit to shatteredsilicon/node_exporter that referenced this pull request Apr 9, 2024
* Modest doc improvements

Signed-off-by: Anthony D'Atri <anthony.datri@gmail.com>
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.

3 participants