Skip to content

docs: correct Github typo across whole website#17671

Merged
wackerow merged 3 commits into
ethereum:devfrom
Uaitt:docs/github-typo
Apr 23, 2026
Merged

docs: correct Github typo across whole website#17671
wackerow merged 3 commits into
ethereum:devfrom
Uaitt:docs/github-typo

Conversation

@Uaitt
Copy link
Copy Markdown
Contributor

@Uaitt Uaitt commented Feb 26, 2026

Github should actually be written as GitHub. This typo appears in many places across the whole website, even in different translations.

That's why there are many files changed :)

@netlify
Copy link
Copy Markdown

netlify Bot commented Feb 26, 2026

Deploy Preview for ethereumorg ready!

Name Link
🔨 Latest commit 7cab2e6
🔍 Latest deploy log https://app.netlify.com/projects/ethereumorg/deploys/69a0112c8a1702000834d9a0
😎 Deploy Preview https://deploy-preview-17671.ethereum.it
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
7 paths audited
Performance: 60 (no change from production)
Accessibility: 94 (🟢 up 1 from production)
Best Practices: 100 (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.

@github-actions github-actions Bot added content 🖋️ This involves copy additions or edits translation 🌍 This is related to our Translation Program labels Feb 26, 2026
@github-actions
Copy link
Copy Markdown
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 Mar 29, 2026
Copy link
Copy Markdown
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.

Hey @Uaitt! Sorry for delay here.. appreciate this polish.

We've ended up with a bunch of merge conflicts, but they're primarily from the non-English version of these pages.

For context, a translation pipeline is being implemented for the site which will soon be run to generate a fresh set of translations based on the English. Subsequent to that it should detect changes to the English that keeps the rest up to date, and we'll be mostly hands-off the non-English content files.

As such, I would recommend we revert the changes to the non-English files in here, and keep it to English only. The changes won't propagate to non-English immediately, but will be coming shortly. That should bring the diff down to about five (5) files -- I can help take care of that to get this over the line.


If you're curious, when the translations are run again, the other files should be corrected with the new setup, enforced with the following:

/**
* Fix case-sensitive brand capitalization mistakes.
*
* Maps wrong-case variants to canonical forms. Skips URLs (github.com etc.)
* and code blocks. Only matches standalone occurrences (not inside URLs).
*/
const BRAND_CAPITALIZATION_FIXES: Record<string, string> = {
Metamask: "MetaMask",
Github: "GitHub",
}

@wackerow wackerow removed the Status: Stale This issue is stale because it has been open 30 days with no activity. label Apr 23, 2026
myelinated-wackerow and others added 2 commits April 22, 2026 18:35
Co-Authored-By: Claude <noreply@anthropic.com>
Co-Authored-By: wackerow <54227730+wackerow@users.noreply.github.com>
# Conflicts:
#	public/content/developers/docs/frameworks/index.md
@netlify
Copy link
Copy Markdown

netlify Bot commented Apr 23, 2026

Deploy Preview for ethereumorg ready!

Name Link
🔨 Latest commit 45fcee4
🔍 Latest deploy log https://app.netlify.com/projects/ethereumorg/deploys/69e98e34a5da98000814330f
😎 Deploy Preview https://deploy-preview-17671.ethereum.it
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
7 paths audited
Performance: 61 (🟢 up 3 from production)
Accessibility: 93 (no change from production)
Best Practices: 100 (no change from production)
SEO: 98 (🔴 down 1 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
Copy Markdown
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.

Went ahead and reverted the non-English changes. Thanks again @Uaitt ! Pulling in

@wackerow wackerow merged commit cb1ceaa into ethereum:dev Apr 23, 2026
10 of 11 checks passed
@Uaitt Uaitt deleted the docs/github-typo branch April 23, 2026 08:15
@wackerow
Copy link
Copy Markdown
Member

@all-contributors please add @Uaitt for bug fix

@allcontributors
Copy link
Copy Markdown
Contributor

@wackerow

@Uaitt already contributed before to bug

@wackerow wackerow mentioned this pull request Apr 24, 2026
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 translation 🌍 This is related to our Translation Program

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants