release: merge release-dev/elip5 token upgrade #1333
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
bowenli86
changed the title
bowenli86
force-pushed
the
release-dev/elip5-token-upgrade
branch
from
8e5b016 to
7f25df3
Compare
bowenli86
force-pushed
the
release-dev/elip5-token-upgrade
branch
from
232a196 to
c38aa29
Compare
**Motivation:** per ELIP5, remove transfer restriction code from eigen mint function will be removed in next PR so we can limit change scope in each PR, and make each change more readable **Modifications:** remove transfer restriction code from eigen **Result:** remove transfer restriction code from eigen
**Motivation:** per ELIP-5, remove transfer restriction function from backing eigen **Modifications:** remove transfer restriction function from backing eigen **Result:** remove transfer restriction function from backing eigen
…gen constructor to break circular dependency on Eigen (#1246) **Motivation:** - consolidate mint function to BackingEigen - change BackingEigen constructor to break circular dependency on Eigen **Modifications:** - remove mint function in Eigen - move all tests to use mint function in BackingEigen - change integration tests setup to have minters mint their own supply - change BackingEigen constructor to break circular dependency on Eigen **Result:** - only BackingEigen has mint capability - no more circular dependency between BackingEigen and Eigen
…bility (#1331) **Motivation:** add TokenWrapped and TokenUnwrapped events in Eigen for observability **Modifications:** add TokenWrapped and TokenUnwrapped events in Eigen add corresponding tests **Result:** we can track wrapping/unwrapping actions between Eigen and BackingEigen
bowenli86
force-pushed
the
release-dev/elip5-token-upgrade
branch
from
c38aa29 to
cb0df90
Compare
bowenli86
requested review from
0xClandestine,
wadealexc,
ypatil12 and
gpsanant
and removed request for
0xClandestine,
wadealexc,
ypatil12 and
gpsanant
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Motivation:
What's ELIP5? https://github.com/eigenfoundation/ELIPs/blob/main/ELIPs/ELIP-005.md
ELIP-5 dev work is done, all commits in release-dev/elip5-token-upgrade have been reviewed and approved by Jeff. To verify, click PR link for each commit.
This PR merges
release-dev/elip5-token-upgradebranch tomain, per eigengit process, and prepare to start necessary auditing onmainModifications:
merge release-dev/elip5-token-upgrade to main
Result:
merge release-dev/elip5-token-upgrade to main