Skip to content
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

[DOCS] Read-me description inconsistent with actual handling of invalid option values #40

Open
mklement0 opened this issue Jan 30, 2015 · 0 comments
Assignees

Comments

@mklement0
Copy link
Contributor

As of v3.0.1, README.md states:

By default, nopt outputs a warning to standard error when invalid options are found.

Note that 'invalid option' refers to an invalid value specified for a known option in this context. I've submitted a PR to clarify this.

In practice, no such warning is printed. Nothing in nopt.js suggests that even an attempt is made to do so.
As an aside, there's a remove variable that doesn't seem to do anything.

Not sure what the actual intent is, but it looks like the warning was intentionally removed and that the read-me needs to be updated accordingly, including removing the following passage, which becomes moot:

If no nopt.invalidHandler is assigned, then it will console.error its whining. If it is assigned to boolean false then the warning is suppressed.

@darcyclarke darcyclarke changed the title Read-me description inconsistent with actual handling of invalid option values [DOCS] Read-me description inconsistent with actual handling of invalid option values Jul 28, 2022
@darcyclarke darcyclarke self-assigned this Jul 28, 2022
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

No branches or pull requests

2 participants