Skip to content

Conversation

Privat33r-dev
Copy link
Contributor

Prior to the change the doc wasn't formatted for Python code, even though mostly Python syntax is used, which might've lead to decrease in comprehension as the syntax wasn't highlighted.

All the code was formatted with the Black formatter to ease comprehension.

Additionally, the following line contained bug:

        if node = '': return ''

The comparison operator (==) was likely meant.

Description

Related Issue

Prior to the change the doc wasn't formatted for Python code, even though mostly Python syntax is used,
which might've lead to decrease in comprehension as the syntax wasn't highlighted.

All the code was formatted with the Black formatter to ease comprehension.

Additionally, the following line contained bug:
```python
        if node = '': return ''
```

The comparison operator (==) was likely meant.
@github-actions github-actions bot added the content 🖋️ This involves copy additions or edits label Dec 11, 2024
Copy link

netlify bot commented Dec 11, 2024

Deploy Preview for ethereumorg ready!

Name Link
🔨 Latest commit 24b4896
🔍 Latest deploy log https://app.netlify.com/projects/ethereumorg/deploys/687996afb4e72d00080b0f51
😎 Deploy Preview https://deploy-preview-14486--ethereumorg.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
7 paths audited
Performance: 53 (🔴 down 2 from production)
Accessibility: 94 (🔴 down 1 from production)
Best Practices: 92 (no change from production)
SEO: 99 (no change from production)
PWA: 59 (no change from production)
View the detailed breakdown and full score reports

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

Copy link

gitguardian bot commented Dec 30, 2024

️✅ There are no secrets present in this pull request anymore.

If these secrets were true positive and are still valid, we highly recommend you to revoke them.
While these secrets were previously flagged, we no longer have a reference to the
specific commits where they were detected. Once a secret has been leaked into a git
repository, you should consider it compromised, even if it was deleted immediately.
Find here more information about risks.


🦉 GitGuardian detects secrets in your source code to help developers and security teams secure the modern development process. You are seeing this because you or someone else with access to this repository has authorized GitGuardian to scan your pull request.

@Privat33r-dev
Copy link
Contributor Author

@Aldekein I guess that it is a false positive. It's not even a part of the PR.

Copy link
Contributor

This issue is stale because it has been open 30 days with no activity.

@github-actions github-actions bot added the Status: Stale This issue is stale because it has been open 30 days with no activity. label Jan 30, 2025
Copy link
Member

@wackerow wackerow left a comment

Choose a reason for hiding this comment

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

Thanks @Privat33r-dev! Sorry for delay, pulling in

@wackerow wackerow merged commit b61d4be into ethereum:dev Jul 18, 2025
2 checks passed
Copy link

gitpoap-bot bot commented Jul 18, 2025

Congrats, your important contribution to this open-source project has earned you a GitPOAP!

GitPOAP: 2025 Ethereum.org Contributor:

GitPOAP: 2025 Ethereum.org Contributor GitPOAP Badge

Join the [ethereum.org Discord server](https://ethereum.org/discord) to explore more ways to contribute to the project. Depending on the tasks you complete, you may also unlock additional rewards. Visit [ethereum.org/contributing](https://ethereum.org/contributing) to learn more.

Head to gitpoap.io & connect your GitHub account to mint!Keep buidling, keep learning, and let's grow the Ethereum open-source community together 🌱

Learn more about GitPOAPs here.

wackerow added a commit that referenced this pull request Jul 18, 2025
#14486 accidentally reverted; reinstates changes from #13758
@Privat33r-dev Privat33r-dev deleted the patch-2 branch July 18, 2025 13:37
@Privat33r-dev Privat33r-dev restored the patch-2 branch July 18, 2025 13:37
@Privat33r-dev Privat33r-dev deleted the patch-2 branch July 18, 2025 13:37
@Privat33r-dev
Copy link
Contributor Author

Thanks for the merge! I am just glad that it was merged while still being relevant.

This was referenced Jul 30, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
content 🖋️ This involves copy additions or edits Status: Stale This issue is stale because it has been open 30 days with no activity.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants