Skip to content

Commit 1ebbd16

Browse files
authored
Update Copilot i18n guidelines for reviews (#12274)
1 parent bd785d7 commit 1ebbd16

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

.github/copilot-instructions.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -50,7 +50,7 @@ npm run watch # Dev mode with hot reload
5050

5151
**DRY:** Abstract repeated code blocks into shared helpers.
5252

53-
**i18n:** Wrap all user-facing strings in translation functions (`$_(...)` or `$:_('...')`). Use named placeholders (`_('Hello, %(name)s')`) and `$:_()` for HTML content to prevent double-escaping.
53+
**i18n:** Wrap all user-facing strings in translation functions (`$_(...)` or `$:_('...')`). Do not use string concatenation to construct user-visible strings; prefer a single parameterized translation string instead. Use named placeholders (`_('Hello, %(name)s')`) rather than positional placeholders such as `%s` so translators have more context. Use `$:_()` for HTML content to prevent double-escaping.
5454

5555
**FastAPI:** When touching FastAPI code, follow [FastAPI best practices](https://raw.githubusercontent.com/fastapi/fastapi/refs/heads/master/fastapi/.agents/skills/fastapi/SKILL.md) and ensure compliance with their recommendations. When in doubt, reference other parts of our FastAPI codebase for how we do things, as we generally follow established best practices.
5656

0 commit comments

Comments
 (0)