Skip to content

ci: split release workflow from PR checks - #35

Merged
striderZA merged 4 commits into
masterfrom
ci/release-workflow-simplification
Aug 2, 2026
Merged

ci: split release workflow from PR checks#35
striderZA merged 4 commits into
masterfrom
ci/release-workflow-simplification

Conversation

@striderZA

@striderZA striderZA commented Aug 2, 2026

Copy link
Copy Markdown
Owner

Summary

Split the old combined GitHub Actions workflow into a PR-only CI workflow and a tag-only release workflow.

Related issue

N/A

Type of change

  • Bug fix
  • New feature
  • Refactor
  • Documentation
  • Build / CI

Test plan

  • cmake --build build
  • ctest --test-dir build --output-on-failure -E "Benchmark|test_ui|ADC DDC output grid spans"
  • Manual verification steps (if applicable):
    • Verified .github/workflows/ci.yml is pull_request-only and keeps format, lint-commits, build, sanitizer, and coverage
    • Verified .github/workflows/release.yml is push.tags: v*-only, keeps only Linux GCC 14 (Debug) + Windows MinGW-w64 artifact builds, preserves the tag/CMake version check, and skips ctest
    • Verified .github/workflows/build.yml was removed
    • Verified CONTRIBUTING.md now describes the split workflow contract

Checklist

  • My code follows the existing code style (see .clang-format)
  • I ran clang-format -i on changed files (N/A: no C++ file changes)
  • I added or updated tests where appropriate (N/A: workflow/docs-only change; verified behavior via targeted checks)
  • Existing tests still pass

@striderZA
striderZA force-pushed the ci/release-workflow-simplification branch from 09c4dcf to e10d3b4 Compare August 2, 2026 19:10
@striderZA
striderZA merged commit 33b1856 into master Aug 2, 2026
8 checks passed
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.

1 participant