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

TAP support #27

Closed
sindresorhus opened this issue Aug 28, 2015 · 9 comments · Fixed by #326
Closed

TAP support #27

sindresorhus opened this issue Aug 28, 2015 · 9 comments · Fixed by #326
Assignees

Comments

@sindresorhus
Copy link
Member

Should be a flag, like $ ava --tap, to get TAP output. This would be a good way to enable reporters without actually supporting reporters. Can just refer people to all the available TAP reporters instead of reinventing the wheel.

@scottcorgan
Copy link

I know that a PR is better than this question, BUT, is this still a priority?

@jprichardson
Copy link

👍

@sindresorhus sindresorhus added this to the 0.6.0 milestone Nov 17, 2015
@jamestalmage
Copy link
Contributor

From my reading of the TAP spec, I it does not seem very well suited to concurrency. You would need to buffer individual tests and then log them all at once so they don't walk all over each-other.

@sindresorhus sindresorhus modified the milestones: 0.6.0, 0.7.0 Nov 24, 2015
@vadimdemedes vadimdemedes mentioned this issue Nov 28, 2015
4 tasks
@vadimdemedes
Copy link
Contributor

Coming Soon!

screen shot 2015-11-29 at 7 18 50 pm

@vadimdemedes
Copy link
Contributor

TAP support is ready, waiting for #279.

@jprichardson
Copy link

After I saw this closed by @vdemedes I assumed it was part of 0.7.0. But didn't see it listed https://github.com/sindresorhus/ava/releases/tag/v0.7.0. Since this issue is still part of 0.7.0, does that mean that it's in 0.7.0 and hence forgotten about in the release announcement or that a new milestone should be associated with this issue?

@sindresorhus sindresorhus modified the milestones: 0.8.0, 0.7.0 Nov 30, 2015
@jamestalmage
Copy link
Contributor

@jprichardson - We decided to push #279 to the next milestone.

@sindresorhus
Copy link
Member Author

It was closed by a mistake. Planned for 0.8.0.

@vadimdemedes vadimdemedes mentioned this issue Dec 12, 2015
6 tasks
@jonathanKingston
Copy link

@jamestalmage From my reading of the TAP spec, I it does not seem very well suited to concurrency.

That is the case certainly, I am happy to discuss some of my suggestions that have been aired on TAP specification discussions however likely won't be adopted very quickly at all (Having a different output might be simpler)

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

Successfully merging a pull request may close this issue.

6 participants