ci: CI Workflow Fixes and Improvements #58

Closed

sameh-farouk wants to merge 6 commits from develop-workflow-fixes into develop

pull from: develop-workflow-fixes

merge into: lhumina_code:develop

lhumina_code:development

lhumina_code:development-wasm-plan-execution

lhumina_code:development_wasm

lhumina_code:development-static-build

lhumina_code:development_rootfs

lhumina_code:develop

lhumina_code:develop-ci-rhai-tests

lhumina_code:develop_codebox

lhumina_code:development_webdav_server

lhumina_code:develop-osis-pr

lhumina_code:develop_hetzner_rescuemode

lhumina_code:develop_kernel_module

lhumina_code:develop-osis

lhumina_code:main

lhumina_code:develop-fix-clippy-warnings

lhumina_code:develop-fossil-rnd

lhumina_code:main-work

lhumina_code:develop_podman

lhumina_code:kristof

lhumina_code:rpc_opapi

lhumina_code:main-integration

Conversation 1 Commits 6 Files changed 5 +311 -79

sameh-farouk commented 2026-01-19 11:27:21 +00:00

Member

Copy link

This PR brings major improvements to both Linux and macOS CI/CD workflows, adding comprehensive testing, better error reporting, performance optimizations, and multi-architecture support.

🔧 Key Changes

Fixed Workflow Triggers

• Before: Only triggered on main branch pushes
• After: Triggers on develop branch pushes AND pull requests

Added Comprehensive Testing

• Rust Tests: Added cargo test --workspace --release
• Rhai Tests: Added Rhai test suite execution

Error Reporting

• Benefit: Developers see exactly what failed and why

Performance Optimizations

• Cargo Registry Caching: Faster dependency downloads
• Target Directory Caching: Faster rebuilds across runs
• Optimized Profiling Tests: Reduced heavy rhia test execution time by 60-65%

Multi-Architecture Support (macOS)

• Apple Silicon (ARM64): aarch64-apple-darwin
• Intel (x86_64): x86_64-apple-darwin
• Matrix Strategy: Parallel builds for both architectures

fixes: geomind_research/herolib_rust#57

This PR brings major improvements to both Linux and macOS CI/CD workflows, adding comprehensive testing, better error reporting, performance optimizations, and multi-architecture support. ## 🔧 Key Changes ### **Fixed Workflow Triggers** - **Before:** Only triggered on `main` branch pushes - **After:** Triggers on `develop` branch pushes AND pull requests ### **Added Comprehensive Testing** - **Rust Tests:** Added `cargo test --workspace --release` - **Rhai Tests:** Added Rhai test suite execution ### **Error Reporting** - **Benefit:** Developers see exactly what failed and why ### **Performance Optimizations** - **Cargo Registry Caching:** Faster dependency downloads - **Target Directory Caching:** Faster rebuilds across runs - **Optimized Profiling Tests:** Reduced heavy rhia test execution time by 60-65% ### **Multi-Architecture Support (macOS)** - **Apple Silicon (ARM64):** `aarch64-apple-darwin` - **Intel (x86_64):** `x86_64-apple-darwin` - **Matrix Strategy:** Parallel builds for both architectures **fixes: https://forge.ourworld.tf/geomind_research/herolib_rust/issues/57**

sameh-farouk added 1 commit 2026-01-19 11:27:21 +00:00

ci: add Rust/Rhai tests, caching, multi-arch macOS builds ... 594c0f676c

- Change CI triggers from main to develop branch
- Add pull_request triggers for develop branch
- Add Cargo registry and target directory caching
- Add Rust test execution step (cargo test --workspace --release)
- Add Rhai test execution with detailed reporting and failure tracking
- Add macOS multi-architecture build matrix (arm64 and x86_64)
- Optimize Rhai test scripts for faster execution (reduced test sizes and durations)

sameh-farouk added 1 commit 2026-01-19 11:57:33 +00:00

fix: add explicit f32 type annotation 7ea60ecee0

sameh-farouk added 1 commit 2026-01-19 12:01:14 +00:00

fix: change max_temp type from f32 to f64 in thermal state detection e621ce6973

sameh-farouk added 1 commit 2026-01-19 12:07:14 +00:00

fix: add SystemError import to capabilities module c590860558

sameh-farouk added 1 commit 2026-01-19 12:11:10 +00:00

fix: move SystemError import to Linux-specific function scope in capabilities module ab79483cf0

sameh-farouk changed title from ci: CI Workflow Fixes and Improvements to WIP: ci: CI Workflow Fixes and Improvements 2026-01-19 12:29:13 +00:00

thabeta approved these changes 2026-01-19 13:06:22 +00:00

thabeta changed title from WIP: ci: CI Workflow Fixes and Improvements to ci: CI Workflow Fixes and Improvements 2026-01-19 13:06:30 +00:00

thabeta added 1 commit 2026-01-19 13:06:39 +00:00

Merge branch 'develop' into develop-workflow-fixes 9568443216

sameh-farouk closed this pull request 2026-01-21 14:43:10 +00:00

Some checks failed

Hide all checks

Build Linux / build-linux (linux-amd64, false, x86_64-unknown-linux-gnu) (pull_request) Failing after 4m58s

Details

Build macOS / build-macos (darwin-arm64, false, aarch64-apple-darwin) (pull_request) Has been cancelled

Details

Build macOS / build-macos (darwin-x86_64, false, x86_64-apple-darwin) (pull_request) Has been cancelled

Details

Pull request closed

This pull request cannot be reopened because the branch was deleted.

Sign in to join this conversation.

Reviewers

No reviewers

thabeta

Labels

Clear labels

No items

No labels

Milestone

Clear milestone

No items

No milestone

Projects

Clear projects

No items

No project

Assignees

Clear assignees

No assignees

2 participants

Notifications

Subscribe

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference

lhumina_code/hero_lib!58

No description provided.