-
Notifications
You must be signed in to change notification settings - Fork 33
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
InvalidArgumentName when argument name has trailing digit #62
Comments
I started to fix this bug, but after looking at the tests realized that this was intentional behavior. I encountered this issue when trying to implement a |
@bzdzb from looking at where ARG_NAME_RE is defined, I suspect that this was not actually intentional. Instead of I'm also thinking that it may make sense to require So these should be valid:
but these should be invalid:
If you'd be willing to make a PR, I'd really appreciate it. |
val url1 by parser.storing(help = "jdbc url. Example: jdbc:postgresql://localhost:5432/db1")
results in:
Exception in thread "main" java.lang.IllegalArgumentException: URL1 is not a valid argument name
Not sure if this is by design or a bug. Changing to the following fixes the issue:
val url_1 by parser.storing(help = "jdbc url. Example: jdbc:postgresql://localhost:5432/db1")
The text was updated successfully, but these errors were encountered: