Skip to content

Comments

feat: add protoc-gen formulas#5

Merged
yordis merged 1 commit intomainfrom
add-protoc-gen-trogon
Jan 17, 2026
Merged

feat: add protoc-gen formulas#5
yordis merged 1 commit intomainfrom
add-protoc-gen-trogon

Conversation

@yordis
Copy link
Member

@yordis yordis commented Jan 17, 2026

Signed-off-by: Yordis Prieto yordis.prieto@gmail.com

@cursor
Copy link

cursor bot commented Jan 17, 2026

PR Summary

Introduces Homebrew support for installing two protoc-gen plugins across macOS (arm64/intel) and Linux (arm64/x86_64).

  • Adds Formula/protoc-gen-connect-go-servicestruct.rb (v0.2.0) with per-arch URLs/sha256, installs protoc-gen-connect-go-servicestruct, and includes a --version test
  • Adds Formula/protoc-gen-elixir-grpc.rb (v0.4.2) with per-arch URLs/sha256, installs protoc-gen-elixir-grpc, and includes a --version test

Written by Cursor Bugbot for commit 290ff7b. This will update automatically on new commits. Configure here.

@coderabbitai
Copy link

coderabbitai bot commented Jan 17, 2026

Walkthrough

Adds two new Homebrew formulae for protoc plugins: protoc-gen-connect-go-servicestruct and protoc-gen-elixir-grpc, each declaring metadata, platform-specific macOS and Linux binaries with SHA256 checksums, an install step placing the binary into bin, and a test that runs --version.

Changes

Cohort / File(s) Summary
New Homebrew Formula: connect-go-servicestruct
Formula/protoc-gen-connect-go-servicestruct.rb
Adds ProtocGenConnectGoServicestruct formula with metadata (desc, homepage, license, version), platform-specific download URLs and SHA256 checksums for macOS (arm64, intel) and Linux (arm64, x86_64), install copying the binary to bin, and a test invoking --version.
New Homebrew Formula: elixir-grpc
Formula/protoc-gen-elixir-grpc.rb
Adds ProtocGenElixirGrpc formula with metadata, platform-specific URLs and SHA256 checksums for macOS (arm64, intel) and Linux (arm64, x86_64), install copying the binary to bin, and a test invoking --version.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

🐰 Two formulas hopped into the brew,
One for connect, one for gRPC too,
Checksums snug for mac and linux lands,
Binaries placed by careful hands,
Hooray — plugins ready, fresh and new! 🎉

🚥 Pre-merge checks | ✅ 1 | ❌ 2
❌ Failed checks (1 warning, 1 inconclusive)
Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
Description check ❓ Inconclusive The description contains only a sign-off line and provides no meaningful information about the changeset or its purpose. Add a brief description of what these formulas do and why they are being added to help reviewers understand the change.
✅ Passed checks (1 passed)
Check name Status Explanation
Title check ✅ Passed The title 'feat: add protoc-gen formulas' accurately and concisely describes the main change: adding two new Homebrew formulas (protoc-gen-connect-go-servicestruct and protoc-gen-elixir-grpc).

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@yordis yordis marked this pull request as ready for review January 17, 2026 03:54
Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

🤖 Fix all issues with AI agents
In `@Formula/protoc-gen-elixir-grpc.rb`:
- Around line 7-17: The formula currently only defines macOS platforms using the
on_macos block with nested on_arm64 and on_intel; add an on_linux block
mirroring that structure and include nested on_arm64 and on_intel blocks that
set the URL to the matching Linux release files
(protoc-gen-elixir-grpc@v0.4.2_Linux_arm64.tar.gz and
protoc-gen-elixir-grpc@v0.4.2_Linux_x86_64.tar.gz) and provide the corresponding
sha256 values; specifically modify the existing on_macos section near the
on_arm64 and on_intel entries and add an on_linux do ... end block with the same
pattern of url "..." and sha256 "..." entries so Linux arm64 and x86_64 binaries
are installed.

Signed-off-by: Yordis Prieto <yordis.prieto@gmail.com>
@yordis yordis force-pushed the add-protoc-gen-trogon branch from 5a4f735 to 290ff7b Compare January 17, 2026 04:11
@yordis yordis merged commit bbbb65e into main Jan 17, 2026
4 checks passed
@yordis yordis deleted the add-protoc-gen-trogon branch January 17, 2026 04:17
@sht-bot sht-bot mentioned this pull request Jan 17, 2026
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.

1 participant