-
Notifications
You must be signed in to change notification settings - Fork 2
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
ASAP 401 Fix #90
base: master
Are you sure you want to change the base?
ASAP 401 Fix #90
Conversation
This looks good to me -- it fixes a dependency issue that has been cropping up for lots of folks. |
I guess I should actually TEST Butler with these updated packages, though... |
When I run {
"name": "medidata-styleguide",
"version": "1.0.0",
"description": "Styleguide for Medidata.",
"main": "index.js",
"scripts": {
"butler": "node_modules/butler/node_modules/.bin/gulp --gulpfile node_modules/butler/Gulpfile.js",
"develop": "node_modules/butler/node_modules/.bin/gulp --gulpfile node_modules/butler/Gulpfile.js develop",
"test": "node_modules/butler/node_modules/.bin/gulp --gulpfile node_modules/butler/Gulpfile.js test",
"deploy": "node_modules/butler/node_modules/.bin/gulp --gulpfile node_modules/butler/Gulpfile.js spress-deploy"
},
"repository": {
"type": "git",
"url": "git+ssh://[email protected]/palantirnet/medidata-styleguide.git"
},
"author": "Palantir.net, Inc. <[email protected]>",
"license": "ISC",
"bugs": {
"url": "https://github.com/palantirnet/medidata-styleguide/issues"
},
"homepage": "https://github.com/palantirnet/medidata-styleguide#readme",
"dependencies": {
"butler": "git://github.com/palantirnet/butler.git#asap-404-fix"
}
} I get an error related to the 'ajv' module:
Do I need to run this with a particular version of node? |
... and when I re-shrinkwrap myself, the branch still doesn't install in a project. This is probably happening because the drupalbox is using a pretty old version of NodeJs. |
@becw I've had no problem installing things on UMUS using this branch, and I'm not able to use it otherwise. Could you test again to see if you get the same error as above so we can try and get this merged? |
Updates shrinkwrap to remove dependency on unavailable uberinternal repo.