Skip to content

chore(ostool): release v0.25.0#155

Closed
github-actions[bot] wants to merge 1 commit into
mainfrom
release-plz-2026-07-22T01-15-46Z
Closed

chore(ostool): release v0.25.0#155
github-actions[bot] wants to merge 1 commit into
mainfrom
release-plz-2026-07-22T01-15-46Z

Conversation

@github-actions

@github-actions github-actions Bot commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

🤖 New release

  • ostool: 0.24.1 -> 0.25.0 (⚠ API breaking changes)

ostool breaking changes

--- failure constructible_struct_adds_field: externally-constructible struct adds field ---

Description:
A pub struct constructible with a struct literal has a new pub field. Existing struct literals must be updated to include the new field.
        ref: https://doc.rust-lang.org/reference/expressions/struct-expr.html
       impl: https://github.com/obi1kenobi/cargo-semver-checks/tree/v0.48.0/src/lints/constructible_struct_adds_field.ron

Failed in:
  field BoardGlobalConfig.server in /tmp/.tmpdxGyM5/ostool/ostool/src/board/global_config.rs:97
  field BoardGlobalConfig.auth_mode in /tmp/.tmpdxGyM5/ostool/ostool/src/board/global_config.rs:102
  field BoardRunConfig.auth_mode in /tmp/.tmpdxGyM5/ostool/ostool/src/board/config.rs:30

--- failure function_parameter_count_changed: pub fn parameter count changed ---

Description:
A publicly-visible function now takes a different number of parameters.
        ref: https://doc.rust-lang.org/cargo/reference/semver.html#fn-change-arity
       impl: https://github.com/obi1kenobi/cargo-semver-checks/tree/v0.48.0/src/lints/function_parameter_count_changed.ron

Failed in:
  ostool::board::terminal::run_serial_terminal now takes 2 parameters instead of 1, in /tmp/.tmpdxGyM5/ostool/ostool/src/board/terminal.rs:54
  ostool::board::serial_stream::connect_serial_stream now takes 2 parameters instead of 1, in /tmp/.tmpdxGyM5/ostool/ostool/src/board/serial_stream.rs:34

--- failure inherent_method_missing: pub method removed or renamed ---

Description:
A publicly-visible method or associated fn is no longer available under its prior name. It may have been renamed or removed entirely.
        ref: https://doc.rust-lang.org/cargo/reference/semver.html#item-remove
       impl: https://github.com/obi1kenobi/cargo-semver-checks/tree/v0.48.0/src/lints/inherent_method_missing.ron

Failed in:
  LoadedBoardGlobalConfig::resolve_server, previously in file /tmp/.tmpVhtyCD/ostool/src/board/global_config.rs:82
  BoardGlobalConfig::resolve_server, previously in file /tmp/.tmpVhtyCD/ostool/src/board/global_config.rs:88

--- failure pub_module_level_const_missing: pub module-level const is missing ---

Description:
A public const is missing or renamed
        ref: https://doc.rust-lang.org/cargo/reference/semver.html#item-remove
       impl: https://github.com/obi1kenobi/cargo-semver-checks/tree/v0.48.0/src/lints/pub_module_level_const_missing.ron

Failed in:
  DEFAULT_BOARD_SERVER_IP in file /tmp/.tmpVhtyCD/ostool/src/board/global_config.rs:9
  DEFAULT_BOARD_SERVER_PORT in file /tmp/.tmpVhtyCD/ostool/src/board/global_config.rs:10

--- failure struct_pub_field_missing: pub struct's pub field removed or renamed ---

Description:
A publicly-visible struct has at least one public field that is no longer available under its prior name. It may have been renamed or removed entirely.
        ref: https://doc.rust-lang.org/cargo/reference/semver.html#item-remove
       impl: https://github.com/obi1kenobi/cargo-semver-checks/tree/v0.48.0/src/lints/struct_pub_field_missing.ron

Failed in:
  field net of struct UbootConfig, previously in file /tmp/.tmpVhtyCD/ostool/src/run/uboot.rs:76
  field server_ip of struct BoardGlobalConfig, previously in file /tmp/.tmpVhtyCD/ostool/src/board/global_config.rs:21
Changelog

0.25.0 - 2026-07-23

