-
Notifications
You must be signed in to change notification settings - Fork 58
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
Adding to CITGM #51
Comments
Very cool to hear! Thanks for the offer. I can try, but @stevelacy, will you be able to help too? |
I can assist |
Is there anything left to be done for this? You can put me as the maintainer. |
Closing since I didn't get a response |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Hey,
We'd like to add
require-dir
to Node.js's Canary In The Gold Mine. (currently tested packages)This means new releases of Node will run
require-dir
's unit tests as integration tests and should prevent issues like #45 from happening in the future.It would have let us catch the issue back at nodejs/node#10789
In order for us to do that - we need @aseemk (or another maintainer) to agree to be reachable and responsive for problems (as they may influence Node version releases directly).
@aseemk what do you say? Would you be up for it? It would have certainly saved me time personally if
require-dir
was there :)The text was updated successfully, but these errors were encountered: