Skip to content

Update merklecpp from 1.1.1 to 1.1.2#7923

Closed
achamayou wants to merge 5 commits into
mainfrom
update_merklecpp_1.1.1_to_1.1.2
Closed

Update merklecpp from 1.1.1 to 1.1.2#7923
achamayou wants to merge 5 commits into
mainfrom
update_merklecpp_1.1.1_to_1.1.2

Conversation

@achamayou

Copy link
Copy Markdown
Member

No description provided.

Copilot AI review requested due to automatic review settings June 5, 2026 12:20
@achamayou achamayou requested a review from a team as a code owner June 5, 2026 12:20

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 bumps the vendored merklecpp dependency from v1.1.1 to v1.1.2, updating the tracked commit/tag metadata and refreshing the in-repo header to match the new upstream version.

Changes:

  • Added a CHANGELOG entry describing the merklecpp dependency update.
  • Updated cgmanifest.json to point at merklecpp v1.1.2 (new tag + commit hash).
  • Updated 3rdparty/internal/merklecpp/merklecpp.h with upstream changes (API/implementation tweaks and additional OpenSSL hash variants).

Custom instructions used:

  • .github/copilot-instructions.md
  • .github/instructions/reviewing.instructions.md

Reviewed changes

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

File Description
CHANGELOG.md Documents the dependency bump (needs PR reference suffix).
cgmanifest.json Updates the tracked merklecpp tag/commit to v1.1.2 for compliance.
3rdparty/internal/merklecpp/merklecpp.h Syncs vendored merklecpp header with upstream v1.1.2 changes.

Comment thread CHANGELOG.md Outdated
Comment thread 3rdparty/internal/merklecpp/merklecpp.h
Comment thread 3rdparty/internal/merklecpp/merklecpp.h
Comment thread CHANGELOG.md Outdated
@achamayou achamayou added this to the 7.0.5 milestone Jun 5, 2026
@achamayou achamayou closed this Jun 8, 2026
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