Fixed

  • (ostool) harden local U-Boot TFTP staging lifecycle (#156)

Other

  • Add board authentication client and unify server config (#154)


This PR was generated with release-plz.

@mai-team-app mai-team-app 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.

审查结论:APPROVE

这是一个由 release-plz 自动生成的发版 PR,将 ostool0.24.1 升级到 0.25.0

变更内容

仅修改 3 个文件(+8/-2):

  • ostool/Cargo.tomlversion 0.24.1 → 0.25.0
  • Cargo.lockostool 版本 0.24.1 → 0.25.0
  • ostool/CHANGELOG.md:新增 ## [0.25.0] 条目,引用 PR #154(Add board authentication client and unify server config)

不含任何代码改动,属于纯版本元数据与 changelog 更新。

一致性与 semver 核对

  • 版本号三处一致:Cargo.toml = Cargo.lock = CHANGELOG.md = 0.25.0
  • ostool 为二进制 crate(含 [[bin]]),无其它 workspace crate 依赖它,不存在遗漏的升级点 ✅
  • 未发现残留的 0.24.1 引用(CHANGELOG 中的 0.24.1 仅出现在比较链接与既有条目标题,符合预期)✅
  • PR #154 引入了破坏性 API 变更(run_serial_terminal / connect_serial_stream 参数数量变化、移除 resolve_server、移除 DEFAULT_BOARD_SERVER_IP/PORT、移除 server_ip 字段)。对 0.x 版本 crate,破坏性变更应触发 minor 版本升级,0.24.1 → 0.25.0 符合 semver 约定 ✅
  • CHANGELOG 条目格式(Keep a Changelog、ostool-vX.Y.Z 比较链接)与既有条目一致 ✅

验证状态

  • CI:当前无 check-run、无既有 review(新开的发版 PR)。
  • 本容器未安装 cargo,因此 cargo fmt --check / clippy / build / test 未能在本地运行。由于本 PR 不含任何代码改动,这些检查的结果不会因本 PR 而变化;版本元数据与 lock 一致性已逐项手动核实。

可选改进(非阻塞)

  • CHANGELOG 使用 ### Other 分类而非 ### Changed/### Added,且未显式说明破坏性 API 变更。这是 release-plz 的默认分类行为,不影响发布;如需更利于下游用户,可在 0.25.0 条目下补充破坏性变更说明。

相似 PR

未发现重复的发版 PR。

无阻塞项,建议合并。

Powered by glm-5.2

@mai-team-app mai-team-app 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.

审查结论:APPROVE

这是一个由 release-plz 自动生成的发版 PR(提交 chore(ostool): release v0.25.0),将 ostool0.24.1 升级到 0.25.0

变更内容

仅修改 3 个文件(+8/-2),无任何源码改动:

  • ostool/Cargo.tomlversion 0.24.1 → 0.25.0
  • Cargo.lockostool 版本 0.24.1 → 0.25.0(diff 仅此一行)
  • ostool/CHANGELOG.md:新增 ## [0.25.0] 条目,引用 PR #154(Add board authentication client and unify server config)

一致性与 semver 核对

  • 版本号三处一致:Cargo.toml = Cargo.lock = CHANGELOG.md = 0.25.0
  • Cargo.lock diff 确认仅修改 ostoolversion 行,依赖列表无任何附带改动 ✅
  • 未发现残留的 0.24.1 版本引用(CHANGELOG 中的 0.24.1 仅出现在既有条目标题与比较链接中,符合预期)✅
  • PR #154 引入了破坏性 API 变更(PR 正文记录了 cargo-semver-checks 的告警:run_serial_terminal/connect_serial_stream 参数数量变化、移除 resolve_server、移除 DEFAULT_BOARD_SERVER_IP/PORT、移除 server_ip 字段)。对 0.x 版本 crate,破坏性变更应触发 minor 版本升级,0.24.1 → 0.25.0 符合 SemVer 约定 ✅
  • CHANGELOG 条目格式(Keep a Changelog、ostool-vX.Y.Z 比较链接、release-plz 默认分类)与既有条目一致 ✅

验证状态

  • 本容器未安装 Rust 工具链(cargo/rustc 不可用),因此 cargo fmt --check / clippy / build / test / cargo publish --dry-run --locked 未能在本地运行。由于本 PR 不含任何代码改动,这些检查结果不会因本 PR 而变化;版本元数据与 lock 一致性已逐项手动核实。
  • CI check-run 当前为空(新开的发版 PR)。

可选改进(非阻塞)

  • CHANGELOG 将 #154 的变更归入 ### Other 而非显式标注破坏性 API 变更。这与 release-plz 在本仓库历史条目中的默认分类行为一致,不影响发布;若希望更利于下游用户,可在 0.25.0 条目下补充破坏性变更说明。

既有审查

此前已有一份 mai-team-app[bot] 的 APPROVE 审查,结论与本审查一致;未发现需要追加的阻塞项。

无阻塞项,版本元数据一致、semver 升级合理,建议合并。

Powered by glm-5.2

@ZR233 ZR233 closed this Jul 23, 2026

@mai-team-app mai-team-app 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.

审查结论:APPROVE

本 PR 是 release-plz 生成的 ostool 0.25.0 发版变更:将 ostool/Cargo.tomlCargo.lock 的版本从 0.24.1 同步为 0.25.0,并新增对应的 CHANGELOG 条目(#154 的 board 认证与服务端配置统一)。未修改运行时代码、依赖解析或公开配置;对现有功能的影响限于发布元数据和下游获取到的新版本,变更看起来隔离。

复核结果

  • 已确认 manifest、lockfile 与首个已发布 changelog 条目均为 0.25.0,且 lockfile 仅修改 ostool 包版本行。
  • git diff --check origin/main...HEAD 通过;审查 helper 自测与 prepare-review 通过,且当前工作区 HEAD 仍为 574a93fdb82531fcb7ba7a551956f0fec0a3c2f8
  • helper 将变更识别为 ostool crate;计划的 cargo fmt --check、聚焦 clippy(-D warnings)和测试均因环境没有 cargo/rustc(命令返回 127)而未运行。该限制已明确记录;本 PR 不含源码改动。
  • 此提交当前没有 GitHub check-run 或 commit status;因此没有可归因于本 PR 的 CI 失败。

历史审查与风险

  • PR #155 没有行内审查或 issue 评论;此前两份针对同一 head 的 APPROVED 审查与本次结论一致。它们不替代本次独立审查。
  • Changelog 中的 #154 曾有默认端口和旧配置迁移问题,后续审查已确认均已修复;本 PR 只发布该已合入变更,未重新引入相关逻辑。
  • 搜索未发现重叠的 0.25.0 发版 PR;相邻的 #156 是独立的 TFTP 生命周期修复。

没有未解决的阻塞问题或额外测试缺口(除上述受环境限制而未执行的 Rust 检查)。

Powered by gpt-5.6-terra

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.

1 participant