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

WIP: Add tutorial for EIP-7702 #669

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

Conversation

akshay-ap
Copy link
Member

Changes in PR:

  • Create a tutorial to use EIP-7702 approach 1: Modified safe proxy to set code for EOA

@akshay-ap akshay-ap self-assigned this Jan 6, 2025
@akshay-ap akshay-ap marked this pull request as draft January 6, 2025 11:19
Copy link

github-actions bot commented Jan 6, 2025

Branch preview

✅ Deployed successfully in branch deployment:

https://feature_eip_7702_tutorial--docs.review.5afe.dev

Copy link

github-actions bot commented Jan 6, 2025

Overall readability score: 31.41 (🟢 +0.1)

File Readability
approach-1.mdx 63.02 (-)
View detailed metrics

🟢 - Shows an increase in readability
🔴 - Shows a decrease in readability

File Readability FRE GF ARI CLI DCRS
approach-1.mdx 63.02 49.31 8.89 10.6 11.65 7.7
  - - - - - -

Averages:

  Readability FRE GF ARI CLI DCRS
Average 31.41 27.35 13.69 17.42 15.72 8.97
  🟢 +0.1 🟢 +0.07 🟢 +0.02 🟢 +0.02 🟢 +0.01 🟢 +0
View metric targets
Metric Range Ideal score
Flesch Reading Ease 100 (very easy read) to 0 (extremely difficult read) 60
Gunning Fog 6 (very easy read) to 17 (extremely difficult read) 8 or less
Auto. Read. Index 6 (very easy read) to 14 (extremely difficult read) 8 or less
Coleman Liau Index 6 (very easy read) to 17 (extremely difficult read) 8 or less
Dale-Chall Readability 4.9 (very easy read) to 9.9 (extremely difficult read) 6.9 or less

@tanay1337 tanay1337 requested a review from valle-xyz January 7, 2025 14:02
@tanay1337 tanay1337 removed their request for review January 21, 2025 09:58
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.

1 participant