Skip to content

Activate conda shell for gcovr lcov report generation step - #66

Merged
harryswift01 merged 1 commit into
mainfrom
65-bug-gcovr-not-found-in-generate-lcov-report-step
Sep 28, 2026
Merged

harryswift01 merged 1 commit into
mainfrom
65-bug-gcovr-not-found-in-generate-lcov-report-step

Conversation

@harryswift01

Copy link
Copy Markdown
Member

Summary

This PR fixes the cpp-coverage job in coveralls-main.yaml, where gcovr was not found on PATH when generating the lcov report.

Changes

Activate the conda shell for the lcov report step:

  • Added shell: bash -l {0} to the "Generate lcov report" step, matching every other step in this job that needs the conda-activated environment

Impact

  • No behavioural change to SIST itself, this is a CI-only fix
  • Fixes the C++ coverage reporting path in coveralls-main.yaml

@harryswift01 harryswift01 self-assigned this Sep 28, 2026
@harryswift01 harryswift01 added the bug Something isn't working label Sep 28, 2026

@jimboid jimboid left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah yeah I have come across this a couple of times. Nice catch.

@harryswift01
harryswift01 merged commit 02b76a7 into main Sep 28, 2026
15 checks passed
@harryswift01
harryswift01 deleted the 65-bug-gcovr-not-found-in-generate-lcov-report-step branch September 28, 2026 11:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: gcovr not found in Generate lcov report step due to missing shell activation

2 participants