Skip to content

Translations update from OSGeo Weblate#277

Open
osgeoweblate wants to merge 1 commit into
pgRouting:developfrom
osgeoweblate:weblate-pgrouting-workshop-index
Open

Translations update from OSGeo Weblate#277
osgeoweblate wants to merge 1 commit into
pgRouting:developfrom
osgeoweblate:weblate-pgrouting-workshop-index

Conversation

@osgeoweblate

@osgeoweblate osgeoweblate commented Jun 14, 2026

Copy link
Copy Markdown

Translations update from OSGeo Weblate for pgRouting workshop/index.

Current translation status:

Weblate translation status

Summary by CodeRabbit

  • Bug Fixes

    • Corrected numeric formatting in Spanish language translations to ensure accurate display of percentages and other formatted values in translated content.
  • Chores

    • Updated translation system metadata and revision information to maintain consistency with current standards.

Currently translated at 100.0% (96 of 96 strings)

Co-authored-by: Celia Virginia Vergara Castillo <vicky@erosion.dev>
Translate-URL: https://weblate.osgeo.org/projects/pgrouting-workshop/basic-graphs/es/
Translation: pgRouting workshop/basic-graphs
@coderabbitai

coderabbitai Bot commented Jun 14, 2026

Copy link
Copy Markdown

Review Change Stack

Walkthrough

The Spanish graphs.po file has two edits: updated header metadata fields (PO-Revision-Date, X-Generator), and a fix to the "10% slower than…" translation where the percent sign is now escaped as 10%% for correct format-string handling.

Changes

Spanish graphs.po translation fix

Layer / File(s) Summary
Header metadata and percent escaping fix
locale/es/LC_MESSAGES/basic/graphs.po
Updates PO-Revision-Date and X-Generator header fields; changes 10% to 10%% in the taxi speed msgstr to correctly escape the percent character for format-string processing.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

A bunny hopped through strings one day,
And found a % gone astray.
"Two %% are better than one!"
She fixed the format, hops and done.
🐇✨

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'Translations update from OSGeo Weblate' accurately describes the main purpose of the PR, which is to apply translation updates from the Weblate integration system.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@locale/es/LC_MESSAGES/basic/graphs.po`:
- Around line 79-81: The duplicated taxi-speed entry at line 495 has
inconsistent percent-escaping compared to the corrected version at line 81. Find
the duplicate msgstr entry for "Speed is 10% slower than that of the particular
vehicles." at line 495 and change `10%` to `10%%` to match the escaping fix
applied at line 81, ensuring consistent python-format handling across both
entries.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 7f800243-4c46-4153-a7fb-231831968812

📥 Commits

Reviewing files that changed from the base of the PR and between 55f670e and 3920728.

📒 Files selected for processing (1)
  • locale/es/LC_MESSAGES/basic/graphs.po

Comment on lines 79 to +81
#, python-format
msgid "Speed is 10% slower than that of the particular vehicles."
msgstr "La velocidad es 10% inferior a la de los vehículos particulares."
msgstr "La velocidad es 10%% inferior a la de los vehículos particulares."

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

⚠️ Potential issue | 🟠 Major | ⚡ Quick win

Apply the same percent-escaping fix to the duplicated taxi-speed entry.

Line 81 is correctly fixed to 10%%, but the duplicated #, python-format entry at Line 495 still uses 10%. Keep escaping consistent across both entries to avoid python-format interpolation/validation errors.

Suggested patch
- msgstr "La velocidad de taxi es 10% inferior a la del vehículo particular."
+ msgstr "La velocidad de taxi es 10%% inferior a la del vehículo particular."
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@locale/es/LC_MESSAGES/basic/graphs.po` around lines 79 - 81, The duplicated
taxi-speed entry at line 495 has inconsistent percent-escaping compared to the
corrected version at line 81. Find the duplicate msgstr entry for "Speed is 10%
slower than that of the particular vehicles." at line 495 and change `10%` to
`10%%` to match the escaping fix applied at line 81, ensuring consistent
python-format handling across both entries.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants