-
Notifications
You must be signed in to change notification settings - Fork 30
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
Feature request: support for the custom node_modules/ and bower_components/ location #10
Comments
For npm this only concerns global packages which are beyond the scope of this module. For local packages this is unsupported and there have been public declarations that it's not ever planned. For Bower I already support this configuration in Am I missing something? |
Sorry, I missed this information about npm.
Before I wrote this issue, I've tested this by guess in my project (where bower dependencies are already installed), and got the wrong result "depsWereOk: false", when it should be true. So I decided that your library simply ignores My {
...
"devDependencies": {
"susy": "2.2.0.beta.3",
"sass-meyer-reset": "*",
"breakpoint-sass": "~2.5.0"
}
} The check-dependencies verbose output: susy: installed: undefined, expected: 2.2.0.beta.3
sass-meyer-reset: installed: undefined, expected: *
Invoke bower install to install missing packages The content of the susy's {
"name": "susy",
...
"devDependencies": {
"true": "2.0.0.alpha.3"
},
"_release": "2.2.0.beta.3",
"_resolution": {
"type": "tag",
"tag": "2.2.0.beta.3",
"commit": "8677fb51a4513b22db41d0d43f48c8a36ee937f4"
},
"_source": "git://github.com/ericam/susy.git",
"_target": "2.2.0.beta.3",
"_originalSource": "susy"
} The content of the sass-meyer-reset's {
"name": "sass-meyer-reset",
"homepage": "https://github.com/ryanburnette/sass-meyer-reset",
"_release": "1dc79b9776",
"_resolution": {
"type": "branch",
"branch": "master",
"commit": "1dc79b977685b4240f8971987293938fbb4e602c"
},
"_source": "git://github.com/ryanburnette/sass-meyer-reset.git",
"_target": "*",
"_originalSource": "sass-meyer-reset",
"_direct": true
} — there is some problem with matching versions of installed packages. |
The problem seems to be that in |
I could ignore packages with unspecified |
It seems that repository, from which I installed this package, doesn't contain a bower.json at all: https://github.com/ryanburnette/sass-meyer-reset But nonetheless, this package is registered in Bower:
P. S. For susy I set specific version (2.2.0.beta.3) by own, and this doesn't work too:
|
A Bower package that doesn't have |
Sorry for the late reply.
According to this: http://bower.io/docs/creating-packages/#register these packages aren't valid (but can be registered though). The sass-meyer-reset's repository doesn't contain a valid
There is a valid https://github.com/ericam/susy/blob/master/bower.json I think the problem is same — the tag But there is a bower install susy#2.2.0.beta.3 {
"name": "susy",
...
"_release": "2.2.0.beta.3",
...
} and bower install susy#2.2.2 {
"name": "susy",
"version": "2.2.2",
...
"_release": "2.2.2",
...
} I've created a new issue related to this problem and continued the discussion here — #11 Closing this for now. |
There is a possibility (both for npm and Bower) to install packages to the custom folder:
npm local install package to custom location
How to change bower's default components folder?
— I think it would be great to introduce a new option, which will set the
depsDir
variable:https://github.com/mzgol/check-dependencies/blob/master/lib/check-dependencies.js#L104
The text was updated successfully, but these errors were encountered: