build(deps): bump cinc-api to v0.13.0 and cinc-supermarket-api to v0.6.0 - #195
Merged
Conversation
Both libraries moved from the tas50 org to cinc-project, mirroring the move this repo made in a8636e3. The new module paths are declared in their go.mod files, so upgrading requires rewriting the import paths rather than just bumping the version. Neither release changes the Go API surface, so no call sites needed edits. Both packages are imported under aliases (cinc, sm), which kept the rewrite confined to import blocks. gofmt reordered those blocks because github.com/cinc-project sorts ahead of github.com/spf13 where github.com/tas50 sorted after it. Also updates the prose references in CLAUDE.md, AGENTS.md, and the design docs, and corrects two doc comments in cli/supermarket that named the library without its -api suffix. Signed-off-by: Tim Smith <tim@mondoo.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Updates the two API libraries to their latest releases.
This is a module rename, not just a version bump
Both libraries moved from the
tas50org tocinc-project, mirroring the move this repo made for its own module in a8636e3. The new path is declared inside theirgo.modfiles, so the old path no longer resolves at these versions:Upgrading therefore means rewriting the import paths, not just bumping versions.
cinc-apitas50/...v0.12.0cinc-project/...v0.13.0cinc-supermarket-apitas50/...v0.4.2cinc-project/...v0.6.0What changed
Neither release changes the Go API surface, so no call sites needed edits. Both packages are imported under aliases (
cinc,sm), which kept the rewrite confined to import blocks.gofmtreordered those import blocks, becausegithub.com/cinc-projectsorts ahead ofgithub.com/spf13wheregithub.com/tas50sorted after it.CLAUDE.md,AGENTS.md, anddocs/dev/, matching how a8636e3 handled the same docs.cli/supermarketthat named the library without its-apisuffix and still pointed at the old org.No changes to
docs/commands/:make docsreports no drift, since no help strings moved.Verification
golangci-lint run ./...: 0 issuesgo build ./...,go vet ./...,go vet -tags acceptance ./test/...: cleangofmt -l .: emptygo test ./...: all packages passgo test -tags acceptance ./test/...: passes against a live cinc-zeromake docs: no driftBecause the diff is a mechanical sweep over 126 files, it was audited with
git diff -U0: every added line containscinc-projectand every removed line containstas50, so nothing unintended crept in.Supply-chain note
The dependency org changed, so that was checked explicitly.
repos/tas50/cinc-apiandrepos/tas50/cinc-supermarket-apiboth return a GitHub transfer redirect tocinc-project/...withfork: falseandsource: null, meaning the original repositories were transferred rather than replaced by look-alikes. The destination is the same org that owns this repo. Both tags were cut by the original maintainer,go mod verifyreports all modules verified,GOSUMDBis the defaultsum.golang.orgwith noGOPRIVATE/GOFLAGSbypass, and there are noreplacedirectives.