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

Maintained fork #56

Open
LKaemmerling opened this issue Sep 11, 2020 · 8 comments
Open

Maintained fork #56

LKaemmerling opened this issue Sep 11, 2020 · 8 comments

Comments

@LKaemmerling
Copy link

LKaemmerling commented Sep 11, 2020

First of all, thank you @endclothing guys for maintaining this package after Jimdo.

Sadly it looks like this repository is not maintained anymore (last activity was in Feb.), so I forked it to https://github.com/promphp/prometheus_client_php. I just released v1.0.3 with a few minor fixes (most important is the removed Guzzle requirement).

I plan to continue to maintain the project, so I would recommend moving to this version. I use the Prometheus client myself in several projects, so you can expect to have an active maintainer with me! (Just as a side note, i work mostly for Open Source stuff @hetznercloud, so i will have time to maintain it)

Maybe @endclothing, if you don't plan to maintain it again, could you add a note to the readme?

@L3tum
Copy link

L3tum commented Sep 17, 2020

How did you remove the Guzzle dependency?

In the PushGateway.php in your repo it still clearly uses it?

@LKaemmerling
Copy link
Author

Yeah you are right Thank you!

I moved it to require-dev, because if was needed within the tests (but phpstorm didn't find the usage within the PushGateway). I will takle this next week when I am back from vacation. My first idea would be to remove the pushgateway support in the first place and add an additional package which's allows usage of the pushgateway, because not everyone needs the pushgateway support.

@L3tum
Copy link

L3tum commented Sep 18, 2020

I'd like to ask you to keep the PushGateway support, though doing so in a separate package would also be okay.

I'll see if I can see to do something on this next week, my work is probably okay with me dedicating some time to this as we're using it in a lot of projects.

@LKaemmerling
Copy link
Author

Hey @L3tum,

i "moved" the whole push-gateway support into a dedicated package: https://github.com/LKaemmerling/prometheus_push_gateway_php

Could you please test too? I plan to release a version (1.0.0 of the Push Gateway and 2.0.0 of the Prometheus client [because the removing is a breaking change]) within the next few days, when i don't encounter any other problems within my projects.

@L3tum
Copy link

L3tum commented Sep 21, 2020

@LKaemmerling Ah, cool! I'll test that when I have some time. Looks good though.

In the meantime I also forked your repo 😄 https://github.com/L3tum/prometheus_client_php
I've done some pretty extensive changes over the weekend to make it more streamline to what a project at my company would look like. The next step would be to throw out all those test cases and make some good ones that don't require spinning up 5 containers. Maybe I'll do a PR for your repo once I'm done

@LKaemmerling
Copy link
Author

Feel free to open a PR :) As i said, i plan to maintain an easy usable fork with as few as needed dependencies :) Maybe your use case might help others too, so feel free to contribute everything and i will try to find a good path between specific "use-cases improvements" and improvements that will help other too :)

@LKaemmerling
Copy link
Author

LKaemmerling commented Sep 22, 2020

Just for the record (cc @L3tum) i just published v2.0.0 of the fork without the PushGateway support. The Pushgateway client was also released as v1.0.0

composer require promphp/prometheus_client_php
composer require promphp/prometheus_push_gateway_php

@LKaemmerling
Copy link
Author

Hey @L3tum,

would you volunteer to maintain the new fork together with me (under https://github.com/PromPHP)?

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

No branches or pull requests

2 participants