Skip to content

chore(deps): updated cliforge import paths after pkg restructuring#59

Merged
rios0rios0 merged 2 commits intomainfrom
chore/update-cliforge-imports
Apr 2, 2026
Merged

chore(deps): updated cliforge import paths after pkg restructuring#59
rios0rios0 merged 2 commits intomainfrom
chore/update-cliforge-imports

Conversation

@rios0rios0
Copy link
Copy Markdown
Owner

Summary

  • updated cliforge import paths from cliforge/selfupdate to cliforge/pkg/selfupdate after the library restructured its packages under pkg/
  • renamed selfupdate.NewSelfUpdateCommand to selfupdate.NewCommand to match the new API
  • bumped cliforge dependency to commit cd7225c (branch chore/add-boilerplate-files)

🚦 Quality checklist

  • Did you add the changes in the CHANGELOG.md?
  • Did you run all the code checks? (go test)
  • Are the tests passing?

Test plan

  • Verified go build ./... compiles successfully
  • Verified go mod tidy runs cleanly
  • Run go test ./... in CI

🤖 Generated with Claude Code

Copilot AI review requested due to automatic review settings April 2, 2026 18:13
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates DevForge to accommodate a cliforge package restructure by switching to the new import path and aligning with the renamed self-update constructor.

Changes:

  • Bumped github.com/rios0rios0/cliforge to v0.1.1-0.20260402173400-cd7225c728db.
  • Updated selfupdate import path to github.com/rios0rios0/cliforge/pkg/selfupdate.
  • Renamed usage from selfupdate.NewSelfUpdateCommand(...) to selfupdate.NewCommand(...).

Reviewed changes

Copilot reviewed 2 out of 3 changed files in this pull request and generated no comments.

File Description
go.mod Updates cliforge dependency version to the new pseudo-version commit.
go.sum Refreshes checksums for the updated cliforge version.
cmd/devforge/main.go Switches selfupdate import path and updates constructor call to the new API name.

@claude
Copy link
Copy Markdown

claude bot commented Apr 2, 2026

Code review

No issues found. Checked for bugs and CLAUDE.md compliance.

@rios0rios0 rios0rios0 force-pushed the chore/update-cliforge-imports branch from 7ddfe9c to 6e924ce Compare April 2, 2026 18:26
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud bot commented Apr 2, 2026

@rios0rios0 rios0rios0 merged commit b78a4e5 into main Apr 2, 2026
20 checks passed
@rios0rios0 rios0rios0 deleted the chore/update-cliforge-imports branch April 2, 2026 21:16
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.

2 participants