-
Notifications
You must be signed in to change notification settings - Fork 3k
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
Spin off testharness? #13072
Comments
I'm not sure I understand the provenance of this issue. But I think that In terms of the repository, it used to be in a separate repo, but that turned out to have substantial overhead when making changes, so I'd be extremely reluctant to split it out again (this is linked to the goal of being the best thing for wpt tests specifically). |
@jgraham: I actually agree with you. I was filing this issue “on behalf” of others who had expressed interest in bringing I'm waiting to see if somebody steps forward to defend the merits of this idea — otherwise, I'm happy to close this issue. |
I don't think we need a separate repo for testharness.js in order to create an npm package.... |
Okay by me; whatever the maintainer(s) find most appropriate :) |
Closing this, since there seems to be consensus about testharness not needing to live in a separate GH repository (but I still think it'll have to be published independently via npm if you guys want any other project to adopt it in the future). |
ie, maintain
testharness.js
as a separate project and publish releases on npm.(I think that's a minimum requirement if the authors want it to be adopted by others.)
cf w3c/nodejs#4
The text was updated successfully, but these errors were encountered: