Skip to content

Conversation

@weblate
Copy link
Contributor

@weblate weblate commented Sep 26, 2025

Translations update from Hosted Weblate for Spook/Inverse helper.

Current translation status:

Weblate translation status

@coderabbitai
Copy link

coderabbitai bot commented Sep 26, 2025

📝 Walkthrough

Walkthrough

Updated translation JSONs for spook_inverse: Finnish, Norwegian (nb_NO), Slovak, Danish, added Ukrainian, and a new empty Vietnamese file. Changes are limited to string content, reformatting, and minor field-ordering — no code, API, or data-structure changes.

Changes

Cohort / File(s) Summary of Changes
Finnish translations
custom_components/spook/integrations/spook_inverse/translations/fi.json
Hyphenation and phrasing updates: "Lähde entiteetti" → "Lähde‑entiteetti", "Piilota lähde entiteetti" → "Piilota lähde‑entiteetti"; user description wording adjusted.
Norwegian (nb_NO) translations
custom_components/spook/integrations/spook_inverse/translations/nb_NO.json
Compound noun standardization: "Kilde entitet" → "Kildeentitet", "Gjem kilde entitet" → "Gjem kildeentitet"; minor description wording tweak.
Slovak translations (formatting)
custom_components/spook/integrations/spook_inverse/translations/sk.json
Reindentation and reordering of keys/blocks; field order changes in some option steps; semantic strings preserved.
Danish translations
custom_components/spook/integrations/spook_inverse/translations/da.json
Wording corrections: replaced occurrences of "inverter"/"Inverter" with "invertér"/"Invertér" across titles, descriptions, and menu options.
Ukrainian translations (new)
custom_components/spook/integrations/spook_inverse/translations/uk.json
Added full Ukrainian translation payload: top-level config, options, user, menu_options, and title entries for binary_sensor and switch.
Vietnamese translations (new/empty)
custom_components/spook/integrations/spook_inverse/translations/vi.json
Added an empty JSON object {} (placeholder for future translations).

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

Suggested labels

maintenance

Suggested reviewers

  • frenck

Poem

A rabbit hops through JSON nights,
I fix the dashes, tune the lights,
Words aligned, translations bright,
Inverse ghosts sleep soft tonight. 🐇✨

Pre-merge checks

✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately describes the main change: translation updates from Hosted Weblate for the Spook/Inverse helper, which aligns with all modifications across multiple translation JSON files.
Description check ✅ Passed The description is directly related to the changeset, clearly stating that it contains translation updates from Hosted Weblate for the Spook/Inverse helper project with status information.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

📜 Recent review details

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between cec8183 and 53a7c37.

📒 Files selected for processing (6)
  • custom_components/spook/integrations/spook_inverse/translations/da.json
  • custom_components/spook/integrations/spook_inverse/translations/fi.json
  • custom_components/spook/integrations/spook_inverse/translations/nb_NO.json
  • custom_components/spook/integrations/spook_inverse/translations/sk.json
  • custom_components/spook/integrations/spook_inverse/translations/uk.json
  • custom_components/spook/integrations/spook_inverse/translations/vi.json
✅ Files skipped from review due to trivial changes (1)
  • custom_components/spook/integrations/spook_inverse/translations/vi.json
🚧 Files skipped from review as they are similar to previous changes (2)
  • custom_components/spook/integrations/spook_inverse/translations/nb_NO.json
  • custom_components/spook/integrations/spook_inverse/translations/sk.json
🔇 Additional comments (3)
custom_components/spook/integrations/spook_inverse/translations/da.json (1)

1-51: LGTM! Danish translation corrections look good.

The terminology updates are consistent throughout the file, correcting the verb form from "Inverter" to "Invertér" (imperative form) and updating the description text. The JSON structure is valid and matches the expected Home Assistant translation file format.

custom_components/spook/integrations/spook_inverse/translations/fi.json (1)

1-51: LGTM! Finnish translation improvements look good.

The hyphenation corrections for compound words ("Lähde-entiteetti" instead of "Lähde entiteetti") follow proper Finnish grammar conventions. The updated description text provides better clarity. The JSON structure is valid and consistent with the expected format.

