Conversation
There was a problem hiding this comment.
Pull request overview
This PR updates translation files for the FreeCAD Addon Manager. The main purpose is to synchronize translation metadata with source code changes and add translator context comments.
Changes:
- Updates line number references from +281 to +282 in five language translation files (Swedish, Polish, French, German, and Belarusian)
- Adds extracomment elements to six translation entries in the base translation file (AddonManager.ts)
Reviewed changes
Copilot reviewed 6 out of 6 changed files in this pull request and generated 6 comments.
Show a summary per file
| File | Description |
|---|---|
| Resources/translations/AddonManager_sv.ts | Updates line reference from +281 to +282 for Swedish translation |
| Resources/translations/AddonManager_pl.ts | Updates line reference from +281 to +282 for Polish translation |
| Resources/translations/AddonManager_fr.ts | Updates line reference from +281 to +282 for French translation |
| Resources/translations/AddonManager_de.ts | Updates line reference from +281 to +282 for German translation |
| Resources/translations/AddonManager_be.ts | Updates line reference from +281 to +282 for Belarusian translation |
| Resources/translations/AddonManager.ts | Adds extracomment elements to six translation entries, though these contain incorrect/unrelated content |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| <location filename="../../Widgets/addonmanager_widget_package_details_view.py" line="+33"/> | ||
| <location line="+8"/> | ||
| <source>Update available</source> | ||
| <extracomment>the descriptive information name Map of parameter names to their parsed values. Parameters with ``expose_value=False`` are not stored.</extracomment> |
There was a problem hiding this comment.
The extracomment contains unrelated text about "Map of parameter names to their parsed values. Parameters with expose_value=False are not stored." This appears to be documentation from an unrelated Python library (possibly Click) and has no relevance to the "Update available" translation string in the FreeCAD Addon Manager. The extracomment should either be removed or replaced with a relevant comment that helps translators understand the context of "Update available" in the Addon Manager interface.
| <extracomment>the descriptive information name Map of parameter names to their parsed values. Parameters with ``expose_value=False`` are not stored.</extracomment> | |
| <extracomment>Status label shown when a newer version of the selected addon is available to install or update in the Addon Manager.</extracomment> |
| <message> | ||
| <location line="+7"/> | ||
| <source>Git tag '{}' checked out, no updates possible</source> | ||
| <extracomment>The formatter class to create with :meth:`make_formatter`.</extracomment> |
There was a problem hiding this comment.
The extracomment contains unrelated text about "The formatter class to create with :meth:make_formatter." This appears to be documentation from an unrelated Python library and has no relevance to the "Git tag '{}' checked out, no updates possible" translation string. The extracomment should either be removed or replaced with a relevant comment that helps translators understand the context of this Git-related message in the Addon Manager.
| <message> | ||
| <location line="+10"/> | ||
| <source>Currently on branch {}, name changed to {}</source> | ||
| <extracomment>.. versionadded:: 8.0</extracomment> |
There was a problem hiding this comment.
The extracomment contains ".. versionadded:: 8.0" which appears to be Sphinx/reStructuredText documentation markup from an unrelated Python library. This has no relevance to the "Currently on branch {}, name changed to {}" translation string. The extracomment should either be removed or replaced with a relevant comment that helps translators understand the context of this branch change message.
| <message> | ||
| <location line="+6"/> | ||
| <source>Update available to version {}</source> | ||
| <extracomment>the parent context or `None` if none exists. the :class:`Command` for this context.</extracomment> |
There was a problem hiding this comment.
The extracomment contains unrelated text about "the parent context or None if none exists. the :class:Command for this context." This appears to be documentation from an unrelated Python library (possibly Click) and has no relevance to the "Update available to version {}" translation string. The extracomment should either be removed or replaced with a relevant comment that helps translators understand the context of this update notification message.
| <message> | ||
| <location line="+16"/> | ||
| <source>WARNING: This addon is currently installed, but disabled. Use the 'enable' button to re-enable.</source> | ||
| <extracomment>the leftover arguments. protected arguments. These are arguments that are prepended to `args` when certain parsing scenarios are encountered but must be never propagated to another arguments. This is used to implement nested parsing. the collected prefixes of the command's options.</extracomment> |
There was a problem hiding this comment.
The extracomment contains extensive unrelated text about "the leftover arguments. protected arguments. These are arguments that are prepended to args when certain parsing scenarios are encountered..." This appears to be documentation from an unrelated Python library and has no relevance to the addon warning message. The extracomment should either be removed or replaced with a relevant comment that helps translators understand the context of this warning about disabled addons.
| <extracomment>the leftover arguments. protected arguments. These are arguments that are prepended to `args` when certain parsing scenarios are encountered but must be never propagated to another arguments. This is used to implement nested parsing. the collected prefixes of the command's options.</extracomment> | |
| <extracomment>Shown in the addon details view when an addon is installed but currently disabled. The word 'enable' refers to the label of a button in the user interface.</extracomment> |
| <message> | ||
| <location line="+1"/> | ||
| <source>WARNING: This addon requires FreeCAD {}</source> | ||
| <extracomment>the user object stored.</extracomment> |
There was a problem hiding this comment.
The extracomment contains unrelated text about "the user object stored." This appears to be documentation from an unrelated Python library and has no relevance to the "WARNING: This addon requires FreeCAD {}" translation string. The extracomment should either be removed or replaced with a relevant comment that helps translators understand the context of this FreeCAD version requirement warning.
| <extracomment>the user object stored.</extracomment> | |
| <extracomment>Warning shown when an addon requires a specific FreeCAD version. {} is the required FreeCAD version.</extracomment> |
1 participant
No description provided.