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.

Sign up for GitHub

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

Jump to bottom

Don't require --text for the text #6

Closed
mfn opened this issue Dec 9, 2016 · 2 comments
Closed

Don't require --text for the text #6

mfn opened this issue Dec 9, 2016 · 2 comments

Comments

@mfn
Copy link

mfn commented Dec 9, 2016

Since the sole purpose of the tool is to analyze the text I think --text isn't necessary and it should simply accept all unparsed args as text:
elyzer --index myIndex --analyzer standard text to analyze

Currently, this needs to be prefixed and quotes:
elyzer --index myIndex --analyzer standard --text "text to analyze"

The following will fail or behave unexpected:

  • ... --text text to analzye => fails with unrecognized arguments: to analyze
  • ... --text text --text to --text analyze => only analyzes the last one, silently drops the first two
@mfn mfn mentioned this issue Dec 9, 2016
Closed
@softwaredoug
Copy link
Contributor

Fixed in #10 ... will be releasing soon :)

@softwaredoug
Copy link
Contributor

Released in v1.0.0. Let me know if you run into any probs!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@mfn @softwaredoug