Skip to content

ci: skip CI when only release.yml changes#1261

Merged
Mossaka merged 1 commit intomainfrom
ci/skip-tests-on-release-yml-changes
Mar 12, 2026
Merged

ci: skip CI when only release.yml changes#1261
Mossaka merged 1 commit intomainfrom
ci/skip-tests-on-release-yml-changes

Conversation

@Mossaka
Copy link
Collaborator

@Mossaka Mossaka commented Mar 12, 2026

Summary

  • Adds paths-ignore: ['.github/workflows/release.yml'] to 10 CI workflow triggers so that PRs modifying only release.yml don't trigger slow integration tests and other unnecessary checks
  • Affected workflows: build, codeql, dependency-audit, lint, test-action, test-chroot, test-coverage, test-examples, test-integration-suite, test-integration
  • workflow_dispatch and push to main triggers are unchanged, so scheduled/manual runs still work

Test plan

  • Verify this PR itself triggers the expected subset of workflows (all workflows should still run since this PR touches workflow files beyond just release.yml)
  • Create a test PR that only modifies release.yml and confirm no test/build workflows are triggered

🤖 Generated with Claude Code

Add paths-ignore for .github/workflows/release.yml to all test and
build workflows so that PRs modifying only the release workflow don't
trigger slow integration tests and other unnecessary CI checks.

Affected workflows: build, codeql, dependency-audit, lint, test-action,
test-chroot, test-coverage, test-examples, test-integration-suite,
test-integration.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings March 12, 2026 17:35
@github-actions
Copy link
Contributor

✅ Coverage Check Passed

Overall Coverage

Metric Base PR Delta
Lines 82.56% 82.70% 📈 +0.14%
Statements 82.57% 82.70% 📈 +0.13%
Functions 83.01% 83.01% ➡️ +0.00%
Branches 75.12% 75.20% 📈 +0.08%
📁 Per-file Coverage Changes (1 files)
File Lines (Before → After) Statements (Before → After)
src/docker-manager.ts 84.1% → 84.6% (+0.54%) 83.4% → 83.9% (+0.52%)

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

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.

Pull request overview

This PR updates GitHub Actions workflow triggers to avoid running the main CI suite on pull requests that only modify the release workflow definition (.github/workflows/release.yml), reducing unnecessary CI load while leaving push to main and workflow_dispatch behavior intact.

Changes:

  • Add paths-ignore: ['.github/workflows/release.yml'] to pull_request triggers across multiple CI workflows.
  • Extend existing paths-ignore lists (where present) to include the release workflow file.

Reviewed changes

Copilot reviewed 10 out of 10 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
.github/workflows/build.yml Ignore PRs that only change .github/workflows/release.yml for the build verification workflow.
.github/workflows/codeql.yml Ignore PRs that only change .github/workflows/release.yml for CodeQL on PRs.
.github/workflows/dependency-audit.yml Add release workflow file to PR paths-ignore for dependency auditing.
.github/workflows/lint.yml Add release workflow file to PR paths-ignore for linting.
.github/workflows/test-action.yml Add release workflow file to PR paths-ignore for action tests.
.github/workflows/test-chroot.yml Ignore PRs that only change .github/workflows/release.yml for chroot integration tests.
.github/workflows/test-coverage.yml Add release workflow file to PR paths-ignore for coverage reporting.
.github/workflows/test-examples.yml Add release workflow file to PR paths-ignore for examples testing.
.github/workflows/test-integration-suite.yml Ignore PRs that only change .github/workflows/release.yml for the integration suite.
.github/workflows/test-integration.yml Ignore PRs that only change .github/workflows/release.yml for integration tests.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines 6 to 10
pull_request:
branches: [main]
paths-ignore:
- '.github/workflows/release.yml'
workflow_dispatch:
Copy link

Copilot AI Mar 12, 2026

Choose a reason for hiding this comment

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

The PR title/description implies that PRs changing only .github/workflows/release.yml will skip CI broadly, but there are other workflows still triggered on pull_request without a matching paths-ignore (e.g. .github/workflows/build-test.lock.yml, smoke-*.lock.yml, container-scan.yml, pr-title.yml). As-is, a release-workflow-only PR will still run those checks; either extend the same paths-ignore to the remaining PR-triggered workflows you want to suppress, or adjust the PR title/description to reflect that only this subset is being skipped.

Copilot uses AI. Check for mistakes.
@github-actions
Copy link
Contributor

Smoke Test Results — PASS

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

@github-actions
Copy link
Contributor

PR titles:

  • ci: skip CI when only release.yml changes
  • fix: make release workflow compatible with branch protection
    Tests:
  • GitHub MCP merged PRs: ✅
  • safeinputs-gh pr list: ✅
  • Playwright title check: ✅
  • Tavily search: ❌
  • File write: ✅
  • Bash cat: ✅
  • Discussion comment: ✅
  • Build: ✅
    Overall: FAIL

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

@github-actions github-actions bot mentioned this pull request Mar 12, 2026
@github-actions
Copy link
Contributor

Smoke test results for run 23015440360:

✅ GitHub MCP — Last 2 merged PRs: #1258 fix: push only tag in release workflow to avoid branch protection violation, #1254 fix: resolve test coverage regression and locked PR comment failure
✅ Playwright — github.com title contains "GitHub"
✅ File Write — /tmp/gh-aw/agent/smoke-test-copilot-23015440360.txt created
✅ Bash Tool — file verified via cat

Overall: PASS | Author: @Mossaka

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

@github-actions
Copy link
Contributor

🏗️ 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 N/A ❌ FAIL
Java caffeine N/A ❌ FAIL
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: 7/8 ecosystems passed — ❌ FAIL


❌ Failure Details

Java (gson, caffeine) — Maven compile failed: network unreachable when downloading Maven plugins from repo.maven.apache.org.

[ERROR] Could not transfer artifact org.apache.maven.plugins:maven-resources-plugin:pom:3.3.1
        from/to central (https://repo.maven.apache.org/maven2): Network is unreachable

Maven requires downloading plugin POMs on first use and cannot reach the Maven Central repository from this environment. The JAVA_TOOL_OPTIONS env var was set with a squid-proxy proxy host (not reachable outside the AWF container), and without that proxy the network is directly unreachable.

Generated by Build Test Suite for issue #1261 ·

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.

2 participants