Skip to content

chore: bump version to 0.5.7 - #60

Merged
zccrs merged 1 commit into
linuxdeepin:masterfrom
wineee:tag
May 22, 2026
Merged

chore: bump version to 0.5.7#60
zccrs merged 1 commit into
linuxdeepin:masterfrom
wineee:tag

Conversation

@wineee

@wineee wineee commented May 22, 2026

Copy link
Copy Markdown
Member

No description provided.

@github-actions

Copy link
Copy Markdown

TAG Bot

TAG: 0.5.7
EXISTED: no
DISTRIBUTION: unstable

@deepin-ci-robot

Copy link
Copy Markdown

deepin pr auto review

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

本次提交主要是一个项目版本发布相关的变更,将项目 TreelandProtocols 的版本从 0.5.6 升级到了 0.5.7,并同步更新了 Debian 打包的 changelog 文件。

由于本次变更仅涉及版本号和更新日志,不包含具体的业务逻辑或可执行代码,因此不存在语法逻辑错误、代码性能瓶颈或代码安全漏洞。不过,从代码规范和项目维护的角度,我仍有以下几点改进建议:

1. 代码规范与一致性

  • Debian Changelog 日期格式:Debian 的 changelog 对日期格式有极其严格的要求,必须遵循 RFC 2822 格式(如:Fri, 22 May 2026 09:43:40 +0800)。你提供的日期格式看起来是正确的,但请确保该日期是使用 dchdebchange 工具自动生成的,或者是通过 date -R 命令获取的,以避免因星期数与年月日不匹配而导致后续 Debian 打包构建失败。
  • CMake 版本要求cmake_minimum_required(VERSION 3.25.0) 版本要求较高。如果本次升级的协议(如 foreign-toplevel 的 attention state)并没有使用到 CMake 3.25 的新特性(如 CMakePresets 的高级用法等),建议评估是否可以降低 CMake 最低版本要求(例如降至 3.16 或 3.20),以增加项目在较旧发行版上的兼容性。

2. 代码质量与工程实践

  • 语义化版本控制:Changelog 中提到 add attention state,这属于新增 API/功能,按照语义化版本控制(SemVer)规范,通常应该升级次版本号(Minor Version,即 0.x 中的 x)。从 0.5.6 升级到 0.5.7 是符合规范的(在 0.y.z 阶段,次版本号变更代表新增功能或不向后兼容的变更)。请确认新增的 attention state 是否向后兼容,如果破坏了旧版协议的兼容性,当前版本号变更也是合理的。
  • Wayland 协议 XML 文件更新:虽然 Diff 中没有显示,但根据 Changelog 记录,你必定修改了相关的 .xml 协议文件。请确保:
    • XML 文件头部的 version 属性已经相应递增。
    • 新增的 attention state 是否需要新增相应的枚举或请求/事件,确保 Wayland 协议的完整性。
    • 文档说明是否已经补充在 XML 的 <description> 标签中。

3. 代码安全与性能

  • 本次变更不涉及可执行代码,因此无安全与性能风险。但作为 Wayland Compositor 的协议层,后续在实现 foreign-toplevelattention state 时,需注意服务端(Compositor)对客户端请求的校验(如 Changelog 提到的 clarify virtual output validation),防止恶意客户端通过频繁发送 attention 请求导致 UI 界面闪烁或拒绝服务。

总结

这是一个标准的版本发布提交,结构清晰,符合 Debian 和 CMake 项目的基本规范。无需强制修改,只需确保上述提到的 Debian 日期格式和协议 XML 版本号的同步即可。如果你有具体的 C++ 实现代码或 Wayland XML 协议文件的 Diff,欢迎进一步提供,我可以为你做更深入的代码审查!

@wineee wineee changed the title chore: bump version to 0.5.7 chore: bump version to 0.8.7 May 22, 2026
@wineee wineee changed the title chore: bump version to 0.8.7 chore: bump version to 0.5.7 May 22, 2026
@deepin-ci-robot

Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: wineee, zccrs

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

@zccrs
zccrs merged commit 00ec43f into linuxdeepin:master May 22, 2026
9 of 10 checks passed
@deepin-bot

deepin-bot Bot commented May 22, 2026

Copy link
Copy Markdown

TAG Bot

Tag created successfully

📋 Tag Details
  • Tag Name: 0.5.7
  • Tag SHA: f2d0a23913574110770487fe6a92a18ab6cfcd7e
  • Commit SHA: 05d6b8348e350e45f64f5aca298d90dc1c7421ba
  • Tag Message:
    Release treeland-protocols 0.5.7
    
    
  • Tagger:
    • Name: wineee
  • Distribution: unstable

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