infra: add CI workflows #15
Merged
Conversation
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
arkjedrz
requested review from
antonkri,
arsibo and
pawelrutkaq
as code owners
|
The created documentation from the pull request is available at: docu-html |
pawelrutkaq
requested changes
Dec 30, 2025
MODULE.bazel
Outdated
| llvm.toolchain( | ||
| cxx_standard = {"": "c++17"}, | ||
| llvm_version = "19.1.0", | ||
| register_toolchains( |
Contributor
There was a problem hiding this comment.
do not register toolchain, use bazelrc.
arkjedrz
force-pushed
the
arkjedrz_align-repo-state
branch
from
5c26e3f to
5f7c0c2
Compare
arkjedrz
force-pushed
the
arkjedrz_align-repo-state
branch
2 times, most recently
from
f88cdbe to
326a14e
Compare
pawelrutkaq
requested changes
Dec 30, 2025
.github/workflows/tests.yml
Outdated
| - name: Run Tests via Bazel | ||
| run: | | ||
| echo "Running: bazel test //src/..." | ||
| bazel test //src/... |
Contributor
There was a problem hiding this comment.
dont we have tests in //score ?
arkjedrz
force-pushed
the
arkjedrz_align-repo-state
branch
from
326a14e to
ed4de84
Compare
pawelrutkaq
reviewed
Dec 30, 2025
PiotrKorkus
reviewed
Dec 30, 2025
PiotrKorkus
left a comment
PiotrKorkus
left a comment
There was a problem hiding this comment.
Also bazel build //... doesnt work locally as well
.github/workflows/build.yml
Outdated
| jobs: | ||
| build: | ||
| name: Build Bazel Code | ||
| runs-on: ${{ vars.REPO_RUNNER_LABELS && fromJSON(vars.REPO_RUNNER_LABELS) || 'ubuntu-latest' }} |
There was a problem hiding this comment.
we dont have self-hosted runners in score, can be just ubuntu-latest
.github/workflows/build.yml
Outdated
Comment on lines
35
to
39
| - name: Install protoc 26.1 | ||
| uses: arduino/setup-protoc@v3 | ||
| with: | ||
| version: 26.1 | ||
| repo-token: ${{ secrets.GITHUB_TOKEN }} |
There was a problem hiding this comment.
step not needed, it works without it
.github/workflows/build_qnx8.yml
Outdated
| contents: read | ||
| pull-requests: read | ||
| with: | ||
| bazel-target: "//src/..." |
| with: | ||
| # the bazel-target depends on your repo specific docs_targets configuration (e.g. "suffix") | ||
| bazel-target: "//:docs -- --github_user=${{ github.repository_owner }} --github_repo=${{ github.event.repository.name }}" | ||
| retention-days: 3 |
There was a problem hiding this comment.
we have 3 days here but docs-cleanup.yml runs daily
There was a problem hiding this comment.
double checked, those are 2 different things being cleaned. Not valid anymore
.github/workflows/gitlint.yml
Outdated
| jobs: | ||
| lint-commits: | ||
| name: check-commit-messages | ||
| runs-on: ${{ vars.REPO_RUNNER_LABELS && fromJSON(vars.REPO_RUNNER_LABELS) || 'ubuntu-latest' }} |
.github/workflows/tests.yml
Outdated
| run: | | ||
| # TODO: restore `//src/...` tests once provided. | ||
| # echo "Running: bazel test //src/..." | ||
| # bazel test //src/... |
There was a problem hiding this comment.
just for the future it can be merged to single bazel test command - bazel test //src/... //score/...
* Log backend for rust frontend This is logger impl that bridges rust log frontend with C++ mw_log implementation Signed-off-by: Pawel Rutka <pawel.rutka.ext@qorix.ai> * Adress review findings * Adapt to cpp/rust structure as in persistency * Align to baselibs --------- Signed-off-by: Pawel Rutka <pawel.rutka.ext@qorix.ai>
- Add CI workflows. - Align Bazel config and deps. - Fix build-preventing and other minor issues.
arkjedrz
force-pushed
the
arkjedrz_align-repo-state
branch
from
ed4de84 to
fcd11d0
Compare
PiotrKorkus
reviewed
Jan 5, 2026
Comment on lines
53
to
57
| git_override( | ||
| module_name = "score_toolchains_rust", | ||
| commit = "bcf8e5364f72cf136ec81960350a82e2b5c45449", | ||
| remote = "https://github.com/eclipse-score/toolchains_rust.git", | ||
| ) |
Contributor
Author
There was a problem hiding this comment.
Won't do due to eclipse-score/toolchains_rust#10
Contributor
There was a problem hiding this comment.
rust toolchain is going to change drastically soon. lets leave it as it is for week or two
MODULE.bazel
Outdated
| bazel_dep(name = "score_toolchains_gcc", version = "0.4", dev_dependency = True) | ||
| # S-CORE process rules | ||
| bazel_dep(name = "score_bazel_platforms", version = "0.0.3") | ||
| bazel_dep(name = "score_docs_as_code", version = "2.0.2") |
FScholPer
previously approved these changes
Jan 5, 2026
Fixes after review.
arkjedrz
force-pushed
the
arkjedrz_align-repo-state
branch
from
b307dfc to
1e6349c
Compare
PiotrKorkus
approved these changes
Jan 5, 2026
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 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.
Uh oh!
There was an error while loading. Please reload this page.