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

Update ERC-7540: Move to Review #52

Merged
merged 11 commits into from
Nov 15, 2023

Conversation

Joeysantoro
Copy link
Contributor

Addresses comments from https://ethereum-magicians.org/t/eip-7540-asynchronous-erc-4626-tokenized-vaults/16153 and transitions the EIP to Review status

@Joeysantoro Joeysantoro changed the title review comments Update ERC-7540 to Review Oct 27, 2023
@github-actions github-actions bot added the w-ci label Oct 27, 2023
@eip-review-bot eip-review-bot self-requested a review October 27, 2023 20:04
@Pandapip1
Copy link
Member

@eth-bot rerun

Copy link

There has been no activity on this pull request for 2 weeks. It will be closed after 3 months of inactivity. If you would like to move this PR forward, please respond to any outstanding feedback or add a comment indicating that you have addressed all required feedback and are ready for a review.

@eip-review-bot
Copy link
Collaborator

eip-review-bot commented Nov 13, 2023

✅ All reviewers have approved.

@eip-review-bot eip-review-bot changed the title Update ERC-7540 to Review Update ERC-7540: Move to Review Nov 13, 2023
SamWilsn
SamWilsn previously approved these changes Nov 14, 2023
Copy link
Contributor

@SamWilsn SamWilsn left a comment

Choose a reason for hiding this comment

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

Some mixed capitalization in your definitions section bullets.

@eip-review-bot eip-review-bot enabled auto-merge (squash) November 14, 2023 15:37
Copy link
Collaborator

@eip-review-bot eip-review-bot left a comment

Choose a reason for hiding this comment

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

All Reviewers Have Approved; Performing Automatic Merge...

Copy link
Collaborator

@eip-review-bot eip-review-bot left a comment

Choose a reason for hiding this comment

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

All Reviewers Have Approved; Performing Automatic Merge...

@SamWilsn SamWilsn closed this Nov 14, 2023
auto-merge was automatically disabled November 14, 2023 15:47

Pull request was closed

@SamWilsn SamWilsn reopened this Nov 14, 2023
@eip-review-bot eip-review-bot enabled auto-merge (squash) November 14, 2023 15:48
Copy link
Collaborator

@eip-review-bot eip-review-bot left a comment

Choose a reason for hiding this comment

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

All Reviewers Have Approved; Performing Automatic Merge...

Copy link
Member

@Pandapip1 Pandapip1 left a comment

Choose a reason for hiding this comment

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

Due to a temporary kludge the links cannot be updated

eip-review-bot
eip-review-bot previously approved these changes Nov 14, 2023
Copy link
Collaborator

@eip-review-bot eip-review-bot left a comment

Choose a reason for hiding this comment

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

All Reviewers Have Approved; Performing Automatic Merge...

Co-authored-by: Gavin John <[email protected]>
auto-merge was automatically disabled November 14, 2023 17:38

Head branch was pushed to by a user without write access

@Joeysantoro Joeysantoro dismissed stale reviews from eip-review-bot and SamWilsn via 3a5828b November 14, 2023 17:38
Co-authored-by: Gavin John <[email protected]>
@github-actions github-actions bot removed the w-ci label Nov 14, 2023
Copy link

The commit 28cec7d (as a parent of 9e2bac6) contains errors.
Please inspect the Run Summary for details.

@github-actions github-actions bot added the w-ci label Nov 14, 2023
@eip-review-bot eip-review-bot enabled auto-merge (squash) November 15, 2023 15:42
Copy link
Collaborator

@eip-review-bot eip-review-bot left a comment

Choose a reason for hiding this comment

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

All Reviewers Have Approved; Performing Automatic Merge...

@eip-review-bot eip-review-bot merged commit 607dd2b into ethereum:master Nov 15, 2023
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.

5 participants