-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
can't run the tests #313
Comments
It looks like there's a regression in the latest jasmine-node. I noticed On July 3, 2014 6:11:13 PM Andrew Kelley notifications@github.com wrote:
|
I recommend locking the major and minor versions of dependencies, including devDependencies. |
We typically do lock devDepencies to major versions-- jasmine-node was not. That said, locking to any version of jasmine-node in this case would not have resolved this failure as the regression occurred in a dependent package of jasmine-node. The only way we could have avoided this would be by generating a shrinkwrap, which unfortunately is not feasible. The workaround for now would be to run: npm install jasmine-reporters@1.x |
Fixed by a temporary adjustment to the package.json in db52a5b. I've actually been working on a long-running branch to move over to mocha, so I will work on that as a permanent fix. |
This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs and link to relevant comments in this thread. |
The text was updated successfully, but these errors were encountered: