Set ci config for cfg mngmt#5
Conversation
License Check Results🚀 The license check job ran with the Bazel command: bazel run //:license-checkStatus: Click to expand output |
|
The created documentation from the pull request is available at: docu-html |
2a4acf3 to
4bdaaad
Compare
- Add 4 CI workflows: tests.yml, static-analysis.yml, bzlmod-lock-check.yml, score-pr-checks.yml - Update .bazelrc: score_communication tracing stub flag, score_variant=True, host_gcc config using @gcc_toolchain - Align MODULE.bazel deps with OSS mirror (rules_python 1.8.3, platforms 1.0.0, rules_rust 0.68.1-score, rules_cc 0.2.16, aspect_rules_lint 2.0.0, score_logging 0.1.0) - Pin score_baselibs to commit 052c2f271be4 (avoids non-dev toolchains_llvm conflict), add rules_boost archive_override - Add score_docs_as_code, score_crates single_version_overrides - Add score_lifecycle_health dep - Remove MODULE.bazel.lock from .gitignore, commit fresh lockfile - Fix copyright header in tests/rust/test_main.rs
4bdaaad to
86f5e96
Compare
michaelsaborov
left a comment
There was a problem hiding this comment.
According to the original task workflows should support
- test (unti testing on host?). No such config found, which is ok for now as existing unit tests are not initially verified on score. Unit testing should be clearly addressed in a follow-up task
- static analysis. Same here. clang-tidy can´t be configured currently, is blocked.
- bzlmod lock check. Same here. A dedicated upstream workflow for this in eclipse-score/cicd-workflows (bzlmod-lock-check.yml) exists is not referenced here.
Is my understanding correct that for PRs, the effective checks are: formatting (buildifier?), copyright, commit-message lint, docs build, and license check?
|
|
||
| jobs: | ||
| bazel-module-name-check: | ||
| runs-on: ubuntu-latest |
There was a problem hiding this comment.
it´s probably safer to run on ubuntu-24.04. Same for other occurrences.
| # /components/*/ @eclipse-score/<module_name>/automotive-score-committers | ||
| # As we cannot use groups yet, we list all members explicitly. | ||
| # Default owners for everything in the repository | ||
| * @antonkri @luizcordeiro @michaelsaborov |
There was a problem hiding this comment.
| * @antonkri @luizcordeiro @michaelsaborov | |
| * @antonkri @castler @LittleHuba @ramceb @luizcordeiro @michaelsaborov |
- CODEOWNERS: apply suggested maintainers list - workflows: replace ubuntu-latest with ubuntu-24.04 in all jobs
|
Your understanding is correct. Here's the current status of each workflow: tests.yml (unit testing on host) static-analysis.yml (clang-tidy) bzlmod-lock-check.yml Effective PR checks currently: |
SWP-249886