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

Jenkins 16580: Display more than only line coverage in graph #50

Merged
merged 10 commits into from
May 30, 2016

Conversation

mheinzerling
Copy link
Collaborator

@jenkinsadmin
Copy link
Member

Thank you for a pull request! Please check this document for how the Jenkins project handles pull requests

@mheinzerling mheinzerling force-pushed the JENKINS-16580 branch 2 times, most recently from b4ea47d to 047dfb6 Compare December 8, 2014 08:09
@mheinzerling
Copy link
Collaborator Author

Fixed the tests to run independent of the system locale and fonts.

@centic9 centic9 changed the title Jenkins 16580 Jenkins 16580: Display more than only line coverage in graph Feb 24, 2015
@mheinzerling mheinzerling reopened this May 25, 2016
@mheinzerling
Copy link
Collaborator Author

Rebased.

@centic9
Copy link
Member

centic9 commented May 25, 2016

it seems you rebased onto changes done in PR66, however this makes it impossible to merge this onto master currently before PR66 is done, was this intentional?

@mheinzerling
Copy link
Collaborator Author

Kind of. This changes are nearly 2 years old and aren't commented or integrated yet. I need the workflow support in my local version, so it is at least useful for me.

@centic9
Copy link
Member

centic9 commented May 25, 2016

Hi,

Are you interested in working on the jacoco-plugin directly? I don't know
much about Jenkins-plugin development and can only spend very little time
on it so it is a bit stale and could use some more man-power.

Dominik.

On Wed, May 25, 2016 at 3:59 PM, mheinzerling [email protected]
wrote:

Kind of. This changes are nearly 2 years old and aren't commented or
integrated yet. I need the workflow support in my local version, so it is
at least useful for me.


You are receiving this because you commented.
Reply to this email directly or view it on GitHub
#50 (comment)

@mheinzerling
Copy link
Collaborator Author

Rebased on master

@centic9 centic9 merged commit 04bdfe9 into jenkinsci:master May 30, 2016
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.

3 participants