Skip to content

epic: depends_on covfie when @main#849

Merged
wdconinc merged 5 commits intodevelopfrom
copilot/add-covfie-dependency-to-epic
Apr 23, 2026
Merged

epic: depends_on covfie when @main#849
wdconinc merged 5 commits intodevelopfrom
copilot/add-covfie-dependency-to-epic

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Feb 26, 2026

epic requires covfie starting from version 26.04, including main. covfie is available in the upstream spack builtin repository.

Needed for eic/epic#934.

Changes

  • epic package: adds depends_on("covfie", when="@26.04.0:"), mirroring the existing irt2 constraint for the same version range.

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Copilot AI changed the title [WIP] Add covfie as a dependency to epic package Add covfie dependency to epic @26.03.0: Feb 26, 2026
Comment thread spack_repo/eic/packages/covfie/package.py Outdated
@wdconinc wdconinc marked this pull request as ready for review February 26, 2026 23:29
Copilot AI review requested due to automatic review settings February 26, 2026 23:29
@wdconinc wdconinc enabled auto-merge (squash) February 26, 2026 23:29
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

This PR adds the covfie dependency to the epic Spack package for versions 26.03.0 and later, as epic now requires this library starting from that version.

Changes:

  • Added covfie as a dependency for epic@26.03.0: to match the requirement introduced in that version

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

@wdconinc wdconinc changed the title Add covfie dependency to epic @26.03.0: epic: depends_on covfie when @26.03.0: Feb 27, 2026
@wdconinc wdconinc requested a review from veprbl February 28, 2026 16:12
@wdconinc wdconinc changed the title epic: depends_on covfie when @26.03.0: epic: depends_on covfie when @26.04.0: Mar 7, 2026
@wdconinc wdconinc requested a review from rahmans1 March 7, 2026 17:54
@wdconinc wdconinc changed the title epic: depends_on covfie when @26.04.0: epic: depends_on covfie when @main Apr 2, 2026
Copilot AI and others added 4 commits April 2, 2026 12:31
@wdconinc wdconinc force-pushed the copilot/add-covfie-dependency-to-epic branch from 937bf6e to ab5f3bd Compare April 2, 2026 17:31
@wdconinc
Copy link
Copy Markdown
Contributor

wdconinc commented Apr 2, 2026

RFR Rescoped to only apply to main (and we'll change when there's a release which actually needs covfie).

@wdconinc wdconinc requested a review from a team April 23, 2026 13:26
@wdconinc
Copy link
Copy Markdown
Contributor

@eic/epic-review-team This PR has been languishing for nearly 2 months now and holds up testing of eic/epic#934 which is needed for both celeritas and adept development.

Copy link
Copy Markdown

@ruse-traveler ruse-traveler left a comment

Choose a reason for hiding this comment

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

LGTM!

@wdconinc wdconinc merged commit 0efb894 into develop Apr 23, 2026
5 checks passed
@wdconinc wdconinc deleted the copilot/add-covfie-dependency-to-epic branch April 23, 2026 15:14
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.

4 participants