Skip to content

feat: irt2 2.1.2#162

Merged
veprbl merged 2 commits intomasterfrom
pr/irt2_2_1_2
Feb 13, 2026
Merged

feat: irt2 2.1.2#162
veprbl merged 2 commits intomasterfrom
pr/irt2_2_1_2

Conversation

@veprbl
Copy link
Copy Markdown
Member

@veprbl veprbl commented Feb 12, 2026

Implements changes needed for WITH_ROOT compilation, fixes for compilation with upcoming EICrecon change.

Copilot AI review requested due to automatic review settings February 12, 2026 19:47
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

Updates the Spack environment to pick up irt2 v2.1.2 and advances the pinned eic/eic-spack reference to a newer commit, aligning container builds with changes needed for WITH_ROOT compilation and upcoming EICrecon adjustments.

Changes:

  • Bump irt2 requirement from @2.1.1 to @2.1.2 in the Spack environment.
  • Update EICSPACK_VERSION to a newer eic/eic-spack commit hash.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
spack-environment/packages.yaml Pins irt2 to @2.1.2 for the environment concretization.
eic-spack.sh Updates the pinned eic/eic-spack commit used to source Spack recipes/config.

Copilot AI review requested due to automatic review settings February 13, 2026 13:14
@veprbl veprbl enabled auto-merge (squash) February 13, 2026 13:14
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

Copilot reviewed 2 out of 2 changed files in this pull request and generated no new comments.

@veprbl veprbl merged commit ee4ccb1 into master Feb 13, 2026
35 checks passed
@veprbl veprbl deleted the pr/irt2_2_1_2 branch February 14, 2026 00:16
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