Skip to content

Make MetadataHash common - #16585

Open
erak wants to merge 2 commits into
test-target-contractfrom
common-metadata-hash
Open

Make MetadataHash common#16585
erak wants to merge 2 commits into
test-target-contractfrom
common-metadata-hash

Conversation

@erak

@erak erak commented Apr 9, 2026

Copy link
Copy Markdown
Collaborator

Part of #16394. Depends on #16897.

Description

The compiler supports different hashing methods for metadata. These methods are currently defined as part of the CompilerStack interface. Given that we want to use this definition without depending on the CompilerStack, it was extracted and moved to a common header.

AI Disclosure

  • assisted with writing unit tests

@github-actions

github-actions Bot commented Apr 9, 2026

Copy link
Copy Markdown

Thank you for your contribution to the Solidity compiler! A team member will follow up shortly.

If you haven't read our contributing guidelines and our review checklist before, please do it now, this makes the reviewing process and accepting your contribution smoother.

If you have any questions or need our help, feel free to post them in the PR or talk to us directly on the #solidity-dev channel on Matrix.

@erak
erak force-pushed the common-metadata-hash branch from 7d0ce74 to f37c119 Compare August 7, 2026 10:44
@erak
erak changed the base branch from develop to test-target-contract August 7, 2026 10:45
@erak
erak marked this pull request as ready for review August 7, 2026 10:47
@erak
erak force-pushed the common-metadata-hash branch from f37c119 to 8da0742 Compare August 7, 2026 15:47
@erak
erak force-pushed the common-metadata-hash branch from 8da0742 to 8f418a0 Compare August 7, 2026 16:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants