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

Fix for JENKINS-17438 #4

Merged
merged 4 commits into from Apr 8, 2013

Conversation

@bobtheshrew
Copy link
Contributor

commented Apr 5, 2013

Updated getXUnitToolInfoObject() to include:
 * BuildListener parameter
 * Use Util.replaceMacro() to expand CustomXSL string

Updated performTests() call to getXUnitToolInfoObject() to include BuildListener parameter
 * Searched rest of the code, this seems to be the only call of getXUnitToolInfoObject()

Addresses Bug: https://issues.jenkins-ci.org/browse/JENKINS-17438

Updated getXUnitToolInfoObject() to include:
 * BuildListener parameter
 * Use Util.replaceMacro() to expand CustomXSL string

Updated performTests() call to getXUnitToolInfoObject() to include BuildListener parameter
 * Searched rest of the code, this seems to be the only call of getXUnitToolInfoObject()

Addresses Bug: https://issues.jenkins-ci.org/browse/JENKINS-17438
@buildhive

This comment has been minimized.

Copy link

commented Apr 5, 2013

Jenkins » xunit-plugin #33 FAILURE
Looks like there's a problem with this pull request
(what's this?)

removed extra parenthesis causing build failure
@buildhive

This comment has been minimized.

Copy link

commented Apr 5, 2013

Jenkins » xunit-plugin #34 FAILURE
Looks like there's a problem with this pull request
(what's this?)

@buildhive

This comment has been minimized.

Copy link

commented on 709154e Apr 5, 2013

Jenkins » xunit-plugin #34 FAILURE
Looks like this commit caused a build failure
(what's this?)

@buildhive

This comment has been minimized.

Copy link

commented Apr 5, 2013

Jenkins » xunit-plugin #35 FAILURE
Looks like there's a problem with this pull request
(what's this?)

@buildhive

This comment has been minimized.

Copy link

commented on 30f377d Apr 5, 2013

Jenkins » xunit-plugin #35 FAILURE
Looks like this commit caused a build failure
(what's this?)

@buildhive

This comment has been minimized.

Copy link

commented Apr 5, 2013

Jenkins » xunit-plugin #36 SUCCESS
This pull request looks good
(what's this?)

gboissinot added a commit that referenced this pull request Apr 8, 2013
@gboissinot gboissinot merged commit 6f0d4a8 into jenkinsci:master Apr 8, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants
You can’t perform that action at this time.