Skip to content

chore: sync translations from crowdin - #798

Open
surajshetty3416 wants to merge 3 commits into
developfrom
l10n_develop
Open

chore: sync translations from crowdin#798
surajshetty3416 wants to merge 3 commits into
developfrom
l10n_develop

Conversation

@surajshetty3416

Copy link
Copy Markdown
Member

No description provided.

@greptile-apps

greptile-apps Bot commented Sep 5, 2026

Copy link
Copy Markdown
Contributor

Confidence Score: 5/5

Safe to merge, though the three inaccurate locale strings should be corrected.

Bosnian and Croatian now misidentify template fixtures as add-ons, while the previously reported Swedish compound split remains outstanding; these affect message clarity rather than behavior.

Files Needing Attention: builder/locale/bs.po, builder/locale/hr.po, builder/locale/sv.po

Prompt To Fix All With AI
### Issue 1
builder/locale/bs.po:4734
**Fixture Mistranslated as Add-on**

The Bosnian translation renders the technical term “template fixture” as “template add-on,” so the validation error misidentifies the rejected filesystem-backed record. The Croatian translation at `builder/locale/hr.po:4734` has the same problem; use wording that preserves the meaning of “fixture” in both locales.

---

For each issue above, determine whether it is valid and should be fixed. If so, fix it directly.

Reviews (2): Last reviewed commit: "chore: Croatian translations" | Re-trigger Greptile

Comment thread builder/locale/sv.po
#: builder/template_sync.py:55
msgid "Unsafe template fixture name: {0}"
msgstr "Osäker malltillägg namn: {0}"
msgstr "Osäker mall tillägg namn: {0}"

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Swedish Compound Split

Changing malltillägg to mall tillägg makes this user-facing validation message grammatically incorrect and less clear. Keep the compound noun joined.

Suggested change
msgstr "Osäker mall tillägg namn: {0}"
msgstr "Osäker malltillägg namn: {0}"
Prompt To Fix With AI
This is a comment left during a code review.
Path: builder/locale/sv.po
Line: 4734

Comment:
**Swedish Compound Split**

Changing `malltillägg` to `mall tillägg` makes this user-facing validation message grammatically incorrect and less clear. Keep the compound noun joined.

```suggestion
msgstr "Osäker malltillägg namn: {0}"
```

---

For each issue above, determine whether it is valid and should be fixed. If so, fix it directly.

Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!

@codecov

codecov Bot commented Sep 5, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 66.06%. Comparing base (b6bd11d) to head (3936dae).

Additional details and impacted files
@@           Coverage Diff            @@
##           develop     #798   +/-   ##
========================================
  Coverage    66.06%   66.06%           
========================================
  Files           93       93           
  Lines        10386    10386           
========================================
  Hits          6862     6862           
  Misses        3524     3524           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Comment thread builder/locale/bs.po
#: builder/template_sync.py:55
msgid "Unsafe template fixture name: {0}"
msgstr "Naziv nesigurnog predloška za učvršćivanje: {0}"
msgstr "Naziv nesigurnog predloška dodatka: {0}"

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Fixture Mistranslated as Add-on

The Bosnian translation renders the technical term “template fixture” as “template add-on,” so the validation error misidentifies the rejected filesystem-backed record. The Croatian translation at builder/locale/hr.po:4734 has the same problem; use wording that preserves the meaning of “fixture” in both locales.

Prompt To Fix With AI
This is a comment left during a code review.
Path: builder/locale/bs.po
Line: 4734

Comment:
**Fixture Mistranslated as Add-on**

The Bosnian translation renders the technical term “template fixture” as “template add-on,” so the validation error misidentifies the rejected filesystem-backed record. The Croatian translation at `builder/locale/hr.po:4734` has the same problem; use wording that preserves the meaning of “fixture” in both locales.

---

For each issue above, determine whether it is valid and should be fixed. If so, fix it directly.

Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant