Skip to content

Conversation

@wdconinc
Copy link
Contributor

@wdconinc wdconinc commented Nov 28, 2025

Briefly, what does this PR introduce?

This PR upgrades edm4hep to v0.99.4 (diff).

Needs:

Copilot AI review requested due to automatic review settings November 28, 2025 19:30
Copy link
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 upgrades the edm4hep package from v0.99.1 to v0.99.4 by cherry-picking two commits from the upstream spack/spack-packages repository. The upgrade brings in deprecation removals and adds latest tags with updated dependencies for the edm4hep event data model framework.

Key Changes:

  • Cherry-picked two upstream spack commits for edm4hep updates
  • Updated edm4hep version requirement from @0.99.1 to @0.99.4

Reviewed changes

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

File Description
spack-packages.sh Added two cherry-pick commit hashes (9cf8ee9 and 78a6c5f) with file mappings and descriptions for edm4hep deprecation removals and version updates
spack-environment/packages.yaml Updated edm4hep version constraint from @0.99.1 to @0.99.4

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

@wdconinc wdconinc force-pushed the edm4hep-0.99.4 branch 2 times, most recently from e08a76c to a6fa239 Compare December 3, 2025 20:53
Copilot AI review requested due to automatic review settings December 3, 2025 20:53
Copy link
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.

Comments suppressed due to low confidence (1)

spack-packages.sh:48

  • The cherry-pick hash 438a7d95de1b81e15107edbf341b20824ec80635 (for "root: add v6.36.06") was removed from the list, but its corresponding description comment at line 105 ## 438a7d95de1b81e15107edbf341b20824ec80635: root: add v6.36.06 is still present in the file. For consistency, when removing a cherry-pick hash, its description comment should also be removed.
c75e10845431600b163c597545bd099e427c62f5
438a7d95de1b81e15107edbf341b20824ec80635

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

Copilot AI review requested due to automatic review settings December 6, 2025 18:41
Copy link
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.


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

@wdconinc wdconinc requested a review from veprbl December 7, 2025 15:42
@wdconinc wdconinc enabled auto-merge (squash) December 7, 2025 16:16
Copilot AI review requested due to automatic review settings December 8, 2025 15:51
Copy link
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.


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

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