Skip to content

Commit

Permalink
Exclude some directories from coverage (root-project#13596)
Browse files Browse the repository at this point in the history
  • Loading branch information
AniqJaved authored and maksgraczyk committed Jan 12, 2024
1 parent 079312d commit 42e5108
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 3 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/root-ci-config/build_root.py
Original file line number Diff line number Diff line change
Expand Up @@ -371,7 +371,7 @@ def rebase(base_ref, head_ref) -> None:
def create_coverage_xml() -> None:
result = subprocess_with_log(f"""
cd '{WORKDIR}/build'
gcovr --cobertura-pretty --gcov-ignore-errors=no_working_dir_found --merge-mode-functions=merge-use-line-min -r ../src ../build -o cobertura-cov.xml
gcovr --output=cobertura-cov.xml --cobertura-pretty --gcov-ignore-errors=no_working_dir_found --merge-mode-functions=merge-use-line-min --exclude-unreachable-branches --exclude-directories="roottest|runtutorials|interpreter" --exclude='.*/G__.*' --exclude='.*/(roottest|runtutorials|externals|ginclude|googletest-prefix|macosx|winnt|geombuilder|cocoa|quartz|win32gdk|x11|x11ttf|eve|fitpanel|ged|gui|guibuilder|guihtml|qtgsi|qtroot|recorder|sessionviewer|tmvagui|treeviewer|geocad|fitsio|gviz|qt|gviz3d|x3d|spectrum|spectrumpainter|dcache|hdfs|foam|genetic|mlp|quadp|splot|memstat|rpdutils|proof|odbc|llvm|test|interpreter)/.*' --gcov-exclude='.*_ACLiC_dict[.].*' '--exclude=.*_ACLiC_dict[.].*' -v -r ../src ../build
""")

if result != 0:
Expand Down
3 changes: 1 addition & 2 deletions .github/workflows/test-coverage.yml
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,7 @@ env:
PYTHONUNBUFFERED: true

jobs:

build-linux:
if: github.repository_owner == 'root-project' || github.event_name == 'pull_request'

Expand Down Expand Up @@ -83,7 +83,6 @@ jobs:
run: 'printf "%s@%s\\n" "$(whoami)" "$(hostname)";
ls -la
'


- name: Apply option override from matrix for this job
env:
Expand Down

0 comments on commit 42e5108

Please sign in to comment.