toastynerd / superagent-cli

MIT License
23 stars 8 forks source link

a few bug fixes #8

Closed aaron-j-krause closed 9 years ago

aaron-j-krause commented 9 years ago

-modified regex to catch strings enclosed in quotes -added a test to arguements-parser-test -lowered argument length threshold to run command line object parser -added an if branch in callbacks to prevent body from logging out twice

toastynerd commented 9 years ago

Ideally these would be two different pull request one to fix the '.' issue and one for the spelling/styling fixes. Also make sure to mention issue in the PR description. Just grab the url of the issue and say something like "fixes https://github.com/toastynerd/superagent-cli/issues/7"

toastynerd commented 9 years ago

I'm going to close this out and have you resubmit this as 2 PRs. One with the slight change to what's getting logged out when both text and body are the same and a separate one with the spelling/styling fixes.