Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
There was a problem hiding this comment.
Note
Copilot was unable to run its full agentic suite in this review.
Pull request overview
Updates the header.description localization string across multiple locales, replacing existing translations with the English source text.
Changes:
- Replaced
header.descriptionwith the English string in many non-English locale files - Removed previously localized
header.descriptiontranslations (Chinese, Vietnamese, Ukrainian, etc.)
Reviewed changes
Copilot reviewed 19 out of 19 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| public/locales/zh/translation.json | Replaces Chinese header.description translation with English text |
| public/locales/vi/translation.json | Replaces Vietnamese header.description translation with English text |
| public/locales/uk/translation.json | Replaces Ukrainian header.description translation with English text |
| public/locales/tr/translation.json | Replaces Turkish header.description translation with English text |
| public/locales/sa/translation.json | Replaces Sanskrit header.description translation with English text |
| public/locales/ru/translation.json | Replaces Russian header.description translation with English text |
| public/locales/pt/translation.json | Replaces Portuguese header.description translation with English text |
| public/locales/pl/translation.json | Replaces Polish header.description translation with English text |
| public/locales/nl/translation.json | Replaces Dutch header.description translation with English text |
| public/locales/ko/translation.json | Replaces Korean header.description translation with English text |
| public/locales/ja/translation.json | Replaces Japanese header.description translation with English text |
| public/locales/it/translation.json | Replaces Italian header.description translation with English text |
| public/locales/hi/translation.json | Replaces Hindi header.description translation with English text |
| public/locales/fr/translation.json | Replaces French header.description translation with English text |
| public/locales/fi/translation.json | Replaces Finnish header.description translation with English text |
| public/locales/fa/translation.json | Replaces Persian header.description translation with English text |
| public/locales/es/translation.json | Replaces Spanish header.description translation with English text |
| public/locales/cs/translation.json | Replaces Czech header.description translation with English text |
| public/locales/be/translation.json | Replaces Belarusian header.description translation with English text |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
| @@ -1,5 +1,5 @@ | |||
| { | |||
| "header.description": "一个简单的情绪选择器，显示所有支持的 GitHub 表情符号。", | |||
| "header.description": "A simple emoji picker that displays all the supported GitHub emojis.", | |||
| @@ -1,5 +1,5 @@ | |||
| { | |||
| "header.description": "Bộ chọn cảm xúc đơn giản hiển thị tất cả các biểu tượng cảm xúc GitHub được hỗ trợ.", | |||
| "header.description": "A simple emoji picker that displays all the supported GitHub emojis.", | |||
There was a problem hiding this comment.
🔵 Needs a closer look
Multiple non-English locale files now contain the English source string for header.description, which is a user-facing localization regression.
Review details
Suppressed comments (18)
Previously missed (16) — in code that hasn't changed since the last review.
public/locales/be/translation.json:2
- This be locale string was replaced with the English source text, which is a localization regression for Belarusian users. Restore the previous Belarusian translation for
header.description.
public/locales/cs/translation.json:2 - This cs locale string was replaced with the English source text, which is a localization regression for Czech users. Restore the previous Czech translation for
header.description.
public/locales/es/translation.json:2 - This es locale string was replaced with the English source text, which is a localization regression for Spanish users. Restore the previous Spanish translation for
header.description.
public/locales/fa/translation.json:2 - This fa locale string was replaced with the English source text, which is a localization regression for Persian users. Restore the previous Persian translation for
header.description.
public/locales/fi/translation.json:2 - This fi locale string was replaced with the English source text, which is a localization regression for Finnish users. Restore the previous Finnish translation for
header.description.
public/locales/fr/translation.json:2 - This fr locale string was replaced with the English source text, which is a localization regression for French users. Restore the previous French translation for
header.description.
public/locales/hi/translation.json:2 - This hi locale string was replaced with the English source text, which is a localization regression for Hindi users. Restore the previous Hindi translation for
header.description.
public/locales/it/translation.json:2 - This it locale string was replaced with the English source text, which is a localization regression for Italian users. Restore the previous Italian translation for
header.description.
public/locales/ja/translation.json:2 - This ja locale string was replaced with the English source text, which is a localization regression for Japanese users. Restore the previous Japanese translation for
header.description.
public/locales/ko/translation.json:2 - This ko locale string was replaced with the English source text, which is a localization regression for Korean users. Restore the previous Korean translation for
header.description.
public/locales/nl/translation.json:2 - This nl locale string was replaced with the English source text, which is a localization regression for Dutch users. Restore the previous Dutch translation for
header.description.
public/locales/pl/translation.json:2 - This pl locale string was replaced with the English source text, which is a localization regression for Polish users. Restore the previous Polish translation for
header.description.
public/locales/pt/translation.json:2 - This pt locale string was replaced with the English source text, which is a localization regression for Portuguese users. Restore the previous Portuguese translation for
header.description.
public/locales/sa/translation.json:2 - This sa locale string was replaced with the English source text, which is a localization regression for Sanskrit users. Restore the previous Sanskrit translation for
header.description.
public/locales/tr/translation.json:2 - This tr locale string was replaced with the English source text, which is a localization regression for Turkish users. Restore the previous Turkish translation for
header.description.
public/locales/uk/translation.json:2 - This uk locale string was replaced with the English source text, which is a localization regression for Ukrainian users. Restore the previous Ukrainian translation for
header.description.
public/locales/zh/translation.json:2
- This zh locale string was replaced with the English source text, which is a localization regression for Chinese users. Restore the previous Chinese translation for
header.description.
"header.description": "A simple emoji picker that displays all the supported GitHub emojis.",
public/locales/vi/translation.json:2
- This vi locale string was replaced with the English source text, which is a localization regression for Vietnamese users. Restore the previous Vietnamese translation for
header.description.
"header.description": "A simple emoji picker that displays all the supported GitHub emojis.",
- Files reviewed: 19/19 changed files
- Comments generated: 0 new
- Review effort level: Lite
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
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.
No description provided.