Skip to content

chore: update version to 6.5.35#198

Merged
lzwind merged 1 commit into
linuxdeepin:masterfrom
wyu71:master
May 19, 2026
Merged

chore: update version to 6.5.35#198
lzwind merged 1 commit into
linuxdeepin:masterfrom
wyu71:master

Conversation

@wyu71

@wyu71 wyu71 commented May 19, 2026

Copy link
Copy Markdown
Contributor
  • bump version to 6.5.35

Log : bump version to 6.5.35

- bump version to 6.5.35

Log : bump version to 6.5.35

@sourcery-ai sourcery-ai 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.

Sorry @wyu71, you have reached your weekly rate limit of 500000 diff characters.

Please try again later or upgrade to continue using Sourcery

@github-actions

Copy link
Copy Markdown

TAG Bot

TAG: 6.5.35
EXISTED: no
DISTRIBUTION: unstable

@deepin-ci-robot

Copy link
Copy Markdown
Contributor

deepin pr auto review

你好!我是CodeGeeX。我已经仔细审查了你提供的 Git Diff 内容。

本次提交主要包含两个部分:一是更新了 Debian 的变更日志(debian/changelog),二是更新了玲珑包的版本号(linglong.yaml)。这属于典型的项目发版与元数据更新提交。

针对本次代码变更,我将从语法逻辑、代码质量、代码性能和代码安全四个维度进行审查,并提供改进意见:

1. 语法逻辑

  • Debian Changelog 格式debian/changelog 的格式非常严格。从 Diff 来看,时间戳写的是 Tue, 19 May 2026 21:09:47 +0800
    • 逻辑问题:年份为 2026年,这显然是一个未来的时间。虽然这不会导致编译失败,但在逻辑上是不合理的,可能是修改系统时间或手动编写时的笔误。
    • 改进意见:请确认当前的实际日期,Debian 的打包工具通常要求时间与提交时间或构建时间保持一致。

2. 代码质量

  • 提交信息的内聚性:Changelog 中记录了四项改动,包括俄语翻译、禁止输入中文的修复、desktop 文件自动翻译以及 CI 的 Debian 检查工作流。
    • 质量问题:这些改动跨越了国际化、Bug修复、自动化构建和CI流程,属于相对杂糅的提交。虽然对于发版更新 Changelog 来说这是正常的汇总,但如果在代码开发阶段,建议将 Bug 修复(禁止输入中文)和 CI/Chore 类改动拆分为独立的 Commit,以便于代码审查和后续的 Git Bisect(二分查找定位问题)。
    • 改进意见:如果这只是发版时的 Changelog 汇总,则没有问题;如果是功能开发分支的合并,建议规范 Commit 的原子性。

3. 代码性能

  • 性能评估:本次变更仅涉及文本配置文件的修改,不涉及任何业务逻辑代码的增删,因此对运行时性能没有任何影响,无需进行性能优化。

4. 代码安全

  • 安全评估:版本号和 Changelog 的更新本身不存在安全漏洞。
  • 潜在风险提示:Changelog 中提到了 fix: Calculators prohibit input of Chinese text.(计算器禁止输入中文文本)。虽然本次 Diff 没有包含该修复的具体代码,但作为审查者,需要提醒:在处理用户输入过滤时,请确保使用白名单机制(如仅允许数字、运算符等)而不是黑名单机制。黑名单容易导致绕过漏洞或引发其他不可预期的输入问题。

💡 额外审查建议

由于本次 Diff 只是发版的“冰山一角”,真正的代码改动并未包含在内。针对 Changelog 中提到的核心修复,我建议你补充审查以下相关代码的 Diff:

关于 "fix: Calculators prohibit input of Chinese text." 的代码审查建议:
如果你有这部分代码的改动,请重点关注以下几点:

  1. 输入校验的位置:是在 UI 层(如 QLineEdit/QML TextField 的 validator)进行拦截,还是在底层计算逻辑中过滤?建议在 UI 层直接拦截,避免非法字符进入底层解析引发异常。
  2. 输入法兼容性:禁止中文输入时,是否会影响其他语言(如日文、韩文)的输入法?是否会导致输入法切换时的崩溃?
  3. 正则表达式的性能与安全:如果使用了正则表达式来过滤输入,请检查是否存在 ReDoS(正则表达式拒绝服务)的风险,并确保正则在处理高频输入时不会造成界面卡顿。

总结:本次 Diff 整体符合发版规范,唯一的逻辑疑点是 Changelog 中的 2026年 的时间戳,请务必核实修正。如果需要我对具体的代码逻辑(如禁止中文输入的修复代码)进行深入审查,欢迎提供那部分的 Git Diff!

@deepin-ci-robot

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: lzwind, wyu71

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@lzwind lzwind merged commit 0534db1 into linuxdeepin:master May 19, 2026
21 of 23 checks passed
@deepin-bot

deepin-bot Bot commented May 19, 2026

Copy link
Copy Markdown
Contributor

TAG Bot

Tag created successfully

📋 Tag Details
  • Tag Name: 6.5.35
  • Tag SHA: c525408d1052db5d098afc29d040ced2848a4ed4
  • Commit SHA: c1bbbbe23fbf9cd1524ca03ff9b24ca8bb2dd684
  • Tag Message:
    Release deepin-calculator 6.5.35
    
    
  • Tagger:
    • Name: wyu71
  • Distribution: unstable

@deepin-bot deepin-bot Bot mentioned this pull request May 19, 2026
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.

3 participants