build(deps): pin cloud-commitments-go to main (6168f8b) - #2102
Conversation
The previous pseudo-version pins (pkg@43ab778da, providers/{aws,azure,gcp}@692cacc627d1)
were on the unmerged cloud-commitments-go PR #1 split branch, which no longer exists
on main after the squash-merge to 6168f8b5360d5189c140cd8f4de614531cffa998. Re-pin all
four modules to that commit.
|
Warning Review limit reached
This review includes 1 billable file and costs up to $0.25.
Reviews can continue after your included limit without a manual trigger. An admin must approve usage-based billing. Or wait 7 minutes for your next included review. View limit detailsLimit details: You’ve used all 3 included reviews currently available. Your 41 included PR review attempts over the past 7 days set your current allowance at 3 reviews per hour. Review configuration: ⚙️ Run configurationConfiguration used: Repository: LeanerCloud/cloud-commitments-cli/.coderabbit.yaml Review profile: CHILL Plan: Essentials Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (1)
Comment |
|
Independent review + local verification: MERGE at 9d3d53b. Re-pins cloud-commitments-go to main (6168f8b5); exactly the expected 8 go.sum lines; go mod verify OK; full -race suite passes against the new library (so no renamed identifier is referenced); checks green. |
Summary
Re-pins
github.com/LeanerCloud/cloud-commitments-go/{pkg,providers/aws,providers/azure,providers/gcp}from the abandoned split-PR pseudo-versions to the commit that actually landed on go main.Why
cloud-commitments-goPR #1 was squash-merged tomainas6168f8b5360d5189c140cd8f4de614531cffa998. This repo's previous pins were pseudo-versions taken from the unmerged split branch before the squash:pkgv0.0.0-20260925082912-43ab778da7acv0.0.0-20260928074610-6168f8b5360dproviders/awsv0.0.0-20260926232454-692cacc627d1v0.0.0-20260928074610-6168f8b5360dproviders/azurev0.0.0-20260926232454-692cacc627d1v0.0.0-20260928074610-6168f8b5360dproviders/gcpv0.0.0-20260926232454-692cacc627d1v0.0.0-20260928074610-6168f8b5360dThose commits (
43ab778da7ac,692cacc627d1) are no longer reachable from go main after the squash-merge, so this repo'sgo.modpointed at history that no longer exists there.Change
All four modules resolved to
v0.0.0-20260928074610-6168f8b5360d(MVS resolved cleanly; the pkg/providers go.mod files inside cloud-commitments-go may still reference older pseudo-versions of each other, but the resulting build graph is consistent).go.workstill only containsuse ., unchanged.The renamed exported identifiers in cloud-commitments-go (
AWSProvider->Provider,XxxAPI->API, etc.) required no call-site changes here;go build/go vetpass with no errors.Verification
go build -o /dev/null ./cmd: passgo vet ./...: pass (no findings)go test -race -short ./...: pass,ok github.com/LeanerCloud/cloud-commitments-cli/cmd 445.556s(single package, no FAIL)go mod tidy -diff: empty (clean)golangci-lint v2.10.1(CI-pinned version,golangci-lint run --timeout=10m, exact CI invocation):0 issuesCloses nothing; this is a housekeeping re-pin, not tied to a specific issue.