search

discord / erlpack

High Performance Erlang Term Format Packer
MIT License
220 stars 65 forks source link

Add Go support #31

Closed IAmJSD closed 4 years ago

IAmJSD commented 4 years ago

Fixes #29

zorkian commented 4 years ago

Hey Jake --

So first, thanks for writing this. Always happy to see people contributing back upstream.

Discord is actually moving away from Go internally, and we're down to the last handful of services, none of which need erlpack support. I want to make sure that anything Discord hosts on our repositories is actually maintained, reviewed, documented, etc etc, and unfortunately we aren't able to do that with Go at this time.

As a different solution, how would you feel about creating your own erlpack-go repository? I'm happy to update the README here and tell people where to find your repository if they need a Go library so they know where to find it. Would that work for you?

IAmJSD commented 4 years ago

Hi @zorkian,

That's completely understandable! I created https://github.com/JakeMakesStuff/go-erlpack last night so that I could work on erlpack integration, so that's the repository which I will maintain from now. 👍

zorkian commented 4 years ago

Thank you! I've updated our README to point to you repository :)

Going to close this PR now. Best of luck! (And of course, if you find any issues in the Python/C++/JS, we're happy to review & take those fixes!)