Skip to content
This repository has been archived by the owner on Jul 11, 2019. It is now read-only.

Some fixes in jacoco config #39

Merged
merged 1 commit into from
Apr 14, 2017
Merged

Some fixes in jacoco config #39

merged 1 commit into from
Apr 14, 2017

Conversation

pieterbos
Copy link
Member

set jacoco to ignore generated antlr-code - quite a lot will never ever be hit and it's not relevant

@codecov
Copy link

codecov bot commented Apr 14, 2017

Codecov Report

Merging #39 into master will increase coverage by 3.41%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##             master      #39      +/-   ##
============================================
+ Coverage     63.24%   66.66%   +3.41%     
+ Complexity     2315     2034     -281     
============================================
  Files           332      332              
  Lines         12304     7166    -5138     
  Branches       1399      903     -496     
============================================
- Hits           7782     4777    -3005     
+ Misses         3752     1966    -1786     
+ Partials        770      423     -347
Impacted Files Coverage Δ Complexity Δ
...ain/java/com/nedap/archie/adlparser/ADLParser.java 60.52% <0%> (+2.02%) 9% <0%> (-281%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6f3b31d...59e14d4. Read the comment docs.

@pieterbos pieterbos merged commit 613be29 into master Apr 14, 2017
@pieterbos pieterbos deleted the jacoco_fixes branch April 14, 2017 11:27
pieterbos added a commit that referenced this pull request Aug 21, 2018
Fix URL parsing in ODIN grammar to work with URls with no path or '/'…
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant