Skip to content
This repository has been archived by the owner on Oct 1, 2020. It is now read-only.

Upgrade JaCoCo to 0.7.7 and ASM to 5.1 #121

Merged
merged 1 commit into from
Jun 7, 2016
Merged

Upgrade JaCoCo to 0.7.7 and ASM to 5.1 #121

merged 1 commit into from
Jun 7, 2016

Conversation

Godin
Copy link
Member

@Godin Godin commented May 26, 2016

No description provided.

@Godin Godin added this to the 2.3.4 milestone May 26, 2016
@Godin Godin force-pushed the issue-121 branch 2 times, most recently from 6c7d724 to 082e0b6 Compare May 26, 2016 23:05
@Godin
Copy link
Member Author

Godin commented Jun 3, 2016

@marchof I've used this PR to check report-aggregate and prepare for upgrade to JaCoCo 0.7.7. Do you think that this change in tests due jacoco/jacoco#387 might have functional consequences?

@Godin Godin changed the title Upgrade to JaCoCo 0.7.7 Upgrade JaCoCo to 0.7.7 and ASM to 5.1 Jun 3, 2016
@marchof
Copy link
Member

marchof commented Jun 3, 2016

@Godin No, I don't think jacoco/jacoco#387 has an impact on EclEmma as exec file processing is fully handled within JaCoCo. Also this issue did not change the file format as such.

@Godin
Copy link
Member Author

Godin commented Jun 6, 2016

ready to be merged, @marchof could you please review?

@marchof
Copy link
Member

marchof commented Jun 7, 2016

LGTM!

@marchof marchof merged commit 5474348 into master Jun 7, 2016
@marchof marchof deleted the issue-121 branch June 7, 2016 21:40
@marchof
Copy link
Member

marchof commented Jun 7, 2016

@Godin Sorry, I didn't select squash.

@jacoco jacoco locked and limited conversation to collaborators Jan 11, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants