Skip to content

Conversation

@jserv
Copy link
Contributor

@jserv jserv commented Jan 28, 2026

ci.yml:

  • Use uppercase display names (GNU/LLVM) with lowercase data values
  • Add random delimiter for GITHUB_OUTPUT to prevent heredoc collision
  • Skip QEMU install for LLVM (build-only validation)
  • Simplify toolchain setup with shared lowercase name output
  • Remove unnecessary permissions and redundant steps
  • Improve cache key format for better organization

codeql.yml:

  • Add concurrency control and timeout
  • Add proper permissions (contents: read, security-events: write)
  • Share toolchain cache with main CI workflow

Summary by cubic

Streamlined CI and CodeQL workflows for faster, safer runs. Standardized toolchain naming, improved caching, tightened permissions, and skipped QEMU for LLVM build-only checks.

  • Refactors

    • Standardized toolchain names: display as GNU/LLVM, data/env as lowercase via shared output.
    • Improved caching: consistent riscv32- keys, actions/cache@v5, shared cache with CodeQL.
    • Adjusted execution: LLVM does build-only without QEMU; GNU installs QEMU and runs full app + functional tests.
    • Tightened permissions and simplified steps: removed unnecessary writes, cleaner summary flow, updated artifacts (download@v7/upload@v6) with 7/30-day retention.
    • CodeQL: added concurrency cancel, weekly schedule, 30-min timeout, proper permissions, and reused toolchain cache.
  • Bug Fixes

    • Used random heredoc delimiters for GITHUB_OUTPUT to prevent multiline output collisions.

Written for commit 7672813. Summary will update on new commits.

ci.yml:
- Use uppercase display names (GNU/LLVM) with lowercase data values
- Add random delimiter for GITHUB_OUTPUT to prevent heredoc collision
- Skip QEMU install for LLVM (build-only validation)
- Simplify toolchain setup with shared lowercase name output
- Remove unnecessary permissions and redundant steps
- Improve cache key format for better organization

codeql.yml:
- Add concurrency control and timeout
- Add proper permissions (contents: read, security-events: write)
- Share toolchain cache with main CI workflow
Copy link

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 2 files

@github-actions
Copy link

Linmo CI Test Results

Overall Status: ✅ passed
Timestamp: 2026-01-28T06:42:00+00:00
Commit: 6effab2

Toolchain Results

Toolchain Build Crash Test Functional
GNU ✅ passed ✅ passed ✅ passed
LLVM ✅ passed ⏭️ skipped ⏭️ skipped

Application Tests

App GNU LLVM
cond ✅ passed ⏭️ skipped
coop ✅ passed ⏭️ skipped
cpubench ✅ passed ⏭️ skipped
echo ✅ passed ⏭️ skipped
hello ✅ passed ⏭️ skipped
mqueues ✅ passed ⏭️ skipped
mutex ✅ passed ⏭️ skipped
pipes ✅ passed ⏭️ skipped
pipes_small ✅ passed ⏭️ skipped
pipes_struct ✅ passed ⏭️ skipped
prodcons ✅ passed ⏭️ skipped
progress ✅ passed ⏭️ skipped
rtsched ✅ passed ⏭️ skipped
semaphore ✅ passed ⏭️ skipped
suspend ✅ passed ⏭️ skipped
test64 ✅ passed ⏭️ skipped
test_libc ✅ passed ⏭️ skipped
timer ✅ passed ⏭️ skipped
timer_kill ✅ passed ⏭️ skipped
umode ⚠️ ⏭️ skipped

Functional Test Details

Test GNU LLVM
mutex:data_consistency ✅ passed ⏭️ skipped
mutex:fairness ✅ passed ⏭️ skipped
mutex:mutual_exclusion ✅ passed ⏭️ skipped
mutex:overall ✅ passed ⏭️ skipped
semaphore:overall ✅ passed ⏭️ skipped
umode:[exception]_illegal_instruction ✅ passed ⚠️
umode:pass:_sys_tid()_returned ✅ passed ⚠️
umode:pass:_sys_uptime()_returned ✅ passed ⚠️

Report generated from test-summary.toml

@jserv jserv merged commit 7c63005 into main Jan 28, 2026
6 checks passed
@jserv jserv deleted the ci-refine branch January 28, 2026 06:42
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.

2 participants