adopt verification report - #330
Open
FScholPer wants to merge 27 commits into
Open
Conversation
License Check Results🚀 The license check job ran with the Bazel command: bazel run --lockfile_mode=error //:license-checkStatus: Click to expand output |
|
The created documentation from the pull request is available at: docu-html |
* verify version in MODULE.bazel * add dry_run support to release workflow * fix: double-zip in upload-artifact by uploading directory * update actions/checkout to v7 and setup-bazel to 0.19 (Node 24) * add job summary and annotations to release verification * version-stamp zip and artifact names with release version * mark dry-run artifacts with _dry-run suffix * fix: coverage n/a by capturing genhtml stdout into env vars * fix: Rust coverage via ferrocene-coverage, fix coverage regex for //:rust_coverage output * ci: tee job summary to stdout for CLI retrieval
* drop MODULE version and simplify release workflow * sync MODULE.bazel.lock after module change
antonkri
previously approved these changes
Jul 13, 2026
* cicd: fix qnx buid workflow * cicd: use hash in copyright workflow and removed merge_group trigger
* ci: update bzlmod lock workflow to v0.0.2 and PR-only Switch the reusable workflow reference from v0.0.0 to v0.0.2 and drop redundant working-directory input. Limit triggers to pull_request events since main is PR-protected and merge queue is not enabled, reducing unnecessary CI runs. * ci: pin bzlmod lock workflow to v0.0.2 commit hash Use the immutable commit SHA for bzlmod-lock-check and keep the concise #v0.0.2 tag annotation for readability.
umaucher
requested changes
Jul 23, 2026
6 tasks
actions/checkout v4.4.0 backported the pull_request_target fork-checkout block, breaking cit-tests (no tests-report -> build-docs fails). Pin to the pre-block v4.2.2, matching the rest of the org. Refs #349
- update to avoid extra generated branches by compiler. - follow up see eclipse-score/cicd-workflows#139
* chore(deps): update dependency score_platform to v0.6.3 * add version 1 to comp_arc_sta and comp_arc_dyn elements in kvs arch --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
FScholPer
marked this pull request as ready for review
August 3, 2026 11:39
PandaeDo
reviewed
Aug 4, 2026
PandaeDo
left a comment
PandaeDo
left a comment
Contributor
There was a problem hiding this comment.
@FScholPer I appreciate the work so far. But I also think that we have to rework the report. For me it's unreadable. It would be great to have a overview and not so many details. The details should be available. But as a consumer of the informations you have to scroll to the complete document to find what you need. As we discussed this is also a POC. Fine for me for the moment. But if possible to spend some more time on improvement as described I would think that this report will find much more acceptance.
aschemmel-tech
left a comment
aschemmel-tech
left a comment
Contributor
There was a problem hiding this comment.
see inline comments
| - For external component Assumptions of Use: coverage by platform safety manual or feature/components incl. test case, passed/failed and completeness verdict | ||
| - This is split in a list of QM requirements tested and a separate list of tests for ASIL rated requirements. | ||
| - List of component requirements (ASIL rated) linked to inspection checklist and verdict (derived from PR export) | ||
|
|
Contributor
There was a problem hiding this comment.
Here it would be good to have overview pie charts like:
- req/aou inspected/valid/invalid
- req without testcases/with testcases linked
- req tested passed/skipped/failed
| **7. Test logs** | ||
| - Test log per test case with status passed/failed/not_run | ||
|
|
||
| .. needtable:: Test Logs Per Testcase |
Contributor
There was a problem hiding this comment.
it is not clear for me what this table shows
links in the "need ID" do not work for me
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
9 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Frank Scholter Peres frank.scholter_peres@mercedes-benz.com, Mercedes-Benz Tech Innovation GmbH
Provider Information