Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@dstansby
Copy link
Member

Fixes #431

@dstansby
Copy link
Member Author

pre-commit.ci autofix

Copy link
Member

@samcunliffe samcunliffe left a comment

Choose a reason for hiding this comment

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

Just to check: two greens is fine because they are different things?
Does inquirer do progress bars?

@samcunliffe samcunliffe added the needs-2-reviewers Could be considered "controversial" so worth a second pair of eyes label Nov 12, 2024
@samcunliffe samcunliffe requested a review from a team November 12, 2024 14:42
@dstansby
Copy link
Member Author

dstansby commented Nov 12, 2024

Does inquirer do progress bars?

Nope - Inquirer focuses on interactive command line interfaces (e.g., text input, radio buttons, checkboxes etc.).

@dstansby
Copy link
Member Author

This should definitely be squash merged!

@samcunliffe samcunliffe requested review from a team and matt-graham December 2, 2024 14:41
@matt-graham matt-graham merged commit b6465db into main Dec 2, 2024
15 checks passed
@matt-graham matt-graham deleted the inquirer branch December 2, 2024 16:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs-2-reviewers Could be considered "controversial" so worth a second pair of eyes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add inquirer to CLI tools

4 participants