Skip to content

Suggest renaming option from name to --name#1355

Merged
davidism merged 1 commit into
pallets:masterfrom
jtrakk:patch-1
Aug 13, 2020
Merged

Suggest renaming option from name to --name#1355
davidism merged 1 commit into
pallets:masterfrom
jtrakk:patch-1

Conversation

@jtrakk

@jtrakk jtrakk commented Jul 13, 2019

Copy link
Copy Markdown
Contributor

Usually when I get this error message, it's because I wrote @click.option('foo') when I meant @click.option('--foo'). So this PR updates the error message to suggest the user fix that.

@davidism

Copy link
Copy Markdown
Member

Reworded the message a bit, updated test, added changelog.

@davidism davidism added this to the 8.0.0 milestone Aug 13, 2020
@davidism
davidism merged commit 0ad4e79 into pallets:master Aug 13, 2020
@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Nov 13, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants