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

Remove unused code from base reporter #2681

Closed
wants to merge 1 commit into from

Conversation

epallerols
Copy link
Contributor

I am doing some clean up. There is no use case for escape=true so I have removed everything that is related to that case.

@coveralls
Copy link

Coverage Status

Changes Unknown when pulling 41ced48 on epallerols:remove-dead-code into ** on mochajs:master**.

@dasilvacontin dasilvacontin added the status: waiting for author waiting on response from OP - more information needed label Apr 20, 2017
@stale stale bot added the stale this has been inactive for a while... label Jul 31, 2017
@stale
Copy link

stale bot commented Jul 31, 2017

I am a bot that watches issues for inactivity.
This issue hasn't had any recent activity, and I'm labeling it stale. In 14 days, if there are no further comments or activity, I will close this issue.
Thanks for contributing to Mocha!

@stale stale bot closed this Aug 14, 2017
@ScottFreeCode ScottFreeCode added status: needs review a maintainer should (re-)review this pull request and removed status: waiting for author waiting on response from OP - more information needed labels Aug 17, 2017
@boneskull boneskull removed the status: needs review a maintainer should (re-)review this pull request label Dec 12, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
stale this has been inactive for a while...
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants