Conversation
Trott
added
doc
|
(This documentation is a stub.) |
test/README.md
Outdated
There was a problem hiding this comment.
A link would be helpful (Web Platform Tests and/or testharness.js). Also the second sentence seems to have an extra "in".
|
I personally don't really think it's necessary to extend this documentation stub, when |
|
@TimothyGu Thanks. I incorporated both links and got a little more specific with the text. PTAL. |
TimothyGu
left a comment
TimothyGu
left a comment
There was a problem hiding this comment.
LGTM. Thanks for writing this.
test/README.md
Outdated
There was a problem hiding this comment.
I would say it's more like a "mock" or "port"? But this description LGTM too.
There was a problem hiding this comment.
"port" works for me. I'll change it.
Trott
force-pushed
the
wpt
branch
2 times, most recently
from
d28ea8c to
c284e24
Compare
test/README.md
Outdated
There was a problem hiding this comment.
Maybe ### WPT (Web Platform Tests) for people who don't know what it is and don't see the link two lines down?
There was a problem hiding this comment.
Parentheses are used to show arguments in the other entries so I probably wouldn't do it exactly that way. But if there's a way to clarify it in the text below, I'm happy to.
There was a problem hiding this comment.
Maybe just start the next line with it, something like:
Web Platform Tests - A port of parts of
test/README.md
Outdated
There was a problem hiding this comment.
PR-URL: nodejs#11127 Reviewed-By: Timothy Gu <timothygu99@gmail.com> Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com> Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com> Reviewed-By: James M Snell <jasnell@gmail.com>
|
Landed in 3fffebb |
PR-URL: #11127 Reviewed-By: Timothy Gu <timothygu99@gmail.com> Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com> Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com> Reviewed-By: James M Snell <jasnell@gmail.com>
PR-URL: nodejs#11127 Reviewed-By: Timothy Gu <timothygu99@gmail.com> Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com> Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com> Reviewed-By: James M Snell <jasnell@gmail.com>
PR-URL: nodejs#11127 Reviewed-By: Timothy Gu <timothygu99@gmail.com> Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com> Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com> Reviewed-By: James M Snell <jasnell@gmail.com>
8 participants
Checklist
Affected core subsystem(s)
doc test