Skip to content

Rename BridgeERC20Core to BridgeFungible#6328

Merged
Amxx merged 2 commits intoOpenZeppelin:masterfrom
Amxx:rename/crosschain/bridge20core-to-bridgefungiblecore
Feb 7, 2026
Merged

Rename BridgeERC20Core to BridgeFungible#6328
Amxx merged 2 commits intoOpenZeppelin:masterfrom
Amxx:rename/crosschain/bridge20core-to-bridgefungiblecore

Conversation

@Amxx
Copy link
Collaborator

@Amxx Amxx commented Feb 5, 2026

Note: the changeset would be part of a new 5.6 RC, but it would be removed for the final release.

PR Checklist

  • Tests
  • Documentation
  • Changeset entry (run npx changeset add)

@Amxx Amxx added this to the 5.6 milestone Feb 5, 2026
@Amxx Amxx requested review from ernestognw and frangio February 5, 2026 16:26
@Amxx Amxx requested a review from a team as a code owner February 5, 2026 16:26
@changeset-bot
Copy link

changeset-bot bot commented Feb 5, 2026

🦋 Changeset detected

Latest commit: 534b71b

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
openzeppelin-solidity Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 5, 2026

Walkthrough

This pull request performs a systematic rename of the core bridge abstraction contract and its associated events. The abstract contract BridgeERC20Core is renamed to BridgeFungible, and the events CrosschainERC20TransferSent and CrosschainERC20TransferReceived are renamed to CrosschainFungibleTransferSent and CrosschainFungibleTransferReceived respectively. Corresponding import statements, inheritance declarations, and documentation references are updated across multiple contract files, test files, and README documentation. A patch-level changeset is added for the openzeppelin-solidity dependency.

Suggested labels

crosschain

🚥 Pre-merge checks | ✅ 2 | ❌ 1
❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title 'Rename BridgeERC20Core to BridgeFungible' directly and clearly describes the main change in the pull request.
Description check ✅ Passed The description explains the changeset is part of a new 5.6 RC to be removed for final release, and confirms all PR checklist items (Tests, Documentation, Changeset entry) are completed.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Comment @coderabbitai help to get the list of available commands and usage tips.

@Amxx Amxx requested a review from gonzaotc February 5, 2026 16:44
Copy link
Member

@ernestognw ernestognw left a comment

Choose a reason for hiding this comment

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

LGTM

@Amxx Amxx merged commit 7bcb960 into OpenZeppelin:master Feb 7, 2026
21 checks passed
Amxx added a commit that referenced this pull request Feb 10, 2026
Signed-off-by: Hadrien Croubois <hadrien.croubois@gmail.com>
@Moses-main

This comment was marked as spam.

@gonzaotc
Copy link
Contributor

gonzaotc commented Mar 2, 2026

Hi team,

I'd like to help with this. Let me know the details!

Hey @Moses-main I've seen you across many issues in the repo asking to get tasks assigned to you. I wanted to let you know that we don't assign tasks to anyone to avoid blocking other contributors. If you see an issue or have an idea for something you're willing to fix, you don't need to wait for our permission, and you're totally welcomed to work on it directly and open a PR for us to review, which we will do happily if it has a minimun of quality and is not AI generated slop, as many cases we have seen recently. Best wishes!

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