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

finish structopt cli implementation and remove old clap parsing #7866

Open
jeremymv2 opened this issue Aug 11, 2020 · 2 comments
Open

finish structopt cli implementation and remove old clap parsing #7866

jeremymv2 opened this issue Aug 11, 2020 · 2 comments
Labels
Team:Habitat All issues tied to Chef habitat team

Comments

@jeremymv2
Copy link
Contributor

jeremymv2 commented Aug 11, 2020

Maintaining both structopt and native clap argument validation and marshaling in our CLI code base has become a non-trivial overhead.

An important tech-debt task should entail converting the full CLI code base to structopt and removing the old clap parsing and marshaling functions.

Ideally, our structopt implementation utilizes native clap consumption of arguments via ENVvars, incorporates desired defaults, and consumes values from cli.toml if they exist. If we can achieve this, we can eliminate a significant amount of old clap specific argument marshaling/validation functions and allow clap to do it up front. This way, what comes out of structopt will already be populated and validated and handing off to edge CLI commands will be much simplified.

@stale
Copy link

stale bot commented Jul 31, 2022

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. We value your input and contribution. Please leave a comment if this issue still affects you.

1 similar comment
@stale
Copy link

stale bot commented Aug 12, 2023

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. We value your input and contribution. Please leave a comment if this issue still affects you.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Team:Habitat All issues tied to Chef habitat team
Projects
None yet
Development

No branches or pull requests

2 participants