lexmag / statix

Fast and reliable Elixir client for StatsD-compatible servers
https://hexdocs.pm/statix
ISC License
275 stars 82 forks source link

Project Support #41

Open jeffutter opened 5 years ago

jeffutter commented 5 years ago

Hi @lexmag ,

I was wondering if this project could use some additional help with support and maintenance. It looks like there have been no releases for a while and not much activity on the repo even with a handful of open PRs.

We use this library in a few projects at Bleacher Report and would be glad to help out where we can.

I would be in support of adding another maintainer or two, or moving the repo to an organization. Feel free to let me know how I can help or use this thread to discuss further.

sb8244 commented 5 years ago

I'd be happy to help out here as well.

mroach commented 5 years ago

@jeffutter Thanks for bringing this up. With issues like #42 that break Statix on the newest OTP, it's becoming more pressing to keep the project maintained. Otherwise, I suspect there will be lots of forking to patch it. It'd be handy to at a "fork of truth" that's maintained until this question is resolved. I'd wager that Bleacher Report would be a good candidate for that fork. Do you have one shared publicly?

jeffutter commented 5 years ago

@mroach We don't have our own fork. We're either using the hex release or https://github.com/rodrigues/statix/tree/events_checks. However, I can create a fork under our org that includes event_checks and fixes for Erlang 21.3.8.4. If that would be helpful, I can probably do that in the next couple of days.

sb8244 commented 5 years ago

My biggest concern with a fork is how it will handle inclusion in Instruments package. We'll need to probably keep the statix package name and set override: true in order to get it working there.

mroach commented 5 years ago

@jeffutter That'd be great to have a fork on the Bleacher Report org. Since statix is now broken, I think we'll see a ramp-up in users looking for a maintained package as they upgrade.

jeffutter commented 5 years ago

I've created a fork over at https://github.com/bleacherreport/statix that has a couple of commits merged in that we use. I also have a branch that fixes OTP-21.3.8.4 here: https://github.com/bleacherreport/statix/tree/otp-21.3.8.4-support I think it works, but it probably needs some cleanup and testing.

lexmag commented 5 years ago

Hi folks,

I apologise for not responding faster. I'll put as much as possible time during coming week to make PR merged and issues closed.