Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Change ABI suffix to Trait in dual case account and eth account traits #1096

Merged

Conversation

ggonzalez94
Copy link
Collaborator

@ggonzalez94 ggonzalez94 commented Aug 12, 2024

Fixes #1000

DON'T MERGE BEFORE 0.15.1 IS RELEASED

  • Added entry to CHANGELOG.md

@ericnordelo
Copy link
Member

Thanks @ggonzalez94. This change while small is breaking the API, and we must add a Change (Breaking) entry to the changelog. Let's not merge it before 0.15.1 is released.

@ggonzalez94 ggonzalez94 self-assigned this Aug 12, 2024
@ggonzalez94
Copy link
Collaborator Author

@ericnordelo just added the changelog. Agreed we shouldn't merge it before 0.15.1 is released.

Copy link
Member

@ericnordelo ericnordelo left a comment

Choose a reason for hiding this comment

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

LGTM! Let's wait for 0.15.1 before merging it.

CHANGELOG.md Outdated Show resolved Hide resolved
@immrsd immrsd self-requested a review August 13, 2024 07:22
Co-authored-by: Eric Nordelo <eric.nordelo39@gmail.com>
@ggonzalez94
Copy link
Collaborator Author

@ericnordelo with the release of 0.15.1 are we good to review and release this?

Copy link
Collaborator

@andrew-fleming andrew-fleming left a comment

Choose a reason for hiding this comment

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

LGTM! We just need to fix conflicts

CHANGELOG.md Outdated Show resolved Hide resolved
Copy link
Collaborator

@immrsd immrsd left a comment

Choose a reason for hiding this comment

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

LGTM!

Copy link
Member

@ericnordelo ericnordelo left a comment

Choose a reason for hiding this comment

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

LGTM!

@ericnordelo ericnordelo merged commit 7cd11ab into OpenZeppelin:main Aug 15, 2024
6 checks passed
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.

Change ABI suffix to Trait in dual case account and eth account traits
4 participants