i18n: Gemini translations (it)#17841
Merged
Merged
Conversation
✅ Deploy Preview for ethereumorg ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
github-actions
Bot
added
content 🖋️
Contributor
|
🌐 Translation review started. View progress |
Contributor
Translation Quality ReviewPR: 17841 | Language: it (Italian) | Files reviewed: 305 | Date: 2026-03-24 Scores by Section:
Critical Issues (14 total) 1. NatSpec Comment Corruption (6 files, 90 instances)
2. Translated skill Frontmatter (12 files)
3. Code Block Divergence (1 file) Warnings (8 total)
Overall: High-quality Italian translation. Issues are mechanical (NatSpec corruption, translated skill fields) not translation quality. Non-tutorial content scores 9.5+/10. Reviewed by Claude Code |
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> Co-Authored-By: wackerow <54227730+wackerow@users.noreply.github.com>
Broaden quoteFrontmatterNonAscii to also quote values containing colon-space or space-hash, which break YAML parsing. Adds 3 tests, documents pattern #47. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> Co-Authored-By: wackerow <54227730+wackerow@users.noreply.github.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> Co-Authored-By: wackerow <54227730+wackerow@users.noreply.github.com>
collapseInlineHtmlFromEnglish was matching across code fences and crossing newlines via \s*, collapsing unrelated </div> tags. Add code fence protection and use [ \t]* instead of \s* for closing tag whitespace. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> Co-Authored-By: wackerow <54227730+wackerow@users.noreply.github.com>
github-actions
Bot
added
documentation 📖
myelinated-wackerow
approved these changes
Mar 25, 2026
Collaborator
myelinated-wackerow
left a comment
myelinated-wackerow
left a comment
There was a problem hiding this comment.
Translation Quality Review
PR: #17841
Language: Italian (it)
Files reviewed: 305 (256 MD + 49 JSON)
Date: 2026-03-25
Quality Scores - Italian (it) - 9.1/10
| Category | Score | Notes |
|---|---|---|
| MDX Syntax Safety | 10/10 | All MDX components properly structured. URLs with parens correctly wrapped. No raw < in prose. |
| Href Integrity | 10/10 | 50+ internal links spot-checked, all use English paths. No translated routes. |
| Structural Match | 10/10 | Header counts match English. Frontmatter clean. Code fences balanced. |
| Brand Name Preservation | 9/10 | All brands preserved. Two upstream "Consensys" casing issues (from English source). |
| Technical Accuracy | 9/10 | Glossary terms correctly used throughout. One minor untranslated "proof-of-stake" instance. |
| Terminology Consistency | 9/10 | All 27 glossary terms verified across 200+ files. |
| Semantic Fidelity | 9/10 | Faithful translations. No semantic inversions. Natural Italian phrasing. |
| Tone/Register | 7/10 | Consistently informal "tu" (good), but translators-guide claims formal policy (pre-existing mismatch). |
Critical Issues: None
No must-fix issues found. All build-breaking problems (YAML quoting, frontmatter protection) resolved by sanitizer.
Warnings (Non-Blocking)
| File | Issue | Details |
|---|---|---|
translators-guide/index.md |
Register policy mismatch | Guide says "formal" but all content uses informal "tu" -- pre-existing |
nodes-and-clients/index.md:21 |
Untranslated term | "proof-of-stake" left in English once; translated correctly elsewhere |
prediction-markets/index.md:36 |
Minor phrasing | "Decentralizzazione" as card title vs "Decentralizzato" |
energy-consumption/index.md |
Naming inconsistency | "Il Merge" vs "The Merge" mixed within file |
zk-rollups/index.md, golang/index.md |
"Consensys" casing | Upstream from English source (should be "ConsenSys") |
Review Methodology
- 3 specialized review agents (Structural, Terminology, Semantic) deployed in parallel
- Structural: MDX syntax, hrefs, code blocks, frontmatter validation
- Terminology: Brand names, glossary compliance (27 community-voted terms), ticker symbols
- Semantic: Meaning preservation, tone/register, untranslated content detection
- Sanitizer: post-import sanitization applied (55 files auto-fixed)
Reviewed by Claude Opus 4.6
Member
Author
|
Translation Failures:
Noting for follow-up task |
This was referenced Mar 27, 2026
This was referenced Mar 27, 2026
Closed
Merged
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
This PR contains automated Gemini translations via Gemini.
🔗 View workflow run
Languages translated
it
JSON changes (
src/intl/{locale}/)Markdown changes (
public/content/translations/{locale}/)