Skip to content
This repository has been archived by the owner on Mar 21, 2022. It is now read-only.

More verbose test reporter #73

Merged
merged 1 commit into from Nov 30, 2018

Conversation

brodybits
Copy link

thanks to @raphinesse (apache/cordova-lib@270310e)

Travis CI passes for these changes in my own fork (https://travis-ci.org/brodybits/cordova-osx/builds/460337335).

@brodybits brodybits changed the title More verbose test reporter [WIP] More verbose test reporter Nov 27, 2018
@brodybits
Copy link
Author

Marked [WIP] pending rebase on master, with Travis CI enabled (#71).

I would like to wait until Travis CI is enabled before rebasing to get a green PR.

Co-authored-by: Raphael von der Grün <raphinesse@gmail.com>
Co-authored-by: Christopher J. Brody <chris.brody@gmail.com>
@brodybits brodybits changed the title [WIP] More verbose test reporter More verbose test reporter Nov 28, 2018
@brodybits
Copy link
Author

Build is now green after rebasing on master, quick review would be appreciated.

@brodybits brodybits mentioned this pull request Nov 28, 2018
Copy link

@knight9999 knight9999 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

So nice.
This works well on my mac.

@brodybits brodybits merged commit 715ef0d into apache:master Nov 30, 2018
@brodybits brodybits deleted the more-verbose-test-reporter branch November 30, 2018 14:27
@brodybits
Copy link
Author

Thanks @knight9999 for the review

brodybits pushed a commit to brodybits/cordova-osx that referenced this pull request Dec 4, 2018
(was added to the wrong place in 715ef0d - PR apache#73)
brodybits pushed a commit that referenced this pull request Dec 4, 2018
(was added to the wrong place in 715ef0d - PR #73)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants