Skip to content
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

fix(reporter/base): explicitly ignore showDiff #1614 #1626

Merged
merged 1 commit into from
Mar 26, 2015

Conversation

a8m
Copy link
Contributor

@a8m a8m commented Mar 25, 2015

Ignore showDiff only if it's explicitly set to false.
see #1614 , #810

@danielstjules
Copy link
Contributor

🎉 :shipit:

Also, those specs are a perfect example of why we should fix #1582 If Base.list failed, process.stdout.write wouldn't be restored. And we can't just do it from afterEach because then the reporter won't print the spec at all. I'm gonna tackle that tonight. :)

@mantoni
Copy link
Contributor

mantoni commented Mar 26, 2015

Thanks @a8m 👍

@dasilvacontin
Copy link
Contributor

LGTM.

dasilvacontin added a commit that referenced this pull request Mar 26, 2015
fix(reporter/base): explicitly ignore showDiff #1614
@dasilvacontin dasilvacontin merged commit ec6440e into mochajs:master Mar 26, 2015
@dasilvacontin
Copy link
Contributor

thanks guys!

@a8m
Copy link
Contributor Author

a8m commented Mar 26, 2015

Thanks @dasilvacontin

@dominicbarnes
Copy link
Contributor

Awesome! Was just wondering about this and glad to see it's been addressed. How long until a release?

@dasilvacontin
Copy link
Contributor

Euh, best way to force a sooner release is to make the releasers aware of the need. 😄

/cc @travisjeffery @boneskull

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

Successfully merging this pull request may close these issues.

5 participants