connectrpc / connect-es

The TypeScript implementation of Connect: Protobuf RPC that works.
https://connectrpc.com/
Apache License 2.0
1.39k stars 82 forks source link

nixpkgs for protoc-gen-es and protoc-gen-connect-es #655

Open dezyh opened 1 year ago

dezyh commented 1 year ago

Is your feature request related to a problem? Please describe. As much as I like it, remote code generation is not permitted at my company. We also currently manage all dependencies through nixpkgs. Currently installing protoc-gen-es and protoc-gen-connect-es is a slightly manual process (not managed by nix shell) but through yarn add.

Describe the solution you'd like It would be much more convenient to have both protoc-gen-es and protoc-gen-connect-es available through nixpkgs.

Please specify whether the request is for Connect for Web or Connect for Node.js. Connect for Web (protoc-gen-es & protoc-gen-connect-es)

Additional Context There are currently a few nixpkgs such as buf, buf-language-server, protoc-gen-connect-go although I do not know if these are officially maintained.

If possible, I would be okay with trying to provide nixpkgs for the above binaries, but I do not personally have the time to maintain them fully (with new releases, etc). I don't personally think it's required that new releases to the above must be released to nixpkgs at the same time. A best effort basis would be sufficient.

If there's any interested, here is the documentation on creating nixpkgs.

timostamm commented 1 year ago

I believe that the other nixpks are community-maintained. Thankfully, @felschr opened a PR to add nixpks in https://github.com/NixOS/nixpkgs/pull/243432. Unfortunately, the PR ran into some of the intricacies of the npm ecosystem that were hopefully resolved.