Skip to content

emrg: make-installer Linux 注释/echo 反映 R116 AppImage 收集(rant #13 收尾) - #392

Merged
argszero merged 1 commit into
masterfrom
docs-make-installer-r116
Aug 4, 2026
Merged

emrg: make-installer Linux 注释/echo 反映 R116 AppImage 收集(rant #13 收尾)#392
argszero merged 1 commit into
masterfrom
docs-make-installer-r116

Conversation

@argszero

Copy link
Copy Markdown
Owner

R116(PR #391)后 Linux 分支同时产出 tar.gz + AppImage,但第 10 行注释与第 117 行 echo 仍描述旧行为("AppImage built by electron-builder in CI"),日志/文档与实际不符。本 PR 修正注释与 echo 反映收集动作。

验证:bash -n OK;pytest 464 passed;import + CLI OK。

R116(PR #391)后 Linux 分支同时产出 tar.gz + AppImage,但注释与 echo
仍停留在"AppImage built by electron-builder in CI"的旧描述——日志与
文档与实际行为不一致。修正为反映收集动作(collected R116),方便
CI 排障时快速确认 AppImage 已被脚本收集到 artifacts。

@argszeroargszero left a comment

Copy link
Copy Markdown
OwnerAuthor

Choose a reason for hiding this comment

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

✅ LGTM — cycle 20260805-0103

纯文档/日志修正(+4/-2):Linux 分支注释 + echo 反映 R116 收集行为,与实际一致,
避免 CI 排障时误判 AppImage 缺失。bash -n OK;pytest 464 passed。

@argszeroargszero left a comment

Copy link
Copy Markdown
OwnerAuthor

Choose a reason for hiding this comment

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

✅ LGTM — cycle 20260805-0104

第二次复核:diff 稳定(+4/-2,注释/echo 反映 R116 收集行为)。纯文档修正,
bash -n OK;pytest 464 passed(cycle 0103 验证)。2 个连续 LGTM(0103/0104),无 ❌。

@argszeroargszero left a comment

Copy link
Copy Markdown
OwnerAuthor

Choose a reason for hiding this comment

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

✅ LGTM — cycle 20260805-0105

第三次复核:diff 稳定(+5/-3 纯注释修正)。3 个连续 LGTM(0103/0104/0105),中间无 ❌,满足合并条件。

@argszero
argszero merged commit 7bc7be5 into masterAug 4, 2026
1 check passed
@argszero
argszero deleted the docs-make-installer-r116 branch August 5, 2026 06:18
Sign up for freeto 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

@argszero