Skip to content

Conversation

@LittleHuba
Copy link
Contributor

No description provided.

@LittleHuba LittleHuba force-pushed the ulhu_extend_ci_jobs branch 15 times, most recently from 4841d8b to 0a32c05 Compare December 18, 2025 08:53
# Workflow configuration for S-CORE CI - Bazel Build & Test communication module for target platforms
# This workflow runs Bazel build and test for QNX when triggered by specific pull request events.

name: Bazel Build & Test communication module (target-platforms)
Copy link
Contributor

Choose a reason for hiding this comment

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

should this use reusable QNX8 workflow from https://github.com/eclipse-score/cicd-workflows ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Unfortunately I can't use the upstream workflows for various reasons.
Though, I want to upstream fixes to cicd-workflows to slowly lift this restriction.

Copy link
Contributor

Choose a reason for hiding this comment

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

You mean you will create PRs to reusable workflows?

For our current matrix builds we are inerested in the outcome of every branch independent of the other branches.
Build fails because the rust toolchain is not hermetic.
@LittleHuba LittleHuba force-pushed the ulhu_extend_ci_jobs branch 4 times, most recently from 5a86dd4 to 2b83664 Compare December 19, 2025 09:07
@LittleHuba LittleHuba force-pushed the ulhu_extend_ci_jobs branch 3 times, most recently from c040741 to a6a5948 Compare December 19, 2025 11:05
LittleHuba and others added 4 commits December 19, 2025 12:13
execute tests and store artifacts
We do not want to duplicate the logic of the workflows.
Since we have the logic except for coverage reports already as part
of other workflows we reduce this workflow to only create a coverage report.

All the other workflows we enable to also be triggered by a release creation.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants