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

Add documentation about TTL and GH issues #208

Merged
merged 1 commit into from
Oct 2, 2018
Merged

Add documentation about TTL and GH issues #208

merged 1 commit into from
Oct 2, 2018

Conversation

beorn7
Copy link
Member

@beorn7 beorn7 commented Oct 2, 2018

  • Add an issue template similar to other repos.
  • Add TTL for metrics as a non-goal.

@juliusv inspired by yet another follow-up to closed #19

Signed-off-by: beorn7 [email protected]

@brian-brazil
Copy link
Contributor

👍

## Feature request
**Use case. Why is this important?**

*Nice to have' is not a good use case :)*
Copy link
Member

Choose a reason for hiding this comment

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

There's a single quote / apostrophe after "have" that doesn't have a corresponding opening quote?

Copy link
Member Author

Choose a reason for hiding this comment

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

Yeah, copy&paste from the prometheus/prometheus issue template. I fixed it there, too: prometheus/prometheus#4688


* Pushgateway command line:

insert full command line
Copy link
Member

Choose a reason for hiding this comment

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

This line has tab indentation, the others have spaces


* Logs:
```
insert Puhsgateway logs relevant to the issue here
Copy link
Member

Choose a reason for hiding this comment

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

Puhsgateway -> Pushgateway

Copy link
Member Author

Choose a reason for hiding this comment

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

fixed

@juliusv
Copy link
Member

juliusv commented Oct 2, 2018

👍 besides nits

- Add an issue template similar to other repos.
- Add TTL for metrics as a non-goal.

Signed-off-by: beorn7 <[email protected]>
@beorn7
Copy link
Member Author

beorn7 commented Oct 2, 2018

Thanks for the usual thorough spelling check. :o)

All fixed, merging now.

@beorn7 beorn7 merged commit 44fe56d into master Oct 2, 2018
@beorn7 beorn7 deleted the beorn7/doc branch October 2, 2018 12:28
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.

Allow timeout for metrics
3 participants