Skip to content

Conversation

@jserv
Copy link
Contributor

@jserv jserv commented Nov 20, 2025

This displays git commit hash (short) with clickable link to GitHub commit in pull request test result comments for easier traceability.


Summary by cubic

Adds a clickable commit link in CI test result comments for quick traceability. The workflow records the commit SHA, short hash, and repo and includes them in the aggregated report.

  • New Features

    • Store commit_sha, commit_short, and github_repo in test results and the TOML summary.
    • Build a GitHub commit URL in format_comment and show “Commit: short” in PR comments.
    • Workflow sets COMMIT_SHA, COMMIT_SHORT, and GITHUB_REPO and passes them to collect-data; LLVM path writes them too.
  • Migration

    • collect-data now requires: <commit_sha> <commit_short> <github_repo> <test_output> [functional_output]. Update any callers.

Written for commit 209c241. Summary will update automatically on new commits.

This displays git commit hash (short) with clickable link to GitHub
commit in pull request test result comments for easier traceability.
@github-actions
Copy link

Linmo CI Test Results

Overall Status: ✅ passed
Timestamp: 2025-11-20T05:27:23+00:00
Commit: a8efaf5

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

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

Report generated from test-summary.toml

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

@jserv jserv merged commit d75c1da into main Nov 20, 2025
5 checks passed
@jserv jserv deleted the ci-revision branch November 20, 2025 05:33
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