Skip to content

ci: add GitHub Actions build workflow using eic-shell#26

Open
Copilot wants to merge 4 commits intomainfrom
copilot/setup-ci-build-workflow
Open

ci: add GitHub Actions build workflow using eic-shell#26
Copilot wants to merge 4 commits intomainfrom
copilot/setup-ci-build-workflow

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 7, 2026

No GitHub Actions CI existed; only a GitLab CI config was present. Adds a workflow mirroring the pattern used by EICrecon and eic-spack to validate builds against the EIC software environment.

Workflow (.github/workflows/linux-eic-shell.yml)

  • Triggers: push to main/tags, PRs, merge groups, daily schedule, workflow_dispatch (configurable org/platform/release)
  • Container: eicweb/eic_xl:nightly via eic/run-cvmfs-osg-eic-shell@main — provides all required deps (EDM4EIC, EDM4HEP, DD4hep, fmt)
  • Matrix:
    • g++ Release
    • clang++ Release + Debug
    • -Werror -Wall -Wextra for both; g++ adds -Wno-error=maybe-uninitialized for known GCC false positives
  • Caching: ccache with layered restore keys (head_refref_namebase_ref) for optimal PR cache reuse
  • Build: standard cmake -B build … && cmake --build … --target install

@wdconinc wdconinc marked this pull request as ready for review April 7, 2026 20:38
Copilot AI review requested due to automatic review settings April 7, 2026 20:38
Copy link
Copy Markdown

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

Note

Copilot was unable to run its full agentic suite in this review.

Adds a GitHub Actions CI workflow to build and install the project inside the EIC software environment (via eic-shell), bringing parity with the existing GitLab CI approach.

Changes:

  • Introduces .github/workflows/linux-eic-shell.yml with push/PR/merge-queue/scheduled/manual triggers.
  • Builds a compiler/build-type matrix (g++ Release; clang++ Release+Debug) with warnings-as-errors.
  • Adds ccache caching and runs the build inside eic/run-cvmfs-osg-eic-shell@main with CVMFS enabled.

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

Comment thread .github/workflows/linux-eic-shell.yml
Comment thread .github/workflows/linux-eic-shell.yml Outdated
Comment thread .github/workflows/linux-eic-shell.yml Outdated
Comment thread .github/workflows/linux-eic-shell.yml
Copilot AI and others added 4 commits May 10, 2026 11:11
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@wdconinc wdconinc force-pushed the copilot/setup-ci-build-workflow branch from 62c9acb to 2ac6891 Compare May 10, 2026 16:11
@wdconinc wdconinc requested a review from veprbl May 10, 2026 16:13
@wdconinc wdconinc enabled auto-merge (squash) May 10, 2026 16:15
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.

3 participants