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

Added to the teamcity reporter to include stdout and suite names #954

Closed
wants to merge 1 commit into from

Conversation

danjenkins
Copy link

Also include the expected / actual results

run with reporter teamcity

And reference http://confluence.jetbrains.com/display/TCD7/Build+Script+Interaction+with+TeamCity#BuildScriptInteractionwithTeamCity-ReportingTests for standards

@tj
Copy link
Contributor

tj commented Aug 18, 2013

gah lol we need to move this one out of core it always seems to be wrong

@danjenkins
Copy link
Author

To be honest, I completely missed the fact it was in the core to start and
I started making my own, then I saw it in the source as I was looking at
other reporters!

Wouldn't mind taking it out into its own module...

Dan

On Aug 18, 2013, at 21:01, TJ Holowaychuk notifications@github.com wrote:

gah lol we need to move this one out of core it always seems to be wrong


Reply to this email directly or view it on
GitHubhttps://github.com//pull/954#issuecomment-22837968
.

@danjenkins
Copy link
Author

What's happening with this? do we want to take it out of core? I don't mind doing that if that's what people want to do, then i'll put in another PR to remove tc from core

@tj
Copy link
Contributor

tj commented Aug 21, 2013

that would be great if you have time! we just need to remove it and maybe add a console.warn() with the url and npm package name of yours so people can install that

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.

2 participants