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

Unexpected behavior when combining dot-notation and camel-case expansion #77

Closed
adius opened this issue Jan 26, 2017 · 4 comments
Closed
Assignees
Labels

Comments

@adius
Copy link

adius commented Jan 26, 2017

--dot-notation.foo.bar yields to

{
  'dot-notation': {foo: {bar: true}},
  dotNotationFooBar: true,
}

I would, however, expect it to get parsed to:

{
  'dot-notation': {foo: {bar: true}},
  dotNotation: {foo: {bar: true}},
}

Is this a bug or the supposed behavior?

@bcoe bcoe added the bug label Mar 22, 2017
@bcoe
Copy link
Member

bcoe commented Mar 22, 2017

@adius this feels like a bug; if you get bored, would happily accept a patch 😉

@iilei
Copy link
Contributor

iilei commented Aug 21, 2018

May I help with the Patch?

iilei added a commit to iilei/yargs-parser that referenced this issue Aug 22, 2018
This fixes issue yargs#77
This *is* a breaking change
iilei added a commit to iilei/yargs-parser that referenced this issue Aug 22, 2018
This fixes issue yargs#77
This *is* a breaking change
@iilei
Copy link
Contributor

iilei commented Oct 15, 2018

Since the latest version this can be closed I suppose

@bcoe
Copy link
Member

bcoe commented Dec 25, 2018

@adius this should be fixed now thanks to @iilei's great work 👍

@bcoe bcoe closed this as completed Dec 25, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants