Conversation
WalkthroughA new Polish translation file ( Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes 🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: da5cf3b28f
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
locales/pl.json
Outdated
| "By {primaryAuthor}": "Autor: {primaryAuthor}", | ||
| "Close (Esc)": "Zamknij (Esc)", | ||
| "Comments": "Komentarze", | ||
| "Don't miss out on the latest issues. Sign up now to get access to the library of members-only issues.": "Nie przegap najnowszych wydań. Zarejestruj się teraz i uzyskaj dostęp do biblioteki wydań tylko dla subskrybentów.", |
There was a problem hiding this comment.
Match canonical apostrophe in translation key
Use the exact source key spelling here: this entry uses Don't (ASCII apostrophe), but the canonical key in locales/en.json/locales/context.json is Don’t (typographic apostrophe). Because locale lookup is exact-string based, this Polish translation will never be selected and that sentence will fall back to English at runtime.
Useful? React with 👍 / 👎.
There was a problem hiding this comment.
Pull request overview
Adds Polish (pl) locale support by introducing a new translation file for the theme’s UI strings.
Changes:
- Added
locales/pl.jsoncontaining Polish translations for existing theme i18n keys.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
locales/pl.json
Outdated
| "By {primaryAuthor}": "Autor: {primaryAuthor}", | ||
| "Close (Esc)": "Zamknij (Esc)", | ||
| "Comments": "Komentarze", | ||
| "Don't miss out on the latest issues. Sign up now to get access to the library of members-only issues.": "Nie przegap najnowszych wydań. Zarejestruj się teraz i uzyskaj dostęp do biblioteki wydań tylko dla subskrybentów.", |
There was a problem hiding this comment.
The translation key here uses a straight apostrophe ("Don't"), but the templates/context use a curly apostrophe ("Don’t"). Because i18n lookups are exact-match, this entry won’t be used and will leave the real key untranslated. Update the key to exactly match the canonical key string (curly apostrophe), keeping the Polish translation value the same.
| "Don't miss out on the latest issues. Sign up now to get access to the library of members-only issues.": "Nie przegap najnowszych wydań. Zarejestruj się teraz i uzyskaj dostęp do biblioteki wydań tylko dla subskrybentów.", | |
| "Don’t miss out on the latest issues. Sign up now to get access to the library of members-only issues.": "Nie przegap najnowszych wydań. Zarejestruj się teraz i uzyskaj dostęp do biblioteki wydań tylko dla subskrybentów.", |
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 5a53579. Configure here.
| "comment": "komentarz", | ||
| "comments": "komentarze", | ||
| "in {primaryTag}": "w {primaryTag}", | ||
| "jamie@example.com": "jan@kowalski.pl", |
There was a problem hiding this comment.
Placeholder email uses potentially real domain instead of example
Medium Severity
The jamie@example.com placeholder in pl.json is translated to jan@kowalski.pl. This uses a common Polish surname as the domain, kowalski.pl, instead of the Polish word for "example" (przykład). This deviates from the convention in other locales, which localize "example" in the domain, and may reference a real registered domain, raising privacy concerns.
Reviewed by Cursor Bugbot for commit 5a53579. Configure here.


Added
locales/pl.jsonwith Polish translations for theme strings.Note
Low Risk
Low risk: adds a new translation JSON file only, with no logic or runtime behavior changes beyond enabling Polish text when selected.
Overview
Adds Polish localization by introducing
locales/pl.jsonwith translations for the theme’s existing strings (navigation, pagination, membership/paywall, and common UI labels).Reviewed by Cursor Bugbot for commit 5a53579. Bugbot is set up for automated code reviews on this repo. Configure here.