fast_xs icon indicating copy to clipboard operation
fast_xs copied to clipboard

Implement JRuby support

Open ChrisBr opened this issue 7 years ago • 9 comments

This is a first approach to implement https://github.com/brianmario/fast_xs/issues/6

Please note that there is one test failing. Also please note that I haven't benchmarked the implementation yet. I'm open to suggestions what would be the best approach here.

ChrisBr avatar Feb 27 '18 21:02 ChrisBr

Has this been superseded by another merge?

headius avatar May 17 '21 20:05 headius

Hey Charlie, not sure but I don't think so. Just thought as it was open for ~2 years now without much progress close it. There was also not much feedback from the maintainer, was there?

ChrisBr avatar May 17 '21 23:05 ChrisBr

@ChrisBr Ahh ok, I understand.

Yeah, would still be nice to get this in, but perhaps @brianmario is not interested?

@brianmario Can we revisit this?

headius avatar May 19 '21 19:05 headius

Hello!

Really sorry, I've fallen very behind on pretty much all of my OSS projects. Tbh I don't really have much time to code these days - I quit "tech" and am a grape farmer now 🧑‍🌾.

That said, I don't think I should need to hold this up any longer.

@headius 👋 I can go ahead and give you contrib access if you wouldn't mind taking it from here?

brianmario avatar May 19 '21 21:05 brianmario

Seems like if nothing else the travis config should be updated to build on modern Ruby versions to get the builds happy(er). I don't think folks are still running on 1.8.7 anymore right?

brianmario avatar May 19 '21 21:05 brianmario

@brianmario Sure, I can help get this integrated. Green CI would clearly be a first step. Maybe a separate PR to drop a few really old versions from CI?

headius avatar May 20 '21 15:05 headius

Ping again... I missed an invite to collaborate, but it would still be nice to get JRuby support in here.

headius avatar Dec 02 '21 15:12 headius

@headius hello again! The collaborator invite is still pending so you should be able to accept it still.

brianmario avatar Dec 02 '21 20:12 brianmario

Seems to have failed again... try again?

headius avatar Dec 03 '21 15:12 headius