Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: Bump version to 6.5.3 #404

Merged
merged 1 commit into from
Mar 20, 2025

Conversation

rb-union
Copy link

@rb-union rb-union commented Mar 20, 2025

Bump version to 6.5.3

Log: Bump version to 6.5.3

Copy link

github-actions bot commented Mar 20, 2025

CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅

@rb-union rb-union changed the base branch from master to develop/snipe March 20, 2025 11:02
Copy link

  • 检测到debian目录文件有变更: debian/control,debian/rules

Copy link

TAG Bot

TAG: 6.5.3
EXISTED: no
DISTRIBUTION: unstable

Bump version to 6.5.3

Log: Bump version to 6.5.3
@deepin-ci-robot
Copy link

deepin pr auto review

这个提交的改动主要是更新了deepin-terminal的版本号,并添加了一个简短的描述。从代码审查的角度来看,这个提交没有实质性的代码改动,因此主要关注点在于提交信息(changelog)的规范性。

提交信息审查

  1. 版本号更新:版本号从6.5.2更新到6.5.3,这是一个合理的版本号更新,表示软件进行了小版本更新。
  2. 描述简洁性:提交信息中包含了refactor: upgrade Qt and DTK compatibility (#402),这个描述简洁明了,清晰地说明了这次提交的改动内容。
  3. 引用问题编号:提交信息中包含了问题编号#402,这是一个良好的实践,有助于追踪和关联问题。

改进建议

  1. 提交信息格式:虽然提交信息格式基本正确,但可以进一步优化,使其更加清晰和一致。例如,可以添加更多的细节,如具体改动的描述或相关链接。
  2. 版本号命名规范:确保版本号遵循项目中的版本命名规范,如果使用语义化版本控制,应确保版本号的更新符合语义化版本控制的规则。

代码审查

由于这个提交没有实际的代码改动,因此不需要进行代码审查。如果有相关的代码改动,请提供具体的代码内容,以便进行详细的代码审查。

Copy link

  • 敏感词检查失败, 检测到3个文件存在敏感词
详情
{
    "src/views/termwidget.cpp": [
        {
            "line": "    QString strUrl = \"https://cn.bing.com/search?q=\" + selectedText();",
            "line_number": 611,
            "rule": "S35",
            "reason": "Url link | fd576f862d"
        },
        {
            "line": "    QString strUrl = \"https://www.baidu.com/s?wd=\" + selectedText();",
            "line_number": 617,
            "rule": "S35",
            "reason": "Url link | 6f8d292d20"
        },
        {
            "line": "    QString strUrl = \"https://github.com/search?q=\" + selectedText();",
            "line_number": 623,
            "rule": "S35",
            "reason": "Url link | e1ba2d7c70"
        },
        {
            "line": "    QString strUrl = \"https://stackoverflow.com/search?q=\" + selectedText();",
            "line_number": 629,
            "rule": "S35",
            "reason": "Url link | bd659725f2"
        }
    ],
    "debian/control": [
        {
            "line": "Homepage: https://www.deepin.com/",
            "line_number": 26,
            "rule": "S35",
            "reason": "Url link | 9095b9e112"
        }
    ],
    "3rdparty/terminalwidget/lib/TerminalDisplay.cpp": [
        {
            "line": "                                                \"<a href=\\"http://en.wikipedia.org/wiki/Flow_control\\">suspended</a>\"",
            "line_number": 3615,
            "rule": "S35",
            "reason": "Url link | 4c5b21a1de"
        }
    ]
}

@deepin-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: lzwind, rb-union

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

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

@rb-union rb-union changed the title Bump version Bump version to 6.5.3 Mar 20, 2025
@rb-union
Copy link
Author

/merge

@deepin-bot deepin-bot bot merged commit 0f7d369 into linuxdeepin:develop/snipe Mar 20, 2025
16 checks passed
@rb-union rb-union changed the title Bump version to 6.5.3 chore: Bump version to 6.5.3 Mar 20, 2025
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