Skip to content
This repository has been archived by the owner on Aug 30, 2021. It is now read-only.

Update Mocha #1200

Merged
merged 1 commit into from
Feb 11, 2016
Merged

Update Mocha #1200

merged 1 commit into from
Feb 11, 2016

Conversation

ilanbiala
Copy link
Member

Mocha caused Node 5 tests to fail (https://travis-ci.org/meanjs/mean/jobs/108313328)

@codydaig
Copy link
Member

Still erroring out:
http://puu.sh/n3e88/8cac83d59d.png
btw, they have the best error message ever.

@ilanbiala
Copy link
Member Author

Based on mochajs/mocha#2102 (comment), it looks like mock-fs is the issue. I'll look into that.

@ilanbiala
Copy link
Member Author

@codydaig let's get this merged in after we merge in #1184 so we make sure it works on all OSes.

@ilanbiala ilanbiala self-assigned this Feb 10, 2016
@ilanbiala ilanbiala added this to the 0.5.0 milestone Feb 10, 2016
@codydaig
Copy link
Member

#1184 has been merged.

@ilanbiala ilanbiala closed this Feb 11, 2016
@ilanbiala ilanbiala reopened this Feb 11, 2016
@ilanbiala
Copy link
Member Author

Awesome, closed an reopened so Travis will re-run.

@codydaig
Copy link
Member

@ilanbiala Passed. Squash, update commit message, and we can merge.

@rhutchison
Copy link
Contributor

LGTM

@ilanbiala
Copy link
Member Author

@codydaig if LGTY, go ahead and merge once tests pass.

lirantal added a commit that referenced this pull request Feb 11, 2016
@lirantal lirantal merged commit 7852144 into master Feb 11, 2016
@codydaig codydaig deleted the node5-support branch February 11, 2016 22:49
@nhhockeyplayer
Copy link

guys is this ready for primetime...mean for node-5.6 ? I would like to bang on it.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants