Skip to content

Conversation

@mstanbCO
Copy link
Contributor

@mstanbCO mstanbCO commented Jan 9, 2026

Summary by CodeRabbit

  • Chores
    • Updated platform SDK dependency to the latest compatible version to improve stability and compatibility with recent platform updates.
    • Removed an unused indirect dependency to streamline the dependency graph.
    • No changes to public APIs or exported interfaces; small, low-risk change set requiring minimal review.

✏️ Tip: You can customize this high-level summary in your review settings.

- Fix form_fields.go to use FieldInput instead of Field type
- Fix get_user.go to handle UserSchemasStatus to UserStatus conversion
@coderabbitai
Copy link

coderabbitai bot commented Jan 9, 2026

Walkthrough

Updated Go module: bumped github.com/conductorone/conductorone-sdk-go from v1.26.0 to v1.27.1 and removed the indirect dependency github.com/ericlagergren/decimal from go.mod. No other code changes.

Changes

Cohort / File(s) Summary
Go module
go.mod
Upgraded github.com/conductorone/conductorone-sdk-go from v1.26.0 to v1.27.1; removed indirect dependency github.com/ericlagergren/decimal.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐇 I nibbled at versions, light and spry,
Pushed a small hop from old to new,
Removed a crumb no longer nigh,
Module gleams with a fresher hue,
Hooray — a tidy, tiny brew! ✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'Update the go sdk to the latest version' accurately reflects the main change in the pull request, which updates the conductorone-sdk-go dependency from v1.26.0 to v1.27.1.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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


📜 Recent review details

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 2af255e and 4b62d70.

⛔ Files ignored due to path filters (10)
  • cmd/cone/get_user.go is excluded by none and included by none
  • go.sum is excluded by !**/*.sum and included by none
  • vendor/github.com/conductorone/conductorone-sdk-go/README.md is excluded by !vendor/** and included by none
  • vendor/github.com/conductorone/conductorone-sdk-go/RELEASES.md is excluded by !vendor/** and included by none
  • vendor/github.com/conductorone/conductorone-sdk-go/conductoroneapi.go is excluded by !vendor/** and included by none
  • vendor/github.com/conductorone/conductorone-sdk-go/connector.go is excluded by !vendor/** and included by none
  • vendor/github.com/conductorone/conductorone-sdk-go/gen.yaml is excluded by !vendor/** and included by none
  • vendor/github.com/conductorone/conductorone-sdk-go/pkg/models/operations/c1apiappv1connectorserviceupdateconnectorschedule.go is excluded by !vendor/** and included by none
  • vendor/github.com/conductorone/conductorone-sdk-go/pkg/models/operations/c1apifunctionsv1functionsservicecommit.go is excluded by !vendor/** and included by none
  • vendor/github.com/conductorone/conductorone-sdk-go/pkg/models/operations/c1apifunctionsv1functionsservicegetcommit.go is excluded by !vendor/** and included by none
📒 Files selected for processing (1)
  • go.mod
🚧 Files skipped from review as they are similar to previous changes (1)
  • go.mod

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

@mstanbCO mstanbCO merged commit b702480 into main Jan 13, 2026
3 checks passed
@mstanbCO mstanbCO deleted the mstanbCO/update_go_sdk branch January 13, 2026 17:22
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.

3 participants