Skip to content

add CI workflow - #28

Merged
jens-maus merged 1 commit into
mainfrom
pr/add-ci-workflow
Sep 7, 2026
Merged

add CI workflow#28
jens-maus merged 1 commit into
mainfrom
pr/add-ci-workflow

Conversation

@jens-maus

@jens-maus jens-maus commented Sep 7, 2026

Copy link
Copy Markdown
Member

This adds an initial CI workflow to check build upon changes.

Summary by CodeRabbit

  • Chores
    • Added automated build validation for four supported Linux platforms.
    • Builds now run consistently across native and cross-compiled environments.
    • Automated checks verify that required binaries, libraries, web assets, and device definitions are present and correctly formatted.
    • Unsupported target platforms now produce a clear build failure instead of an ambiguous result.
    • Build jobs can run automatically for updates, pull requests, or manual verification.

@coderabbitai

coderabbitai Bot commented Sep 7, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Team

Run ID: a9ded6b7-46b2-4343-b4fc-5afb5f8eaa59

📥 Commits

Reviewing files that changed from the base of the PR and between ff6939f and 5539190.

📒 Files selected for processing (1)
  • .github/workflows/ci.yml

Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review.


📝 Walkthrough

Walkthrough

Changes

Cross-platform CI build

Layer / File(s) Summary
Workflow triggers and platform matrix
.github/workflows/ci.yml
The workflow runs on pushes to main, pull requests, and manual dispatch. It defines four target platforms, read-only permissions, concurrency cancellation, checkout, and conditional QEMU setup.
Containerized target build
.github/workflows/ci.yml
Each target builds in a Debian Bookworm container. The workflow installs dependencies, prepares cross-compiler links, and runs the selected CMake preset.
Staged artifact verification
.github/workflows/ci.yml
The workflow checks required artifacts, binary formats, webui files, and device-type XML counts when assets are enabled.

Estimated code review effort: 4 (Complex) | ~45 minutes

Merge Risk: ⚪ Minimal · up to 55391

This change adds a read-only, containerized CI workflow that builds and verifies artifacts for four target platforms. No concrete merge-blocking risk remains.

Sequence Diagram(s)

sequenceDiagram
  participant GitHubActions
  participant QEMU
  participant Docker
  participant CMake
  participant Verification
  GitHubActions->>QEMU: Set up emulation for non-native targets
  GitHubActions->>Docker: Run Debian Bookworm build container
  Docker->>CMake: Configure and build target preset
  CMake-->>Docker: Create staged rootfs artifacts
  Docker->>Verification: Check artifacts and binary formats
  Verification-->>GitHubActions: Pass or fail the build
Loading
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately summarizes the primary change: adding a GitHub Actions CI workflow. It is concise and specific enough for the changeset.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch pr/add-ci-workflow

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@jens-maus
jens-maus merged commit 1b58142 into main Sep 7, 2026
5 checks passed
@jens-maus
jens-maus deleted the pr/add-ci-workflow branch September 7, 2026 21:25
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.

1 participant