Skip to content

Add C++ coverage reusable workflow#63

Merged
FScholPer merged 1 commit intoeclipse-score:mainfrom
RSingh1511:rs/swp-243749
Feb 19, 2026
Merged

Add C++ coverage reusable workflow#63
FScholPer merged 1 commit intoeclipse-score:mainfrom
RSingh1511:rs/swp-243749

Conversation

@RSingh1511
Copy link
Contributor

No description provided.

Copy link
Contributor

@dcalavrezo-qorix dcalavrezo-qorix left a comment

Choose a reason for hiding this comment

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

lgtm

@FScholPer FScholPer merged commit 773e90a into eclipse-score:main Feb 19, 2026
2 checks passed
disk-cache: ${{ github.workflow }}
repository-cache: true
bazelisk-cache: true
cache-save: ${{ github.event_name != 'pull_request' }}
Copy link
Contributor

@pawelrutkaq pawelrutkaq Feb 24, 2026

Choose a reason for hiding this comment

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

shall be cache-save: ${{ github.event_name == 'push' }} and maybe only on maib. If not only on main then disk-cache: ${{ github.workflow }} must contain a branch in name.

repository-cache: true
bazelisk-cache: true
cache-save: ${{ github.event_name != 'pull_request' }}

Copy link
Contributor

Choose a reason for hiding this comment

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

Maybe we shall put somewhere that if user wants to use CACHE it has to do in .bazelrc https://github.com/eclipse-score/reference_integration/blob/main/.bazelrc#L84

uses: actions/upload-artifact@v6
with:
name: ${{ format('{0}_cpp_test_logs{1}', github.event.repository.name, inputs.artifact-name-suffix) }}
path: |
Copy link
Contributor

Choose a reason for hiding this comment

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

shoudnt this 3 steps be just one step ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Development

Successfully merging this pull request may close these issues.

4 participants