-
Notifications
You must be signed in to change notification settings - Fork 459
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
[WIP] Rewrite docs to include examples #115
Conversation
README.md
Outdated
|
||
## Example | ||
## How to use |
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.
Usage
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.
Done 🔥
README.md
Outdated
send(res, 200, 'Ready!') | ||
} | ||
``` | ||
npm install micro --save |
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.
Command line indicator (dollar sign) missing. Also please put the flag before the package name!
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.
Done 🔥
README.md
Outdated
``` | ||
|
||
To run the microservice on port `3000`, use the `micro` command: | ||
Add a script to your package.json like this: |
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.
package.json
, not package.json
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.
Done 🔥
# Conflicts: # README.md
This reverts commit dff3225.
Oh wow, thanks for this PR! I learned a lot by looking at the revised README. |
README.md
Outdated
|
||
```bash | ||
micro -H localhost sleep.js | ||
{ |
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.
Missing opening code fence (```)
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.
Fixed 🙌
README.md
Outdated
} | ||
``` | ||
|
||
#### Api |
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.
API (it's an abbreviation)
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.
Fixed 🙌
README.md
Outdated
} | ||
``` | ||
|
||
#### Api |
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.
API (it's an abbreviation)
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.
Fixed 🙌
README.md
Outdated
|
||
**`createError(code, msg, orig)`** | ||
#### Api |
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.
API (it's an abbreviation)
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.
Fixed 🙌
README.md
Outdated
@@ -249,6 +238,24 @@ function handleErrors (fn) { | |||
} | |||
``` | |||
|
|||
#### Api |
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.
API (it's an abbreviation)
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.
Fixed 🙌
Added some comments, hope you don't mind 😊 |
I wonder if we should disable |
I rewrote / moved a lot of stuff around. Please have a look :) Will add the examples soon.