Skip to content

Add CI workflow for main branch tests#98

Merged
ViTeXFTW merged 1 commit intodevfrom
claude/add-status-badge-XlGSF
Feb 23, 2026
Merged

Add CI workflow for main branch tests#98
ViTeXFTW merged 1 commit intodevfrom
claude/add-status-badge-XlGSF

Conversation

@ViTeXFTW
Copy link
Copy Markdown
Owner

Summary

This PR introduces a dedicated GitHub Actions workflow for running tests on the main branch, replacing the previous PR-based test workflow reference.

Key Changes

  • Added .github/workflows/main-tests.yml: New workflow that runs on every push to main and on manual dispatch

    • Configured with a test build type environment variable
    • Tests across multiple platforms and compilers:
      • Linux: GCC and Clang
      • Windows: MSVC and Clang
    • Includes dependency installation for Linux (compiler, CMake, Ninja)
    • Runs CMake configuration, build, and CTest with failure output
    • Uses recursive submodule checkout for dependencies
  • Updated README.md: Changed the test status badge to reference the new main-tests.yml workflow instead of the previous pr-test.yml workflow

Implementation Details

  • The workflow uses a matrix strategy with fail-fast: false to ensure all platform/compiler combinations are tested even if one fails
  • Platform-specific steps handle differences between Linux and Windows (dependency installation and CMake configuration)
  • Compiler environment variables (CC/CXX) are only set for Linux jobs where they're needed

https://claude.ai/code/session_01GGUqtnAHdmrCDegnjHRFpY

Adds main-tests.yml which triggers on push to main, running the full
compiler matrix (gcc/clang on Linux, msvc/clang on Windows). This
replaces the pr-test.yml badge reference in README, which was stale
after release-branch merges since pr-test.yml only triggers on PRs.

https://claude.ai/code/session_01GGUqtnAHdmrCDegnjHRFpY
@ViTeXFTW
Copy link
Copy Markdown
Owner Author

@greptileai

@greptile-apps
Copy link
Copy Markdown

greptile-apps Bot commented Feb 23, 2026

Greptile Summary

This PR adds a dedicated CI workflow for testing the main branch, ensuring continuous validation of production code.

Key Changes:

  • Added main-tests.yml workflow that runs on pushes to main and manual dispatch
  • Tests run across 4 configurations: Linux (gcc/clang) + Windows (msvc/clang)
  • Updated README badge to show main branch test status instead of PR test status
  • Workflow structure mirrors the existing pr-test.yml test job for consistency

The implementation is clean and follows the existing patterns from pr-test.yml. All referenced CMake presets (gcc-test, clang-test, msvc-test) exist in the project's CMakePresets.json.

Confidence Score: 5/5

  • This PR is safe to merge with minimal risk
  • The workflow file is well-structured, follows established patterns from the existing pr-test.yml, and only adds new functionality without modifying existing code. The README change is a simple badge URL update.
  • No files require special attention

Important Files Changed

Filename Overview
.github/workflows/main-tests.yml New workflow for main branch testing - correctly configured with matrix strategy for Linux (gcc/clang) and Windows (msvc/clang), proper dependency installation, and test execution
README.md Updated test badge to reference new main-tests.yml workflow instead of pr-test.yml

Last reviewed commit: e8e209e

@ViTeXFTW ViTeXFTW marked this pull request as ready for review February 23, 2026 12:45
@ViTeXFTW ViTeXFTW merged commit 5faccfb into dev Feb 23, 2026
4 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.

2 participants