-
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
test: load testModules automatically #876
test: load testModules automatically #876
Conversation
@mhdawson I have added the multilevel traversal feature and the explicit |
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.
LGTM
@RaisinTen thanks for the recent PRs. I'll give the other team members a bit of time to review and try to land this late next week. |
eb14116
to
76b3cfc
Compare
PR-URL: #876 Reviewed-By: Michael Dawson <midawson@redhat.com>
Landed as 1c040ee |
PR-URL: nodejs/node-addon-api#876 Reviewed-By: Michael Dawson <midawson@redhat.com>
PR-URL: nodejs/node-addon-api#876 Reviewed-By: Michael Dawson <midawson@redhat.com>
PR-URL: nodejs/node-addon-api#876 Reviewed-By: Michael Dawson <midawson@redhat.com>
PR-URL: nodejs/node-addon-api#876 Reviewed-By: Michael Dawson <midawson@redhat.com>
No description provided.