Skip to content
This repository has been archived by the owner on Oct 4, 2024. It is now read-only.

Reenable strict mode so that shell gives helpful messages for unknown… #51

Closed
wants to merge 1 commit into from

Conversation

janedegtiareva
Copy link
Contributor

… commands

@janedegtiareva
Copy link
Contributor Author

fix for #50, which is a regression

@vgrichina
Copy link
Contributor

AFAIK, this will break near-shell for cases when config is present, but given command doesn't accept all of the options in config.

@ilblackdragon
Copy link
Member

@janedegtiareva can we define strict only on the top commands? e.g. near blah fails but not near send --blah=whatever?

@janedegtiareva
Copy link
Contributor Author

We can add all the config options to shell and expect no unrecognized config options. Opinions?

@mikedotexe
Copy link
Contributor

We can add all the config options to shell and expect no unrecognized config options. Opinions?

This is essentially the approach I took here:
#262

Plus some other fun snags to work through.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
P1 High priority
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants