8000 PathFlag regression in v3 · Issue #2125 · urfave/cli · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

PathFlag regression in v3 #2125

New issue

Have a question about this project? Sign up for a f 8000 ree GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
3 tasks done
imunhatep opened this issue May 7, 2025 · 3 comments · Fixed by #2136
Closed
3 tasks done

PathFlag regression in v3 #2125

imunhatep opened this issue May 7, 2025 · 3 comments · Fixed by #2136
Labels
area/v3 relates to / is being considered for v3 kind/bug describes or fixes a bug status/triage maintainers still need to look into this

Comments

@imunhatep
Copy link

My urfave/cli version is

v3

Checklist

  • Are you running the latest v3 release? The list of releases is here.
  • Did you check the manual for your release? The v3 manual is here
  • Did you perform a search about this problem? Here's the GitHub guide about searching.

Dependency Management

  • My project is using vendoring.

Describe the bug

A clear and concise description of what the bug is.

To reproduce

&cli.Command{
		Name:      "example",
		Flags: []cli.Flag{
			&cli.PathFlag{
				Name:     "file",
				Aliases:  []string{"f"},
			},
		},
	}

Expected behavior

PathFlag exists/replaced with some approach
@imunhatep imunhatep added kind/bug describes or fixes a bug status/triage maintainers still need to look into this area/v3 relates to / is being considered for v3 labels May 7, 2025
@dearchap
Copy link
Contributor

@imunhatep Please use StringFlag and set the "TakesFile" field to true

@imunhatep
Copy link
Author

Thank you for clarifying this one, did I miss this one in the migration documentation v2 -> v3? Probably can be closed, if only to clarify this part in the docs.

@dearchap
Copy link
Contributor

Will keep it open until we can fix the docs. Thanks for reporting

@dearchap dearchap mentioned this issue May 18, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
area/v3 relates to / is being considered for v3 kind/bug describes or fixes a bug status/triage maintainers still need to look into this
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix Docs(issue_2125) Add PathFlag to StringFlag migration dearchap/cli
2 participants
@imunhatep @dearchap
0