Skip to content

Update module github.com/urfave/cli/v2 to v3#60

Merged
bluet merged 7 commits intomainfrom
renovate/github.com-urfave-cli-v2-3.x
Apr 20, 2026
Merged

Update module github.com/urfave/cli/v2 to v3#60
bluet merged 7 commits intomainfrom
renovate/github.com-urfave-cli-v2-3.x

Conversation

@renovate
Copy link
Copy Markdown
Contributor

@renovate renovate Bot commented Apr 15, 2026

This PR contains the following updates:

Package Change Age Confidence
github.com/urfave/cli/v2 v2.27.7 → v3.8.0 age confidence

Release Notes

urfave/cli (github.com/urfave/cli/v2)

v3.8.0

Compare Source

What's Changed

New Contributors

Full Changelog: urfave/cli@v3.7.0...v3.8.0

v3.7.0

Compare Source

What's Changed

New Contributors

Full Changelog: urfave/cli@v3.6.2...v3.7.0

v3.6.2

Compare Source

What's Changed

New Contributors

Full Changelog: urfave/cli@v3.6.1...v3.6.2

v3.6.1

Compare Source

What's Changed

New Contributors

Full Changelog: urfave/cli@v3.6.0...v3.6.1

v3.6.0

Compare Source

What's Changed

New Contributors

Full Changelog: urfave/cli@v3.5.0...v3.6.0

v3.5.0

Compare Source

What's Changed

New Contributors

Full Changelog: urfave/cli@v3.4.1...v3.5.0

v3.4.1

Compare Source

What's Changed

Full Changelog: urfave/cli@v3.4.0...v3.4.1

v3.4.0

Compare Source

What's Changed

New Contributors

Full Changelog: urfave/cli@v3.3.9...v3.4.0

v3.3.9

Compare Source

What's Changed

New Contributors

Full Changelog: urfave/cli@v3.3.8...v3.3.9

v3.3.8

Compare Source

What's Changed

Full Changelog: urfave/cli@v3.3.7...v3.3.8

v3.3.7

Compare Source

What's Changed

New Contributors

Full Changelog: urfave/cli@v3.3.6...v3.3.7

v3.3.6

Compare Source

What's Changed

Full Changelog: urfave/cli@v3.3.5...v3.3.6

v3.3.5

Compare Source

What's Changed

Full Changelog: urfave/cli@v3.3.4...v3.3.5

v3.3.4

Compare Source

What's Changed

New Contributors

Full Changelog: urfave/cli@v3.3.3...v3.3.4

v3.3.3

Compare Source

What's Changed

New Contributors

Full Changelog: urfave/cli@v3.3.2...v3.3.3

v3.3.2

Compare Source

What's Changed

New Contributors

Full Changelog: urfave/cli@v3.3.1...v3.3.2

v3.3.1

Compare Source

What's Changed

Full Changelog: urfave/cli@v3.3.0...v3.3.1

v3.3.0

Compare Source

What's Changed

New Contributors

Full Changelog: urfave/cli@v3.2.0...v3.3.0

v3.2.0

Compare Source

Breaking change IntFlag now uses int type and not int64. Please change to using Int64Flag for int64 types. Similar behavior for UintFlag as well. See https://pkg.go.dev/github.com/urfave/cli/v3 for a full list of flag types. See #​2094 for full patch for this

What's Changed

New Contributors

Full Changelog: urfave/cli@v3.1.1...v3.2.0

v3.1.1

Compare Source

v3.1.0

Compare Source

What's Changed

New Contributors

Full Changelog: urfave/cli@v3.0.0-beta1.01...v3.1.0

Configuration

📅 Schedule: (UTC)

  • Branch creation
    • At any time (no schedule defined)
  • Automerge
    • At any time (no schedule defined)

🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.

♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

🔕 Ignore: Close this PR and you won't be reminded about this update again.

  • If you want to rebase/retry this PR, check this box

This PR was generated by Mend Renovate. View the repository job log.

@bluet bluet requested a review from Copilot April 19, 2026 16:22
Copilot AI reviewed Apr 19, 2026
View reviewed changes
Copy link
Copy Markdown

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

This PR aims to upgrade the project’s CLI dependency from github.com/urfave/cli/v2 to github.com/urfave/cli/v3, but the current changes only adjust module metadata and do not migrate the codebase off v2.

Changes:

  • Updated go.mod to add a direct requirement on github.com/urfave/cli/v3 v3.8.0 while still requiring github.com/urfave/cli/v2 v2.27.7.
  • Updated go.sum to include the cli/v3 go.mod checksum.

Reviewed changes

Copilot reviewed 1 out of 2 changed files in this pull request and generated 1 comment.

File Description
go.mod Adds a direct dependency on cli/v3 but retains cli/v2, leaving the repo not actually migrated.
go.sum Adds cli/v3 checksum entry, reflecting it’s in the module graph but not necessarily used by builds.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread go.mod Outdated
Comment on lines +5 to +6
require github.com/urfave/cli/v3 v3.8.0 // direct

Copy link

Copilot AI Apr 19, 2026

Choose a reason for hiding this comment

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

