Skip to content

chore(cli): display error for unknown arguments - #193

Closed
Tlacenka wants to merge 2 commits into
mainfrom
detect-unknown-arguments
Closed

Tlacenka wants to merge 2 commits into
mainfrom
detect-unknown-arguments

Conversation

@Tlacenka

@Tlacenka Tlacenka commented Nov 1, 2023 •

Copy link
Copy Markdown
Collaborator

I found out there is a strict option for the yargs which provides the desired behaviour.
Documentation here.

New behaviour

image

@Tlacenka Tlacenka linked an issue Nov 1, 2023 that may be closed by this pull request
@Tlacenka Tlacenka self-assigned this Nov 1, 2023
@BioPhoton

Copy link
Copy Markdown
Collaborator

Could you also show the old behaviour please

@BioPhoton BioPhoton added 🤓 UX UX improvement for CLI users 🧩 cli labels Nov 11, 2023
@BioPhoton

Copy link
Copy Markdown
Collaborator

how to go on here?

@Tlacenka Tlacenka closed this Nov 14, 2023
@BioPhoton
BioPhoton deleted the detect-unknown-arguments branch November 20, 2023 23:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🧩 cli 🤓 UX UX improvement for CLI users

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Inform user about invalid arguments

2 participants