sync: from linuxdeepin/dde-session-shell - #528
Conversation
There was a problem hiding this comment.
Sorry @deepin-ci-robot, you have reached your weekly rate limit of 500000 diff characters.
Please try again later or upgrade to continue using Sourcery
Reviewer's guide (collapsed on small PRs)Reviewer's GuideThis PR syncs a portion of the Portuguese translation file with the upstream linuxdeepin/dde-session-shell project by replacing unfinished entries with complete translations for MFA login-related messages. File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
c209263 to
f5d9b13
Compare
Synchronize source files from linuxdeepin/dde-session-shell. Source-pull-request: linuxdeepin/dde-session-shell#79
f5d9b13 to
b488772
Compare
deepin pr auto review🤖 AI 代码审查报告📊 总体评价
🔍 详细分析1. 语法逻辑 ✅评价: 优秀 ✅ 通过 潜在问题: 建议: XML结构正确,所有translation标签格式完整,numerus复数条目包含正确的6个numerusform子元素(阿拉伯语复数规则)。%n占位符在所有翻译中正确保留。 2. 代码质量 ✅评价: 良好 ✅ 通过 潜在问题:
建议: 修正所有阿拉伯语numerus条目的第3个numerusform(index 2,n=2的双数形式)。涉及4个源字符串共8处以上的翻译条目:1."Verification failed, %n chances left" - 第3形式改为"فشل التحقق، %n فرصتان متبقيتان";2."Please try again %n minutes later" - 第3形式改为"يرجى المحاولة مرة أخرى بعد %n دقيقتان";3."Please try again %n minute(s) later" - 第3形式改为"يرجى المحاولة مرة أخرى بعد %n دقيقتان";4."Your password will expire in %n days" - 第3形式改为"ستنتهي صلاحية كلمة المرور الخاصة بك خلال %n يومان، يرجى تغييرها في الوقت المناسب"。 3. 代码性能 ✅评价: 优秀 ✅ 通过 潜在问题: 建议: 翻译文件为静态XML数据文件,在构建时编译,无运行时性能影响。无性能优化空间。 4. 代码安全 🔒评价: 优秀 ✅ 通过
安全漏洞详情: 建议: 翻译文件仅包含UI显示文本,无用户输入处理、无代码执行、无注入风险。未发现任何安全漏洞。建议保持翻译内容的审查流程,确保不引入敏感信息。 💡 改进建议代码示例<!-- 修正前(错误:第3形式使用单数) -->
<translation>
<numerusform>فشل التحقق، %n فرصة متبقية</numerusform>
<numerusform>فشل التحقق، %n فرصة متبقية</numerusform>
<numerusform>فشل التحقق، %n فرصة متبقية</numerusform> <!-- 错误:n=2应使用双数 -->
<numerusform>فشل التحقق، %n فرص متبقية</numerusform>
<numerusform>فشل التحقق، %n فرصة متبقية</numerusform>
<numerusform>فشل التحقق، %n فرصة متبقية</numerusform>
</translation>
<!-- 修正后(正确:第3形式使用双数) -->
<translation>
<numerusform>فشل التحقق، %n فرصة متبقية</numerusform>
<numerusform>فشل التحقق، %n فرصة متبقية</numerusform>
<numerusform>فشل التحقق، %n فرصتان متبقيتان</numerusform> <!-- 正确:n=2使用双数 -->
<numerusform>فشل التحقق، %n فرص متبقية</numerusform>
<numerusform>فشل التحقق، %n فرصة متبقية</numerusform>
<numerusform>فشل التحقق، %n فرصة متبقية</numerusform>
</translation>本报告由 AI 代码审查工具自动生成 |
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: BLumia, deepin-ci-robot The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
Synchronize source files from linuxdeepin/dde-session-shell.
Source-pull-request: linuxdeepin/dde-session-shell#79
Summary by Sourcery
Complete missing localization strings in the Arabic and Portuguese translation catalogs.
Enhancements: