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

Create HIP: Add no-op compatibility for cancun blobs #866

Merged
merged 7 commits into from
Feb 16, 2024

Conversation

shemnon
Copy link
Contributor

@shemnon shemnon commented Jan 24, 2024

Description::

HIP to add changes for EVM compatibility in the absence of transaction blobs. Covers two new opcodes, VERSIONEDHASH and BLOBBASEFEE

Related issue(s):

Fixes #

Notes for reviewer:

This is one of four related Cancun HIPs. Because of the broad coverage it was broken into 4 pieces.

Checklist

  • Documented (Code comments, README, etc.)
  • Tested (unit, integration, etc.)

Hip to add changes for EVM compatibility in the absence of transaction blobs.

Signed-off-by: Danno Ferrin <[email protected]>
Copy link

netlify bot commented Jan 24, 2024

Deploy Preview for hedera-hips ready!

Name Link
🔨 Latest commit 00d8049
🔍 Latest deploy log https://app.netlify.com/sites/hedera-hips/deploys/65cce75d0e424900071efd4b
😎 Deploy Preview https://deploy-preview-866--hedera-hips.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@shemnon shemnon changed the title Add no-op compatibility for cancun blobs Create HIP: Add no-op compatibility for cancun blobs Jan 24, 2024
@shemnon shemnon mentioned this pull request Jan 24, 2024
2 tasks
Signed-off-by: Danno Ferrin <[email protected]>
HIP/hip-866.md Outdated Show resolved Hide resolved
HIP/hip-866.md Show resolved Hide resolved
HIP/hip-866.md Show resolved Hide resolved
Signed-off-by: Danno Ferrin <[email protected]>
david-bakin-sl
david-bakin-sl previously approved these changes Feb 5, 2024
Copy link
Member

@david-bakin-sl david-bakin-sl left a comment

Choose a reason for hiding this comment

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

LGTM

HIP/hip-866.md Outdated Show resolved Hide resolved
Signed-off-by: Danno Ferrin <[email protected]>
Nana-EC
Nana-EC previously approved these changes Feb 10, 2024
Copy link
Contributor

@Nana-EC Nana-EC left a comment

Choose a reason for hiding this comment

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

LG.
1 spelling issue

HIP/hip-866.md Outdated Show resolved Hide resolved
Co-authored-by: Nana Essilfie-Conduah <[email protected]>
Signed-off-by: Danno Ferrin <[email protected]>
Nana-EC
Nana-EC previously approved these changes Feb 14, 2024
Copy link
Contributor

@Nana-EC Nana-EC left a comment

Choose a reason for hiding this comment

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

LG

Copy link
Member

@david-bakin-sl david-bakin-sl left a comment

Choose a reason for hiding this comment

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

LGTM recheck

@shemnon shemnon merged commit 85637d9 into hashgraph:main Feb 16, 2024
6 checks passed
kimbor pushed a commit to kimbor/hedera-improvement-proposal that referenced this pull request Jun 24, 2024
Signed-off-by: Danno Ferrin <[email protected]>
Signed-off-by: Danno Ferrin <[email protected]>
Co-authored-by: Nana Essilfie-Conduah <[email protected]>
Co-authored-by: Michael Garber <[email protected]>
Signed-off-by: Kim Rader <[email protected]>
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.

4 participants