-
Notifications
You must be signed in to change notification settings - Fork 17
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
Smoke testing with npm
modules
#14
Comments
@indutny are you thinking that this module should be included in citgm? or did you have something else in mind? |
@thealphanerd it may be a good idea to include it in cigtm eventually, indeed. However, I was thinking about doing the smoke tests for node.js addons. Added you to cc, because you have tons of experience with smoke testing 😉 |
definitely interested in helping with this. If you can come up with a list of requirements I can definitely start digging in. Specifically outlining the manual process for |
@thealphanerd thank you! What about something like this:
|
So citgm allows you to provide a custom lookup table, so we could easily make a lookup table of only native modules. Is there a flag or env var that can be passed to npm to use a specific gyp? If so I think this should be pretty painless to do with citgm, we may even be able to do it with the off the shelf citgm |
@thealphanerd sounds lovely! I'll get back to you once we will get things moving with node-gyp. Thank you so much for your help! |
One of the main goals for this project is to be used in the node-gyp. This means, though, that we should make sure that the most of the existing
binding.gyp
files will work withgyp.js
. Thus, "smoke testing" is needed badly.cc @thealphanerd
The text was updated successfully, but these errors were encountered: