Skip to content

Commit

Permalink
Add VER to the list of known LCov directives (#414)
Browse files Browse the repository at this point in the history
* Fix description for LcovCoverageReporter

* Add VER to the list of known LCov directives

Add VER directive to the list of known directives to support the LCov reports with it. VER is an optional directive that can be added by some tools. For example, gcovr is adding it right now.
  • Loading branch information
esuldin authored Aug 29, 2024
1 parent c3ae7dd commit a7caeee
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion diff_cover/violationsreporters/violations_reporter.py
Original file line number Diff line number Diff line change
Expand Up @@ -264,7 +264,7 @@ def measured_lines(self, src_path):

class LcovCoverageReporter(BaseViolationReporter):
"""
Query information from a Cobertura|Clover|JaCoCo XML coverage report.
Query information from a LCov coverage report.
"""

def __init__(self, lcov_roots, src_roots=None):
Expand Down Expand Up @@ -324,6 +324,7 @@ def parse(lcov_file):
"BRF",
"BRH",
"BRDA",
"VER",
]:
# these are valid lines, but not we don't need them
continue
Expand Down

0 comments on commit a7caeee

Please sign in to comment.