-
Notifications
You must be signed in to change notification settings - Fork 0
Open
Labels
Description
curious about why the superfluous
index.jsfiles
- See if serverless.yml will accept a more streamlined structure.
Would suggest adding jsdoc blocks
- Add jsdoc blocks
Given that you’re using
latestin babel, you can do more es6 things. https://github.com/chasingmaxwell/FeedBoost/blob/master/src/lib/token/index.js#L15 can just be{code}
- Use shorthand in places where the key and name of the variable for the value are the same in objects.
https://github.com/chasingmaxwell/FeedBoost/blob/master/src/lib/token/index.js#L7 no need to allocate a variable for immediate return
- Simplify by returning the result of the function invocation rather than creating a variable.
I would consider getting [a request promise library] where you can instantiate with values that won’t change, like a base url.
- Evaluate the use of request-promise.