-
-
Notifications
You must be signed in to change notification settings - Fork 165
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
publish dependency testing scripts as module? #650
Comments
I think that's a great idea, mind if I add it under the SSBC org instead? |
sure, go ahead |
I needed this for testing flume, so I did it: https://github.com/ssbc/compatibility |
Oh, I fixed up some things and was working on that in this branch: https://github.com/ssbc/ssb-server/tree/extract-pretest I defaulted to testing all dependencies and was waiting until we could get pull-stream using compatible versions again. |
@christianbundy oh I didn't realize. did you change the logic about how you check dependencies? I think we should set a concrete goal: getting someone who is maintaining an alternate ssb-server distribution (i.e. @mixmix ;) to run the tests as part of their release cycle. see ssbc/patchbay#340 |
@dominictarr Yeah, by default this runs tests for all parent modules (e.g. patchbay) who have intersecting dependencies with child modules (e.g. ssb-server) so that you don't have to hardcode any deps you'd like to test. For example:
I was thinking I'd likely add back the customization of an allowlist/blocklist, but I wanted to make sure I got all of the logic working correctly. I think we're on the same page, my stretch goal was just to make it so that no customization was needed (because it's so easy to forget to add stuff to a hardcoded list). |
okay, but the problem I'm encountering is that your script was quite pedantic about ensuring that the extra devdeps didn't conflict. in many cases you have bumped a major version, but it doesn't mean it's actually broken a particular dep. what if it only added extra deps but didn't over ride deps the root module has hard coded? |
Is this still relevant? If so, what is blocking it? Is there anything you can do to help move it forward? |
@christianbundy I think we should publish the various scripts we have to test deps (scripts/pretest.js and test/deps.sh) as a module that makes this pattern easy to reuse.
I would do it but christian you wrote most of it so I'd rather see your name on it on npm
The text was updated successfully, but these errors were encountered: