Skip to content

Commit 44db963

Browse files
committed
fixup! 8948d
Address code review comments
1 parent a5793a5 commit 44db963

File tree

1 file changed

+6
-6
lines changed

1 file changed

+6
-6
lines changed

README.md

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -26,17 +26,17 @@ and then in modules implementing task queues and connection pools.
2626
$ npm install --save loopback-context cls-hooked
2727
```
2828

29-
Make sure you are running on a Node version supported by this module!
30-
Please check the output of `npm install` and make sure there are
31-
no `engine` related warnings.
29+
Make sure you are running on a Node.js version supported by this module
30+
(`^4.5`, `^5.10` or `^6.0`). When installing, check the output of `npm install`
31+
and make sure there are no `engine` related warnings.
3232

3333
## Usage
3434

3535
### Setup cls-hooked
3636

3737
To minimize the likelihood of loosing context in your application, you should
3838
ensure that `cls-hooked` is loaded as the first module of your application, so
39-
that it can wrap certain Node APIs before any other modules start using these
39+
that it can wrap certain Node.js APIs before any other modules start using these
4040
APIs.
4141

4242
Our recommended approach is to add `-r cls-hooked` to node's list of
@@ -125,7 +125,7 @@ chain (before the middleware that depends on
125125
invoked multiple times on the same request and returns immediately in
126126
subsequent runs.**
127127

128-
Here’s sample code which uses a middleware function to place the currently
128+
Here is a snippet using a middleware function to place the currently
129129
authenticated user into the context so that remote methods may use it:
130130

131131
**server/middleware/store-current-user.js**
@@ -158,7 +158,7 @@ module.exports = function(options) {
158158
{
159159
"initial": {
160160
"loopback-context#per-request": {}
161-
}
161+
},
162162
"auth": {
163163
"loopback#token": {}
164164
},

0 commit comments

Comments
 (0)