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

Synchronize Makefile.common from prometheus/prometheus #717

Merged
merged 1 commit into from Feb 25, 2020

Conversation

prombot
Copy link
Contributor

@prombot prombot commented Feb 25, 2020

Propagating changes from master Makefile.common located in prometheus/prometheus.

Signed-off-by: prombot <prometheus-team@googlegroups.com>
@simonpasquier simonpasquier merged commit 673e4a1 into master Feb 25, 2020
@simonpasquier simonpasquier deleted the makefile_common branch February 25, 2020 08:38
@beorn7
Copy link
Member

beorn7 commented Feb 25, 2020

Thanks @simonpasquier . In general, as the maintainer of the repo, I'd like to have a say before something is merged, unless it is super urgent. Even if it looks like a trivial change, there might be a weirdness in a particular repo that will lead to problems in subtle ways.

@simonpasquier
Copy link
Member

@beorn7 sorry for stepping on your toes. This change was only affecting projects using Circle CI while client_golang is still using Travis.
Which bears the question: would you be against moving from Travis to Circle? My usual argument for this is consistency across projects and avoid maintaining 2 CI systems...

@beorn7
Copy link
Member

beorn7 commented Feb 25, 2020

That would be wonderful. Any help would be much appreciated.

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.

None yet

3 participants