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: TXINDEX precompile #8981

Open
wants to merge 9 commits into
base: master
Choose a base branch
from

Conversation

Marchhill
Copy link
Contributor

@Marchhill Marchhill commented Oct 21, 2024

This EIP proposes to add a new precompile that returns the index of the transaction being executed within the current block.

@Marchhill Marchhill requested a review from eth-bot as a code owner October 21, 2024 09:47
@github-actions github-actions bot added c-new Creates a brand new proposal s-draft This EIP is a Draft t-core labels Oct 21, 2024
@eth-bot
Copy link
Collaborator

eth-bot commented Oct 21, 2024

File EIPS/eip-7793.md

Requires 1 more reviewers from @g11tech, @lightclient, @SamWilsn

@eth-bot eth-bot added e-consensus Waiting on editor consensus e-review Waiting on editor to review labels Oct 21, 2024
Copy link

The commit 9fd19ed (as a parent of 50a1bad) contains errors.
Please inspect the Run Summary for details.

@github-actions github-actions bot added the w-ci Waiting on CI to pass label Oct 21, 2024
@github-actions github-actions bot removed the w-ci Waiting on CI to pass label Oct 30, 2024
@frangio
Copy link
Contributor

frangio commented Oct 31, 2024

Previous discussion: #222

smartprogrammer93 and others added 3 commits October 31, 2024 18:16
* change to precompile

* update description

* reword rationale
@Marchhill Marchhill changed the title Add EIP: TXINDEX opcode Add EIP: TXINDEX precompile Nov 8, 2024
@Marchhill Marchhill requested a review from abcoathup December 3, 2024 21:53
@Marchhill
Copy link
Contributor Author

Hey @abcoathup you think this is ready to merge?

@abcoathup
Copy link
Contributor

@Marchhill you need an EIP editor to review (I am not an editor).

There is a TODO in the security section that you should remove/address.

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-core
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants