Skip to content

Add CVMFS eic-shell CI workflow for npsim build, simulation/capybara regression checks, and capybara Pages publishing#46

Merged
wdconinc merged 8 commits into
mainfrom
copilot/setup-ci-build-workflow
May 17, 2026
Merged

Add CVMFS eic-shell CI workflow for npsim build, simulation/capybara regression checks, and capybara Pages publishing#46
wdconinc merged 8 commits into
mainfrom
copilot/setup-ci-build-workflow

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented May 17, 2026

  • Review new PR comment and inspect affected workflow sections
  • Update environment variable prepending to be safe with empty vars using ${VAR:+:$VAR}
  • Remove Linux DD4HEP_LIBRARY_PATH export usage per feedback
  • Validate workflow YAML and run targeted checks
  • Run CodeQL check
  • Reply to the new PR comment with commit hash

Copilot AI linked an issue May 17, 2026 that may be closed by this pull request
Agent-Logs-Url: https://github.com/eic/npsim/sessions/cbe083a0-60ae-4514-970c-83cb02338b16

Co-authored-by: wdconinc <4656391+wdconinc@users.noreply.github.com>
Copilot AI changed the title [WIP] Setup CI build workflow for gcc and clang in cvmfs Add CVMFS eic-shell CI workflow for npsim build and simulation/capybara regression checks May 17, 2026
Copilot AI requested a review from wdconinc May 17, 2026 14:46
Comment thread .github/workflows/linux-eic-shell.yml Outdated
Copy link
Copy Markdown
Contributor

@wdconinc wdconinc left a comment

Choose a reason for hiding this comment

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

@copilot This is still missing the capybara report publishing to GitHub pages and pull request comment posting that is in eic/epic.

Copy link
Copy Markdown
Contributor Author

Copilot AI commented May 17, 2026

@copilot This is still missing the capybara report publishing to GitHub pages and pull request comment posting that is in eic/epic.

Implemented in 9d1d35f: I added capybara report publishing to GitHub Pages plus PR comment posting (find/create-or-update comment) following the eic/epic pattern, adapted for npsim URLs.

Copilot AI changed the title Add CVMFS eic-shell CI workflow for npsim build and simulation/capybara regression checks Add CVMFS eic-shell CI workflow for npsim build, simulation/capybara regression checks, and capybara Pages publishing May 17, 2026
Copilot AI requested a review from wdconinc May 17, 2026 15:06
Comment thread .github/workflows/linux-eic-shell.yml Outdated
Copilot AI requested a review from wdconinc May 17, 2026 15:30
@wdconinc wdconinc marked this pull request as ready for review May 17, 2026 15:44
Copilot AI review requested due to automatic review settings May 17, 2026 15:44
Copy link
Copy Markdown
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

Adds a new GitHub Actions CI workflow that builds npsim inside the CVMFS-hosted eic-shell environment, runs small simulation/regression checks (including Capybara comparisons against previous artifacts), and publishes aggregated Capybara reports to GitHub Pages.

Changes:

  • Introduces an eic-shell-based build job (g++/clang++) and uploads an install artifact for downstream jobs.
  • Adds two simulation/regression jobs (gun + DIS) that produce ROOT outputs and generate Capybara reports against prior artifacts when available.
  • Aggregates Capybara artifacts and publishes per-PR and main summaries to GitHub Pages, with an auto-updated PR comment.

💡 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
Comment thread .github/workflows/linux-eic-shell.yml
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 Outdated
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 Outdated
Comment thread .github/workflows/linux-eic-shell.yml Outdated
wdconinc and others added 4 commits May 17, 2026 11:04
Co-authored-by: Wouter Deconinck <wdconinc@gmail.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@wdconinc wdconinc merged commit 0e56126 into main May 17, 2026
26 checks passed
@wdconinc wdconinc deleted the copilot/setup-ci-build-workflow branch May 17, 2026 16:43
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.

Note: this repo needs CI.

3 participants