Skip to content

Commit eb6e9a0

Browse files
mhdawsonjasnell
authored andcommitted
doc: remove 'you' from writing-tests.md
Noticed this while reading through writing-tests.md today. As per style guide avoid the use of you, your etc. Rational as per: http://www2.ivcc.edu/rambo/tip_formal_writing_voice.htm PR-URL: #13319 Reviewed-By: Vse Mozhet Byt <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Rich Trott <[email protected]> Reviewed-By: Luigi Pinca <[email protected]>
1 parent b16dd98 commit eb6e9a0

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

doc/guides/writing-tests.md

+4-4
Original file line numberDiff line numberDiff line change
@@ -232,8 +232,8 @@ For performance considerations, we only use a selected subset of ES.Next
232232
features in JavaScript code in the `lib` directory. However, when writing
233233
tests, for the ease of backporting, it is encouraged to use those ES.Next
234234
features that can be used directly without a flag in [all maintained branches]
235-
(https://github.com/nodejs/lts), you can check [node.green](http://node.green)
236-
for all available features in each release.
235+
(https://github.com/nodejs/lts). [node.green](http://node.green/) lists
236+
available features in each release.
237237

238238
For example:
239239

@@ -274,8 +274,8 @@ These imported tests will be wrapped like this:
274274
/* eslint-enable */
275275
```
276276

277-
If you want to improve tests that have been imported this way, please send
278-
a PR to the upstream project first. When your proposed change is merged in
277+
To improve tests that have been imported this way, please send
278+
a PR to the upstream project first. When the proposed change is merged in
279279
the upstream project, send another PR here to update Node.js accordingly.
280280
Be sure to update the hash in the URL following `WPT Refs:`.
281281

0 commit comments

Comments
 (0)