Skip to content

fix: 修复基于 commit SHA 的分支过滤,修复各分支检查更新全部报错#65

Merged
1 commit merged into
developfrom
fix/check-update-all-branches
May 13, 2026
Merged

fix: 修复基于 commit SHA 的分支过滤,修复各分支检查更新全部报错#65
1 commit merged into
developfrom
fix/check-update-all-branches

Conversation

@Colorful-glassblock
Copy link
Copy Markdown
Owner

Problem

githubApi.getReleases() 将 targetCommitish 与字符串 CURRENT_BRANCH(= develop) 直接比较,但 GH Release targetCommitish 实际存储的是 commit SHA,恒不匹配,导致 filter 永远返回空,各分支检查更新全部失败。

- getReleases() 原逻辑将 targetCommitish 与字符串 "develop" 直接比较,
  targetCommitish 实际存储的是 commit SHA,恒不匹配,导致 filter 永远返回空
- 新增 getAllReleases() 负责纯 API 拉取,getBranchHeadSha() 解析分支 HEAD SHA
- getReleases() 改用与 getBranchHeadSha(CURRENT_BRANCH) 的比较结果过滤
- 分支 HEAD SHA 解析失败时降级(不分支过滤),避免网络异常直接炸
- getLatestPreReleaseBuild() 加 page <= 10 上限防止极端情况无限循环
Copy link
Copy Markdown
Owner Author

@Colorful-glassblock Colorful-glassblock left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@Colorful-glassblock Colorful-glassblock closed this pull request by merging all changes into develop in 6399a9d May 13, 2026
@Colorful-glassblock Colorful-glassblock deleted the fix/check-update-all-branches branch May 13, 2026 12:34
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