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

Feature/add prometheus exporter #40

Merged

Conversation

t9zx
Copy link

@t9zx t9zx commented Jul 11, 2020

Hello danielwippermann,

about 10 months ago I forked resol-vbos and added a Prometheus exporter to the json-live-data-server. Since then it's running stable. Feel free to merge it back into master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 89.629% when pulling f294c82 on t9zx:feature/add_prometheus_exporter into 34acee6 on danielwippermann:master.

@danielwippermann danielwippermann merged commit c3cc0b6 into danielwippermann:master Jul 11, 2020
@danielwippermann
Copy link
Owner

Hi @t9zx!

Welcome and thank you very much for this pull request. Would you mind if I add your name to the list of contributors in the README?

Stay safe and healthy,
Daniel

@t9zx
Copy link
Author

t9zx commented Jul 17, 2020

Hello DanielWippermann,

thanks for merging and asking about adding me to the list of contributors. Feel free to add me.

Best regards

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