feat(go-sdk): streaming list objects endpoint#651
Conversation
|
Note Other AI code review bot(s) detectedCodeRabbit has detected other AI code review bot(s) in this pull request and will avoid duplicating their findings in the review comments. This may lead to a less comprehensive review. WalkthroughThe Makefile's Go client build process was modified to use Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Pre-merge checks and finishing touches❌ Failed checks (1 warning)
✅ Passed checks (2 passed)
✨ Finishing touches
🧪 Generate unit tests (beta)
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. Comment |
There was a problem hiding this comment.
Pull Request Overview
This PR enables the streaming list objects endpoint for the Go SDK by modifying the build process to use a streaming-specific client generator target instead of the standard one.
Key Changes:
- Modified the Go SDK build target to use
build-client-streamedinstead ofbuild-client
There was a problem hiding this comment.
Actionable comments posted: 1
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
Makefile(1 hunks)
🧰 Additional context used
📓 Path-based instructions (1)
Makefile
📄 CodeRabbit inference engine (.github/copilot-instructions.md)
Makefile: Update OPEN_API_REF in the Makefile when targeting new OpenFGA API versions
Update Docker image tags in the Makefile when upgrading build tools
Use official, tagged Docker images for containerized builds
Files:
Makefile
Description
This was used to generate the API layer changes for the streaming list objects endpoint
What problem is being solved?
For the issue #76 in the go-sdk
generates openfga/go-sdk#252
How is it being solved?
What changes are made to solve it?
References
Review Checklist
mainSummary by CodeRabbit