Skip to content

Adds subcommand suggestions for apparent typos. #26

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

clconway
Copy link

This adds a dependency on github.com/agext/levenshtein to compute the edit
distance from the given subcommand to the program's registered subcommand names.

Fixes #24

This adds a dependency on github.com/agext/levenshtein to compute the edit
distance from the given subcommand to the program's registered subcommand names.
@clconway
Copy link
Author

@adjackura Ping?

@adjackura
Copy link
Collaborator

Sorry for not setting this earlier. I'd prefer not to add dependencies to this package as it is critical for a number of systems and just adding in typo help doesn't seem like a big enough reason to change that right now.

@fgm
Copy link

fgm commented Mar 2, 2021

One thing which could be done to support the use case without adding dependencies is support a generic catchall handler, as suggested in #30 : that way anyone wishing to add a levenshtein (or whatever) approximate matcher could add it on its own without having to fork this package.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Subcommand suggestions for typos
3 participants