Skip to content

chore: replace my fork of bitwarden SDK and update go version#83

Merged
Skarlso merged 1 commit into
mainfrom
replace-my-fork
Jun 21, 2026
Merged

chore: replace my fork of bitwarden SDK and update go version#83
Skarlso merged 1 commit into
mainfrom
replace-my-fork

Conversation

@Skarlso

@Skarlso Skarlso commented Jun 21, 2026

Copy link
Copy Markdown
Collaborator

Problem Statement

What is the problem you're trying to solve?

Related Issue

Fixes #...

Proposed Changes

How do you like to solve the issue and why?

Checklist

  • I have read the contribution guidelines
  • All commits are signed with git commit --signoff
  • My changes have reasonable test coverage
  • All tests pass with make test-ci

Changes

  • Go version: Updated from 1.23 to 1.26 across Dockerfile, Makefile, and go.mod
  • Bitwarden SDK: Removed personal fork reference and upgraded from v2.0.0 to v2.1.0, now using the official github.com/bitwarden/sdk-go/v2
  • golangci-lint: Updated from v2.11.3 to v2.12.2

@coderabbitai

coderabbitai Bot commented Jun 21, 2026

Copy link
Copy Markdown

Review Change Stack

Warning

Review limit reached

@Skarlso, we couldn't start this review because you've reached your PR review rate limit.

More reviews will be available in 55 minutes and 4 seconds. Learn how PR review limits work.

Your organization has used up its prepaid credits, and credit purchases are no longer available. Enable the review add-on in the billing tab to keep reviews running — you're only billed for reviews past your plan's rate limits ($0.25/file).

⌛ How to resolve this issue?

After more reviews become available, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based credits.

🚦 How do rate limits work?

CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan refill rate.

For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, the refill rate gradually slows as usage increases. The highest same-day bursts are limited more strictly.

Please see our Fair Usage Limits Policy for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 09a99140-9c72-4951-8a80-c19776af0063

📥 Commits

Reviewing files that changed from the base of the PR and between 61b1149 and c241a14.

⛔ Files ignored due to path filters (1)
  • go.sum is excluded by !**/*.sum
📒 Files selected for processing (4)
  • .golangci.yaml
  • Dockerfile
  • Makefile
  • go.mod

Walkthrough

Bumps the Go toolchain to 1.26 in both go.mod and the Dockerfile build stage. Removes the replace directive for the Bitwarden SDK fork and upgrades github.com/bitwarden/sdk-go/v2 to the official v2.1.0. Updates golangci-lint to v2.12.2 in the Makefile.

Changes

Toolchain and dependency version bumps

Layer / File(s) Summary
Go 1.26, bitwarden SDK v2.1.0, and build tooling
go.mod, Dockerfile, Makefile
go.mod sets the Go directive to 1.26, bumps github.com/bitwarden/sdk-go/v2 from v2.0.0 to v2.1.0, and removes the replace directive redirecting to the Skarlso fork. Dockerfile updates the build stage base image from golang:1.22 to golang:1.26. Makefile bumps GOLANGCI_LINT_VERSION from v2.11.3 to v2.12.2.

Possibly related PRs

  • external-secrets/bitwarden-sdk-server#73: Introduced the replace directive redirecting github.com/bitwarden/sdk-go/v2 to the Skarlso fork — this PR reverses that change by removing the directive and pointing to the official v2.1.0 release.
🚥 Pre-merge checks | ✅ 2
✅ Passed checks (2 passed)
Check name Status Explanation
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

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


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.

Signed-off-by: Gergely Brautigam <182850+Skarlso@users.noreply.github.com>
@Skarlso
Skarlso merged commit 8ecfb98 into main Jun 21, 2026
4 checks passed
@Skarlso
Skarlso deleted the replace-my-fork branch June 21, 2026 17:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

1 participant