Skip to content
New issue

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

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

Consolidate all Context code into Command #1587

Closed
Tracked by #1531
meatballhat opened this issue Nov 15, 2022 · 0 comments · Fixed by #1784
Closed
Tracked by #1531

Consolidate all Context code into Command #1587

meatballhat opened this issue Nov 15, 2022 · 0 comments · Fixed by #1784
Assignees
Labels
area/v3 relates to / is being considered for v3
Milestone

Comments

@meatballhat
Copy link
Member

meatballhat commented Nov 15, 2022

(wild generalizations warning) The Context struct exists for reasons mostly related to the use of stdlib flag and has a concept of "lineage" that forms a tree with other Context nodes. I would like to see this lineage relationship move to Command given the existence of .Commands for traversing outward, and for Command to be the source of truth for values that are fetched via the type-specific flag accessor functions like .String(string) string, .Int(string) int, .Bool(string) bool.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/v3 relates to / is being considered for v3
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants