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

Update cortex vendoring #325

Merged
merged 2 commits into from
Mar 15, 2019
Merged

Conversation

gouthamve
Copy link
Member

This updates it to latest master.

@tomwilkie
Copy link
Contributor

I think you need to rebuild the protos and check them in.

@vtolstov
Copy link

@tomwilkie does it possible that cortex authors push proto generated files to it repo? it hard to check godoc and use it like this..
p.s. i'm write issue in it repo too.

Signed-off-by: Goutham Veeramachaneni <gouthamve@gmail.com>
Signed-off-by: Goutham Veeramachaneni <gouthamve@gmail.com>
@tomwilkie
Copy link
Contributor

@vtolstov Yeah we should do that.

@gouthamve gouthamve merged commit ddddd14 into grafana:master Mar 15, 2019
@gouthamve gouthamve deleted the update-vendoring branch March 15, 2019 12:59
periklis pushed a commit to periklis/loki that referenced this pull request Jul 16, 2024
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