Feature/make latest mains working for 0.6#87
Conversation
There was a problem hiding this comment.
Pull request overview
Updates the integration workspace so the latest main branches/modules build and run correctly against the 0.6 setup, including refreshed module pins and improved CI reporting.
Changes:
- Update module/toolchain dependencies (incl. Python rules and Rust toolchains) and refresh “known good” SHAs.
- Make integration test logs/summaries config-specific and publish a consolidated CI summary via a new script.
- Adjust build/test target sets (notably for
score_feo) and CI workflow defaults/secrets handling.
Reviewed changes
Copilot reviewed 15 out of 15 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
| scripts/run_unit_tests.sh | Adjusts feo unit test invocation to exclude a formatting check target. |
| scripts/publish_integration_summary.py | Adds a script to generate a consolidated GitHub Actions step summary from per-config build summaries. |
| scripts/integration_test.py | Makes log and summary filenames config-specific to avoid overwrites in matrix builds. |
| score_toolchains.MODULE.bazel | Adds additional GCC/QCC toolchains and switches Rust toolchain provisioning to score_toolchains_rust. |
| score_modules.MODULE.bazel | Updates git_override commit pins for several Score modules. |
| known_good.json | Refreshes pinned SHAs and adds score_baselibs_rust to the known-good set. |
| integration/bazel_common/score_modules.MODULE.bazel | Adds score_itf dependency to shared integration module set. |
| integration/MODULE.bazel | Removes redundant score_itf dep (now included via bazel_common). |
| build_config.json | Adds score_baselibs_rust build group and refines score_feo build targets/exclusions. |
| README.md | Expands CLI equivalents for workspace setup steps and links to integration folder docs. |
| MODULE.bazel | Bumps rules_python, updates score_crates, and adds a single_version_override. |
| .github/workflows/reusable_smoke-test.yml | Inherits secrets, adds checkout for summary publishing, and switches to Python summary script. |
| .github/workflows/reusable_integration-build.yml | Updates default integration config name. |
| .bazelrc | Removes a couple of logging flags and adds an extra Rust toolchain for x86_64-linux builds. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
qnx simply fails to download. documentation... its broken somewhere... again :/ |
|
The created documentation from the pull request is available at: docu-html |
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 15 out of 15 changed files in this pull request and generated 1 comment.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| gcc.toolchain( | ||
| name = "score_gcc_toolchain", | ||
| target_cpu = "x86_64", | ||
| target_os = "linux", | ||
| version = "12.2.0", | ||
| use_default_package = True, | ||
| ) |
There was a problem hiding this comment.
The toolchain name score_gcc_toolchain is defined on line 22 but is not included in the use_repo() call. Instead, score_gcc_x86_64_toolchain is referenced in the use_repo() on line 54, but this toolchain is never defined. Either rename the toolchain definition on line 22 from score_gcc_toolchain to score_gcc_x86_64_toolchain, or update the use_repo() call to reference score_gcc_toolchain instead of score_gcc_x86_64_toolchain.
Next time jsut rerun QNX please. Dont merge red PRs. at least for code pipelines. |
3 participants
No description provided.