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

Recommend to get promtool from a binary distribution. #8491

Merged
merged 1 commit into from
Feb 15, 2021

Conversation

beorn7
Copy link
Member

@beorn7 beorn7 commented Feb 15, 2021

Rather than compile it yourself, which doesn't work as shown anymore
because of Go Modules.

Rather than compile it yourself, which doesn't work as shown anymore
because of Go Modules.

Signed-off-by: beorn7 <beorn@grafana.com>
@beorn7 beorn7 requested a review from roidelapluie February 15, 2021 21:00
@roidelapluie roidelapluie merged commit eae197c into master Feb 15, 2021
@roidelapluie roidelapluie deleted the beorn7/doc branch February 15, 2021 21:06
@roidelapluie
Copy link
Member

Thanks!

@roidelapluie
Copy link
Member

Should we cherry-pick it in release-2.25?

@beorn7
Copy link
Member Author

beorn7 commented Feb 15, 2021

I think it's just a minor annoyance, but your call.

@roidelapluie
Copy link
Member

Please cherry pick. It is not minor since it happens at the moment we remove the vendor directory.

@beorn7
Copy link
Member Author

beorn7 commented Feb 15, 2021

Good point.

@beorn7
Copy link
Member Author

beorn7 commented Feb 15, 2021

#8493

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