custom_components/spook/integrations/spook_inverse/translations/uk.json (1)

1-51: LGTM! Complete Ukrainian translation added successfully.

This is a comprehensive Ukrainian translation file that replaces the previous empty file. The JSON structure is valid, contains all necessary sections (config, options, user steps, titles), and properly encodes Cyrillic characters. The structure is consistent with other translation files in the integration.


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.

Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between e3e8602 and ceabcbe.

📒 Files selected for processing (3)
  • custom_components/spook/integrations/spook_inverse/translations/fi.json (3 hunks)
  • custom_components/spook/integrations/spook_inverse/translations/nb_NO.json (2 hunks)
  • custom_components/spook/integrations/spook_inverse/translations/sk.json (1 hunks)
🔇 Additional comments (2)
custom_components/spook/integrations/spook_inverse/translations/nb_NO.json (1)

7-22: LGTM on the Norwegian updates

Vocabulary now consistently uses “Kildeentitet”, and the helper description reads naturally. No further action needed.

custom_components/spook/integrations/spook_inverse/translations/fi.json (1)

6-44: Finnish wording looks good

Hyphenation (“Lähde-entiteetti”) and the refreshed description align with the English source and stay idiomatic.

Comment on lines +21 to +24
"menu_options": {
"switch": "Inverse prepínač",
"binary_sensor": "Inverse a binárne senzor"
},
Copy link

Choose a reason for hiding this comment

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

⚠️ Potential issue | 🟠 Major

Fix the Slovak binary sensor label

The string now reads “Inverse a binárne senzor”, which mixes the English article “a” with Slovak grammar and degrades clarity. Please drop the stray article (e.g., “Inverse binárny senzor”) so the label is correct for Slovak speakers.

-                    "binary_sensor": "Inverse a binárne senzor"
+                    "binary_sensor": "Inverse binárny senzor"
📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
"menu_options": {
"switch": "Inverse prepínač",
"binary_sensor": "Inverse a binárne senzor"
},
"menu_options": {
"switch": "Inverse prepínač",
"binary_sensor": "Inverse binárny senzor"
},
🤖 Prompt for AI Agents
In custom_components/spook/integrations/spook_inverse/translations/sk.json
around lines 21 to 24, the Slovak label "Inverse a binárne senzor" wrongly
includes the English article "a" and incorrect adjective/noun agreement; replace
that value with "Inverse binárny senzor" so the article is removed and the
adjective agrees with the masculine noun, preserving the existing JSON structure
and punctuation.

@weblate weblate force-pushed the weblate-spook-inverse-helper branch from ceabcbe to 3564b47 Compare October 1, 2025 21:02
@sonarqubecloud
Copy link

sonarqubecloud bot commented Oct 1, 2025

@weblate weblate force-pushed the weblate-spook-inverse-helper branch from 3564b47 to cec8183 Compare December 14, 2025 18:00
Milan Šalka and others added 6 commits December 30, 2025 09:03
Currently translated at 100.0% (21 of 21 strings)

Translation: Spook/Inverse helper
Translate-URL: https://hosted.weblate.org/projects/spook/inverse-helper/sk/
Currently translated at 100.0% (21 of 21 strings)

Translation: Spook/Inverse helper
Translate-URL: https://hosted.weblate.org/projects/spook/inverse-helper/nb_NO/
Currently translated at 100.0% (21 of 21 strings)

Translation: Spook/Inverse helper
Translate-URL: https://hosted.weblate.org/projects/spook/inverse-helper/fi/
Currently translated at 100.0% (21 of 21 strings)

Translation: Spook/Inverse helper
Translate-URL: https://hosted.weblate.org/projects/spook/inverse-helper/da/
Currently translated at 100.0% (21 of 21 strings)

Translation: Spook/Inverse helper
Translate-URL: https://hosted.weblate.org/projects/spook/inverse-helper/uk/
@weblate weblate force-pushed the weblate-spook-inverse-helper branch from cec8183 to 53a7c37 Compare December 30, 2025 09:03
@sonarqubecloud
Copy link

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.

6 participants