-
Notifications
You must be signed in to change notification settings - Fork 3
GO sdk unmarshel fix #129
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
GO sdk unmarshel fix #129
Conversation
|
Important Review skippedReview was skipped due to path filters ⛔ Files ignored due to path filters (1)
CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including You can disable this status message by setting the WalkthroughThe ConductorOne SDK Go dependency was updated from version v1.27.1 to v1.27.2-0.20260113184555-c2a4cbf81d4c in the go.mod file. No other code changes were made. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. Comment |
There was a problem hiding this 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 @go.mod:
- Line 14: Update the repository documentation and the commit message that
introduced the pseudo-version "github.com/conductorone/conductorone-sdk-go
v1.27.2-0.20260113184555-c2a4cbf81d4c" to explicitly state the bug or behavior
this unreleased commit fixes, add a short migration plan that states when/if you
will replace the pseudo-version with a stable v1.27.2 (or higher) release from
the SDK maintainers, and list any known limitations or workarounds; specifically
mention the module string "github.com/conductorone/conductorone-sdk-go
v1.27.2-0.20260113184555-c2a4cbf81d4c" in the commit message and in a top-level
changelog or docs entry along with an expected timeline and the steps to revert
to the canonical release once available.
📜 Review details
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
⛔ Files ignored due to path filters (3)
go.sumis excluded by!**/*.sumand included by nonevendor/github.com/conductorone/conductorone-sdk-go/pkg/models/shared/escalation.gois excluded by!vendor/**and included by nonevendor/modules.txtis excluded by!vendor/**and included by none
📒 Files selected for processing (1)
go.mod
Upgrades sdk version to use a temp fix added to the sdk.
Summary by CodeRabbit
✏️ Tip: You can customize this high-level summary in your review settings.