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

Add EIP: ABI specification for REVERT reason string #7186

Closed
wants to merge 5 commits into from

Conversation

RenanSouza2
Copy link
Contributor

Hello everyone

This PR documents what is called ERC838 discussed in this link: #838
Solidity already implements this and I suggest to keep the same number

@github-actions github-actions bot added c-new Creates a brand new proposal s-draft This EIP is a Draft t-erc labels Jun 17, 2023
@eth-bot
Copy link
Collaborator

eth-bot commented Jun 17, 2023

File EIPS/eip-838.md

Requires 1 more reviewers from @axic, @Pandapip1, @SamWilsn, @xinbenlv

@eth-bot eth-bot changed the title ABI specification for REVERT reason string Add EIP: ABI specification for REVERT reason string Jun 17, 2023
@eth-bot eth-bot added e-consensus Waiting on editor consensus e-review Waiting on editor to review labels Jun 17, 2023
@github-actions github-actions bot added the w-ci Waiting on CI to pass label Jun 17, 2023
@github-actions github-actions bot removed the w-ci Waiting on CI to pass label Jun 21, 2023
@github-actions
Copy link

The commit 762c275 (as a parent of 886f9ed) contains errors.
Please inspect the Run Summary for details.

@SamWilsn
Copy link
Contributor

I am closing this pull request because we are in the process of separating EIPs and ERCs into distinct repositories. Unfortunately, as far as we are aware, GitHub does not provide any tools to ease this migration, so every pull request will need to be re-opened manually.

As this is a PR to create / modify an ERC, I will kindly ask you to redirect this to the new repository at ethereum/ERCs. We have prepared a guide to help with the process.

If there is relevant history here, please link to this PR from the new pull request.

On behalf of the EIP Editors, I apologize for this inconvenience.

@SamWilsn SamWilsn closed this Oct 25, 2023
@RenanSouza2 RenanSouza2 deleted the ERC-revert-string branch March 18, 2024 21:32
@RenanSouza2 RenanSouza2 restored the ERC-revert-string branch June 11, 2024 14:37
@RenanSouza2 RenanSouza2 deleted the ERC-revert-string branch November 16, 2024 02:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
c-new Creates a brand new proposal e-consensus Waiting on editor consensus e-review Waiting on editor to review s-draft This EIP is a Draft t-erc w-ci Waiting on CI to pass
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants