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

conflicts_with not working when loading from yaml #271

Closed
XAMPPRocky opened this issue Sep 22, 2015 · 2 comments
Closed

conflicts_with not working when loading from yaml #271

XAMPPRocky opened this issue Sep 22, 2015 · 2 comments
Labels
A-parsing Area: Parser's logic and needs it changed somehow. C-bug Category: Updating dependencies

Comments

@XAMPPRocky
Copy link

Trying what was suggested in #267 I added the conflicts_with flag. This had no effect, and the clap still errors wince there is no input

Command: tokei -l

Error:

error: The following required arguments were not supplied:
    '<input>...'

USAGE:
    tokei <input>... --languages

For more information try --help

CLI file

name: Tokei
version: 1.1.1
author: Aaron P. <theaaronepower@gmail.com>
about: A quick CLOC (Count Lines Of Code) tool
args:
  - exclude:
      short: e
      long: exclude
      help: Will ignore all files and directories containing the word ie --exclude node_modules
      takes_value: true
  - sort:
      short: s
      long: sort
      takes_value: true
      possible_values: [files, total, blanks, code, commments]
      help: Will sort based on a certain column ie --sort=files will sort by file count.
  - input:
      index: 1
      multiple: true
      required: true
      help: The input file(s)/directory(ies)
  - languages:
      short: l
      long: languages
      conflicts_with:
          - input
      help: prints out supported languages and their extensions
@Vinatorul Vinatorul mentioned this issue Sep 22, 2015
11 tasks
@sru sru added C-bug Category: Updating dependencies C: args A-parsing Area: Parser's logic and needs it changed somehow. labels Sep 22, 2015
@kbknapp
Copy link
Member

kbknapp commented Sep 22, 2015

Once #272 is merged we'll update crates.io Two version in one day! 😄 haha

@Aaronepower thanks for taking the time to file these issues! 👍

homu added a commit that referenced this issue Sep 23, 2015
fix(Conflicts): fixes bug with conflicts not removing required args

Closes #271
@homu homu closed this as completed in #272 Sep 23, 2015
@kbknapp
Copy link
Member

kbknapp commented Sep 23, 2015

@Aaronepower v1.4.2 is on crates.io and should have fixed this bug for you.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-parsing Area: Parser's logic and needs it changed somehow. C-bug Category: Updating dependencies
Projects
None yet
Development

No branches or pull requests

3 participants