Adds support for loading reporter from an absolute or relative path. #2659

Merged
merged 1 commit into from Jan 10, 2017

Projects

None yet

3 participants

@sul4bh
Contributor
sul4bh commented Jan 4, 2017

This closes #2434

Created this PR based on @cletusw comment here: #2453 (comment)

@boneskull Please review.

@sul4bh sul4bh Adds support for loading reporter from an absolute or relative path.
265299d
@jsf-clabot
jsf-clabot commented Jan 4, 2017 edited

CLA assistant check
All committers have signed the CLA.

@Munter
Member
Munter commented Jan 4, 2017

Looks good to me

@sul4bh
Contributor
sul4bh commented Jan 4, 2017

Not sure why the build job #3029.7 failed. Probably a one-off failure.

@Munter
Member
Munter commented Jan 4, 2017

We've been having some false negatives on the integrations tests lately. I've opened #2660 to try and resolve it. Also restarted the failing travis test

@sul4bh
Contributor
sul4bh commented Jan 9, 2017

Any update on this?

@Munter Munter merged commit 8a3cab0 into mochajs:master Jan 10, 2017

3 checks passed

continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
licence/cla Contributor License Agreement is signed.
Details
@Munter Munter removed the needs-review label Jan 10, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment