-
Notifications
You must be signed in to change notification settings - Fork 0
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
Release 0.2.0 #21
Release 0.2.0 #21
Conversation
…ent upon --help flag
* Fixing version output by adding the version attribute using commander * Adding quotes to version number
…isted in package.json
a response of length 0
* Setting up chalk support * Adding red to errors and adding cyan to the divider. Keeping the definition in white
* Add contributing guidelines * Add contributing to readme. * Update Contributing gudelines
Should we be merging this with merge commit or squash? Thoughts? |
@rainrivas I think we can merge w/o squash. |
@jwu910 so we will merge commit right? (I am pretty sure you don't rebase commits if it's a release) |
@rainrivas Correct. I can agree with that. Merge commit for releases. |
} | ||
}); | ||
trimRes.forEach((result) => { | ||
if (typeof (result.definition) !== undefined) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is there a case that this would be undefined?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@jwu910 if there is a bad response and the response doesn't have a "definition" element. It's sort of a catch all. For example
{
"tags": [],
"result_type": "no_results",
"list": [],
"sounds": []
}
vs actually finding results
{
"tags": [
"sleng",
"colloquialism",
"neologism",
"slang"
],
"result_type": "exact",
"list": [
{
"definition": "SLNG is actually slang for...slang.",
"permalink": "http://slng.urbanup.com/1146042",
"thumbs_up": 9,
"author": "DaWhitey",
"word": "slng",
"defid": 1146042,
"current_vote": "",
"example": "Mah is slng for My \"Ur slng so cheep\"",
"thumbs_down": 4
}
],
"sounds": []
}
Currently published v0.2.0 as slng@beta on npm.
Once PR is merged, I'll republish and tag as slng@latest
Fixes
-V, --version