Skip to content

feat: combine all build-test workflows into single build-test.md#1157

Open
Copilot wants to merge 2 commits intomainfrom
copilot/combine-build-test-workflows
Open

feat: combine all build-test workflows into single build-test.md#1157
Copilot wants to merge 2 commits intomainfrom
copilot/combine-build-test-workflows

Conversation

Copy link
Contributor

Copilot AI commented Mar 5, 2026

Consolidates the 8 separate per-language build-test agentic workflows (bun, cpp, deno, dotnet, go, java, node, rust) into a single build-test.md that runs all ecosystems as sequential tasks and posts one combined summary table per PR.

Combined workflow

  • 8 tasks executed sequentially by the same agent, one per ecosystem
  • Merged runtimes (node 20, go 1.22, rust stable, java 21, dotnet 8.0) and network domains
  • Single build-test label replaces 8 per-language labels
  • Per-ecosystem failure isolation — failures don't block remaining tasks
  • 45-minute timeout to accommodate all ecosystems

Output format

One PR comment with a combined table:

Ecosystem Project Build/Install Tests Status
Bun elysia ✅/❌ X/Y ✅/❌
... ... ... ... ...
Rust zoxide ✅/❌ X/Y ✅/❌

Files changed

  • Created build-test.md + compiled build-test.lock.yml
  • Deleted 16 files: build-test-{bun,cpp,deno,dotnet,go,java,node,rust}.{md,lock.yml}
  • Updated postprocess-smoke-workflows.ts — 8 paths → 1
  • Updated docs (ci-smoke.md, test-infra.md, SKILL.md) to reflect consolidation

Copilot AI and others added 2 commits March 5, 2026 19:01
Co-authored-by: Mossaka <5447827+Mossaka@users.noreply.github.com>
@Mossaka Mossaka requested a review from Copilot March 5, 2026 19:33
@Mossaka Mossaka marked this pull request as ready for review March 5, 2026 19:33
@Mossaka Mossaka closed this Mar 5, 2026
@Mossaka Mossaka reopened this Mar 5, 2026
@github-actions
Copy link
Contributor

github-actions bot commented Mar 5, 2026

✅ Coverage Check Passed

Overall Coverage

Metric Base PR Delta
Lines 82.28% 82.42% 📈 +0.14%
Statements 82.25% 82.39% 📈 +0.14%
Functions 82.60% 82.60% ➡️ +0.00%
Branches 74.43% 74.52% 📈 +0.09%
📁 Per-file Coverage Changes (1 files)
File Lines (Before → After) Statements (Before → After)
src/docker-manager.ts 83.0% → 83.6% (+0.55%) 82.4% → 82.9% (+0.53%)

Coverage comparison generated by scripts/ci/compare-coverage.ts

@github-actions
Copy link
Contributor

github-actions bot commented Mar 5, 2026

Smoke Test Results — Run #22733315837

✅ GitHub MCP — Last 2 merged PRs: #1156 "fix(security): secure SSL Bump key wiping via tmpfs", #1154 "fix(docker): disable IPv6 when ip6tables unavailable" (both by @Mossaka)
✅ Playwright — github.com title contains "GitHub"
✅ File Write — /tmp/gh-aw/agent/smoke-test-copilot-22733315837.txt created and verified
✅ Bash Tool — cat confirmed file contents

Overall: PASS | PR author: @Copilot | Assignees: @Mossaka @Copilot

📰 BREAKING: Report filed by Smoke Copilot for issue #1157

@github-actions
Copy link
Contributor

github-actions bot commented Mar 5, 2026

Smoke Test Results

✅ GitHub MCP: #1156 fix(security): secure SSL Bump key wiping via tmpfs, #1154 fix(docker): disable IPv6 when ip6tables unavailable
✅ Playwright: Page title contains "GitHub"
✅ File write: /tmp/gh-aw/agent/smoke-test-claude-22733315828.txt created
✅ Bash verify: File contents confirmed

Overall: PASS

💥 [THE END] — Illustrated by Smoke Claude for issue #1157

@github-actions
Copy link
Contributor

github-actions bot commented Mar 5, 2026

PR titles: fix(security): secure SSL Bump key wiping via tmpfs; fix(docker): disable IPv6 when ip6tables unavailable; feat: combine all build-test workflows into single build-test.md; fix(ci): resolve integration test suite failures on main
1 GitHub MCP: ✅ | 2 safeinputs-gh: ✅ | 3 Playwright: ✅ | 4 Tavily: ❌
5 File write: ✅ | 6 Bash cat: ✅ | 7 Discussion comment: ✅ | 8 Build: ✅
Overall status: FAIL

🔮 The oracle has spoken through Smoke Codex for issue #1157

@github-actions
Copy link
Contributor

github-actions bot commented Mar 5, 2026

🏗️ Build Test Suite Results

Ecosystem Project Build/Install Tests Status
Bun elysia 1/1 passed ✅ PASS
Bun hono 1/1 passed ✅ PASS
C++ fmt N/A ✅ PASS
C++ json N/A ✅ PASS
Deno oak N/A 1/1 passed ✅ PASS
Deno std N/A 1/1 passed ✅ PASS
.NET hello-world N/A ✅ PASS
.NET json-parse N/A ✅ PASS
Go color passed ✅ PASS
Go env passed ✅ PASS
Go uuid passed ✅ PASS
Java gson 1/1 passed ✅ PASS
Java caffeine 1/1 passed ✅ PASS
Node.js clsx passed ✅ PASS
Node.js execa passed ✅ PASS
Node.js p-limit passed ✅ PASS
Rust fd 1/1 passed ✅ PASS
Rust zoxide 1/1 passed ✅ PASS

Overall: 8/8 ecosystems passed — ✅ PASS

Generated by Build Test Suite for issue #1157 ·

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants