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

Add few tap reporters #213

Merged
merged 3 commits into from
Nov 16, 2015
Merged

Add few tap reporters #213

merged 3 commits into from
Nov 16, 2015

Conversation

axross
Copy link
Contributor

@axross axross commented Nov 15, 2015

I created two TAP reporters. Could you add these to README#pretty reporters?

tap-notify

the Notifier for the TAP. This pass through TAP messages to stdout. You can use as tape *.test.js | tap-notify | tap-some-reporter.

screenshot1

screenshot2

tap-diff

Simple reporter. This shows diffs when assertion failed.

screenshot3

screenshot4

@Raynos
Copy link
Collaborator

Raynos commented Nov 16, 2015

Can we remove the white space commit ?

The extra projects look great.

@axross
Copy link
Contributor Author

axross commented Nov 16, 2015

@Raynos

Thank you so much!
I reverted a commit Remove unneeded whitespaces in README. Could you check it?

Raynos added a commit that referenced this pull request Nov 16, 2015
@Raynos Raynos merged commit 0d078f3 into tape-testing:master Nov 16, 2015
@Raynos
Copy link
Collaborator

Raynos commented Nov 16, 2015

Thanks! :)

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.

None yet

2 participants