go.mod now requires urfave/cli/v3 as a direct dependency while the codebase still imports github.com/urfave/cli/v2 (e.g., cmd/syspkg/main.go). As a result, this PR does not actually upgrade the CLI usage and leaves both major versions in the module graph. Either migrate imports/usages to github.com/urfave/cli/v3 (and then remove the v2 requirement), or drop the v3 requirement to keep the module state consistent with the code.

Suggested change
require github.com/urfave/cli/v3 v3.8.0 // direct

Copilot uses AI. Check for mistakes.
@bluet
Copy link
Copy Markdown
Owner

bluet commented Apr 20, 2026

@copilot do a proper v2 -> v3 migration

ref: https://cli.urfave.org/migrate-v2-to-v3/

Copilot AI and others added 3 commits April 20, 2026 07:46
Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 20, 2026 •
edited
Loading

@copilot do a proper v2 -> v3 migration

ref: https://cli.urfave.org/migrate-v2-to-v3/

Done in commit f840a53. The full v2 → v3 migration includes:

  • Updated import path to github.com/urfave/cli/v3
  • cli.App → cli.Command (v3 uses a single Command type)
  • All action signatures: func(c *cli.Context) error → func(_ context.Context, cmd *cli.Command) error
  • app.Run(os.Args) → app.Run(context.Background(), os.Args)
  • Flag/arg accessors: c.Bool(name) → cmd.Bool(name), c.Args() → cmd.Args()
  • EnableBashCompletion → EnableShellCompletion
  • Subcommands field → Commands
  • go.mod now only has cli/v3; go mod tidy removed the previously v2-only transitive deps (go-md2man, blackfriday, xrash/smetrics)

All tests pass, build is clean.

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • cli.urfave.org
    • Triggering command: /home/REDACTED/work/_temp/ghcca-node/node/bin/node /home/REDACTED/work/_temp/ghcca-node/node/bin/node --enable-source-maps /home/REDACTED/work/_temp/copilot-developer-action-main/dist/index.js (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

Copilot AI requested a review from bluet April 20, 2026 07:52
@renovate
Copy link
Copy Markdown
Contributor Author

renovate Bot commented Apr 20, 2026

Edited/Blocked Notification

Renovate will not automatically rebase this PR, because it does not recognize the last commit author and assumes somebody else may have edited the PR.

You can manually request rebase by checking the rebase/retry box above.

⚠️ Warning: custom changes will be lost.

@bluet
Copy link
Copy Markdown
Owner

bluet commented Apr 20, 2026

@copilot whitelist added: cli.urfave.org

also make sure all tests passed, and list if there's any breaking changes

Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 20, 2026

@copilot whitelist added: cli.urfave.org

also make sure all tests passed, and list if there's any breaking changes

All tests pass (go test ./... — all packages green). Also fixed a gofmt alignment issue in main.go that was causing the "Lint and Format" CI step to fail (commit 275cf97).

Breaking changes from v2 → v3 (per the migration guide) that were applied:

v2 v3 Applied
import "github.com/urfave/cli/v2" import "github.com/urfave/cli/v3" ✅
cli.App{} cli.Command{} ✅
EnableBashCompletion: true EnableShellCompletion: true ✅
(&cli.App{}).Run(os.Args) (&cli.Command{}).Run(context.Background(), os.Args) ✅
ActionFunc func(*Context) error ActionFunc func(context.Context, *cli.Command) error ✅
cli.Context.Bool/Args/etc. cli.Command.Bool/Args/etc. ✅
Subcommands: []*cli.Command{} Commands: []*cli.Command{} ✅

Not applicable to this codebase: FilePath/EnvVars on flags (→ Sources), TimestampFlag layout, PathFlag, Authors struct, BashComplete, altsrc.

@bluet
Copy link
Copy Markdown
Owner

bluet commented Apr 20, 2026

@copilot resolve merge conflicts properly

@bluet
Merge remote-tracking branch 'origin/main' into renovate/github.com-u…
8576bdd 
…rfave-cli-v2-3.x

# Conflicts:
#	go.mod
#	go.sum

Co-authored-by: bluet <51141+bluet@users.noreply.github.com>
Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 20, 2026

@copilot resolve merge conflicts properly

Done in commit 8576bdd. Merged origin/main and resolved the go.mod/go.sum conflicts by keeping only github.com/urfave/cli/v3 v3.8.0 (dropping the stale cli/v2 + its transitive deps that origin/main still had). Ran go mod tidy to cleanly regenerate go.sum, and all tests pass.

@bluet
Copy link
Copy Markdown
Owner

bluet commented Apr 20, 2026

@copilot resolve merge conflicts properly

@bluet
Merge remote-tracking branch 'origin/main' into renovate/github.com-u…
8d1fb32 
…rfave-cli-v2-3.x

# Conflicts:
#	go.mod
#	go.sum

Co-authored-by: bluet <51141+bluet@users.noreply.github.com>
Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 20, 2026

@copilot resolve merge conflicts properly

Done in commit 8d1fb32. origin/main had picked up a new commit (97c9752) that reverted back to cli/v2, causing fresh conflicts in go.mod and go.sum. Merged origin/main, resolved by keeping our cli/v3 requirement, ran go mod tidy to regenerate go.sum, and all tests pass.

@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Apr 20, 2026

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@bluet bluet merged commit 36f89ec into main Apr 20, 2026
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot Copilot left review comments

@bluet bluet Awaiting requested review from bluet

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@bluet