author(zhlineskip): Add co-author info by myhsia · Pull Request #912 · CTeX-org/ctex-kit

myhsia · 2026-06-28T13:51:26Z

@RuixiZhang42 已把我添加至 zhlineskip 的 co-author, 我需要在下次上传的 note 中作出说明. 所以在发版前, 有个不确定的问题: l3build 是支持在 uploadconfig 表单中使用 note 这个选项的, 但是在最新的 CI 发版中使用方法我暂时不了解.

From CTAN:

Hi Ruixi, Mingyu,

I have added Mingyu as an allowed uploader for zhlineskip. If Mingyu becomes credited as co-author, we will add this information in the next upload, such that the information in our database matches the code. Please leave a note with the next upload to remind us.

Best regards,

Vincent Goulet
for the CTAN Team

github-actions · 2026-06-28T13:52:46Z

🔍 PR 审查

项目	结果
结论	💬 COMMENT
审查截止	`e462f09b091c2d0168d8ac97400a087c998a4e40`

整体改动清晰，纯元数据变更（co-author 信息），无逻辑代码修改。有 1 个小问题。

🟢 小问题 (1)

文件: zhlineskip/build.lua
问题: email 字段从 ruixizhang42@gmail.com 完全替换为 myhsia@outlook.com，而 maintainer 字段保留了两位维护者 "Ruixi Zhang, Mingyu Xia"。如果 email 仅用于 CTAN 上传联系人，这是合理的；但如果希望两位维护者的联系方式都可见，可考虑同时列出两个邮箱（如 "ruixizhang42@gmail.com, myhsia@outlook.com"）。
建议: 确认 email 字段的用途——若仅作为 CTAN 上传联系邮箱则无需修改，若需保留双方联系方式则补充原始邮箱。

RuixiZhang42 · 2026-06-28T14:00:13Z

email 字段确仅作为 CTAN 上传的联系邮箱，无需修改

…dInfo` 那里就错乱了; 关于我的邮箱: 我希望 `myhsia@outlook.com` 邮箱作为 CTAN 反馈的地方——它只接受比如 GitHub、CTAN 有时系统自动 Announcement 这种很“人机“的信息; 而 `xiamingyu@westlake.edu.cn` 则是接受真人亲自手写的反馈沟通信息, 所以我有意在 `lua` 中分离.

myhsia · 2026-06-28T14:02:39Z

目前 TODO: 确认发布时在 uploadconfig 中注入 note 的正确方法.

github-actions · 2026-06-28T14:03:30Z

🔍 PR 增量审查

项目	结果
结论	⚠️ REQUEST_CHANGES
审查范围	`e462f09b..c4b05a00`

第二次提交回应了上次审查的 email 字段问题，调整了 maintainer 字段和邮箱分离策略（outlook 接机器通知，westlake 接真人反馈）。增量改动中发现 2 个需要关注的问题。

🔴 阻塞问题 (1)

文件: zhlineskip/README.md vs zhlineskip/zhlineskip.dtx
问题: 独立 README.md 中 Mingyu Xia 的邮箱为 myhsia@outlook.com，但 zhlineskip.dtx 的 %<*readme> 段落（即 README 的 docstrip 生成源）已在第二次提交中改为 xiamingyu@westlake.edu.cn。两者不同步——下次 l3build doc 生成后，独立 README.md 会被覆盖为 xiamingyu@westlake.edu.cn，或者如果不重新生成，发行版中会存在两份内容不一致的 README。
建议: 选择其中一个邮箱统一 README.md 和 dtx readme 段落，或者立即从 dtx 重新生成 README.md 使两者一致。

🟠 重要建议 (1)

文件: zhlineskip/build.lua
问题: maintainer 字段从 "Ruixi Zhang, Mingyu Xia" 改为仅 "Mingyu Xia"。理解 l3build tag 的 \GetIdInfo 解析限制（逗号分隔会导致错乱），但 maintainer 字段会直接作为 CTAN 上传表单的 author 元数据。仅列出一位可能导致 CTAN 记录与 license 头部声明（"The Current Maintainer of this work is Ruixi Zhang and Mingyu Xia"）不一致。
建议: 确认 l3build upload 是否也受同样的逗号解析限制。如果 maintainer 仅影响 upload 而非 tag，可以考虑保留两位维护者名称（如用 and 连接："Ruixi Zhang and Mingyu Xia"），仅在 \GetIdInfo 行单独处理。

审查截止: c4b05a0

RuixiZhang42 · 2026-06-28T14:05:03Z

 This work has the LPPL maintenance status `maintained`.

-The Current Maintainer of this work is **Ruixi Zhang**.
+The Current Maintainer of this work is **Ruixi Zhang** and **Mingyu Xia**.


RuixiZhang42 · 2026-06-28T14:05:13Z

 %   This work has the LPPL maintenance status `maintained'.             *
 %                                                                       *
-%   The Current Maintainer of this work is Ruixi Zhang.                 *
+%   The Current Maintainer of this work is Ruixi Zhang and Mingyu Xia.  *


RuixiZhang42 · 2026-06-28T14:05:24Z

 This work has the LPPL maintenance status `maintained`.

-The Current Maintainer of this work is **Ruixi Zhang**.
+The Current Maintainer of this work is **Ruixi Zhang** and **Mingyu Xia**.


RuixiZhang42 · 2026-06-28T14:05:42Z

  This work has the LPPL maintenance status `maintained'.             *
                                                                      *
-  The Current Maintainer of this work is Ruixi Zhang.                 *
+  The Current Maintainer of this work is Ruixi Zhang and Mingyu Xia.  *


github-actions · 2026-06-28T14:14:42Z

🔍 PR 增量审查

项目	结果
结论	💬 COMMENT
审查范围	`c4b05a00..fcc62b35`

上次阻塞问题（README.md 与 dtx 邮箱不一致）已修复，统一为 xiamingyu@westlake.edu.cn。语法 "is" → "are" 已按 review comment 修正。有 1 个小问题。

🟢 小问题 (1)

文件: zhlineskip/README.md、zhlineskip/zhlineskip.dtx（共 4 处）
问题: "The Current Maintainer of this work are" 主谓不一致——主语 "Maintainer" 是单数，动词 "are" 是复数。LPPL 标准措辞在多位维护者时应使用复数形式 "The Current Maintainers"。
建议: 将 4 处 "The Current Maintainer of this work are" 统一改为 "The Current Maintainers of this work are"。

审查截止: fcc62b3

github-actions · 2026-06-28T14:17:11Z

🔍 PR 增量审查

项目	结果
结论	💬 COMMENT
审查范围	`fcc62b35..6de1b6c3`

上次审查建议的 Maintainer → Maintainers 主谓一致问题已在 zhlineskip.dtx 中修复（共 3 处），改动准确。

🟢 小问题 (1)

文件: zhlineskip/README.md
问题: README.md 中仍为 The Current Maintainer of this work are，未同步 dtx 中的 Maintainers 修正。虽然 README.md 理论上由 dtx 的 %<*readme> 段落通过 l3build doc 生成，但当前独立文件与源不一致。
建议: 从 dtx 重新生成 README.md，或手动将 README.md 中的 Maintainer 改为 Maintainers 保持一致。

审查截止: 6de1b6c

Liam0205 · 2026-06-28T14:19:38Z

CTAN 针对同一个 package，如果有新的提交者，那么需要邮件给已知的提交者做确认。这部分需要 @RuixiZhang42 回复邮件确认一下。

CI 发布的流程，我这边来处理一下。大体要改几个东西。一是 uploadconfig 要加上 maintainers 字段，看看怎么做。二是要邀请 @myhsia 加入 ctex-org 并成为 zhlineskip 这个 package 的 reviewer——于是可以自己点发布。

github-actions · 2026-06-28T14:19:57Z

🔍 PR 增量审查

项目	结果
结论	✅ APPROVE
审查范围	`fcc62b35..73766cb0`

上次审查指出的语法问题（"The Current Maintainer of this work are" → "The Current Maintainers of this work are"）已在 zhlineskip/README.md 和 zhlineskip/zhlineskip.dtx 的全部 4 处统一修正，dtx 注释边框对齐也正确保持。增量改动无问题。

审查截止: 73766cb

Liam0205 · 2026-06-28T14:21:14Z

确认的部分不着急。我们先把当前仓库的问题解决掉。

大体上一会儿需要移动 zhlienskip-v1.0f 这个 tag。推送后 CI 会自动 pre-release。

确认无误后，可以手动 run uplaod to ctan （大概是这个名字）这个 CI。然后它就会用 pre-release 里的 ctan zip 去提交发版。

myhsia · 2026-06-28T14:22:02Z

Done.

@Liam0205 可以的, 提交方面只剩下 uploadconfig = { note = ... } 的问题需要配置, 其他没问题了.

Liam0205 · 2026-06-28T14:28:14Z

进度同步：

已通过 GitHub API 向 @myhsia 发送 CTeX-org 组织邀请（role=member），状态 pending。@myhsia 请检查邮箱或 https://github.com/orgs/CTeX-org/invitation 接受邀请。
接受后我会立即把 @myhsia 加入 ctan-release-zhlineskip environment 的 required reviewers（目前只有 @RuixiZhang42 一人）。届时 @myhsia 也可以审批 zhlineskip 的 CTAN 发版。
CTAN 上传 note 字段的注入方式我会另开 PR 处理——把它做成 "upload to ctan" workflow 手动启动时的输入参数。完成后会在这里回链。

@RuixiZhang42 关于 CTAN 邮件那边新增维护者的回邮确认，等仓库这边一切就绪后再处理也可。

myhsia · 2026-06-28T14:29:40Z

workflow

已接受

Liam0205 · 2026-06-28T14:33:01Z

进度更新：

@myhsia 已接受 CTeX-org 组织邀请。
已把 @myhsia 加入 developers team（获得仓库 write 权限）。
已把 @myhsia 加入 ctan-release-zhlineskip environment 的 required reviewers，与 @RuixiZhang42 并列。今后 zhlineskip 的 CTAN 发版任一人可在 GH Actions 上点 Approve 放行。

note 参数注入 workflow 的 PR 我马上推上来，到时回链。

新增 workflow_dispatch input 'note' (可选, ≤4096 字符), 用于在上传 CTAN 时附带内部备注 — 例如 #912 添加 zhlineskip co-author 时需告知 CTAN team 更新数据库. l3build CLI 不暴露 --note (--message 写的是 announcement, 不是 note), 只能通过 uploadconfig.note / note_file. 沿用 uploader/email 的模式从 env 读入 CTAN_NOTE, 由 workflow 用 input 注入. 本地跑 l3build upload 也是同一套. 修改: - support/build-config.lua: ctex_kit_uploadconfig 新增 note 字段, 从 CTAN_NOTE env 读. 配套小工具 ctex_kit_env_or_nil 把空串和 nil 一并视作未设置, 避免 workflow input 留空时把空 note 提交给 CTAN. - .github/workflows/release-ctan-upload.yml: 新增 note input + 注入 CTAN_NOTE env, Summary 显示 note (走 env 而非 \${{ }} 字符串插值, 防 shell 元字符注入).

Liam0205 · 2026-06-28T14:55:54Z

进度更新：

feat(ci): release-ctan-upload 支持 CTAN note 内部备注作为手动 input #913 已 rebase merge 到 master，release-ctan-upload workflow 现已支持手动启动时填写 note 参数：
- 留空则不写 CTAN note 字段
- 填写则写入 uploadconfig.note（≤4096 byte，超长会在 workflow 内 early-fail）
- 已本地用 l3build upload --debug 模式三场景验证（unset / 空串 / 有值）

接下来本 PR 可以合入了。CTAN 实际上传时记得在 note 输入框写：「Mingyu Xia 已成为 co-author，请按 PR #912 描述更新 CTAN 数据库」之类的说明，CTAN reviewer 就会看到。

myhsia · 2026-06-28T15:22:49Z

确认的部分不着急。我们先把当前仓库的问题解决掉。

大体上一会儿需要移动 zhlienskip-v1.0f 这个 tag。推送后 CI 会自动 pre-release。

确认无误后，可以手动 run uplaod to ctan （大概是这个名字）这个 CI。然后它就会用 pre-release 里的 ctan zip 去提交发版。

@Liam0205 目前我可以 upload 但是不能 release tag, 目前 release 里的 zhlineskip 1.0f 的 zip 还是昨天的——虽然可以手动编辑

Liam0205 · 2026-06-28T15:33:41Z

@myhsia 怎么操作的？提示啥？

myhsia · 2026-06-28T15:35:23Z

@myhsia 怎么操作的？提示啥？

无法操作运行 https://github.com/CTeX-org/ctex-kit/actions/workflows/release.yml

Liam0205 · 2026-06-28T15:37:22Z

@myhsia 那个不是手动操作的。你重新打一个 tag，推上去，就会自动运行。

Liam0205 · 2026-06-28T15:46:05Z

@myhsia 用 git 打名为 zhlineskip-v1.0f 的 tag。你可能要搜搜，如何覆盖已有 tag。

myhsia · 2026-06-28T15:50:52Z

@myhsia 用 git 打名为 zhlineskip-v1.0f 的 tag。你可能要搜搜，如何覆盖已有 tag。
OK 了已发布.

Liam0205 · 2026-06-28T15:54:12Z

@myhsia 然后再手动运行 Release CTAN upload

myhsia · 2026-06-28T15:55:30Z

@myhsia 然后再手动运行 Release CTAN upload

已经可以了，Approve后上传倒计时马上结束

Liam0205 · 2026-06-28T15:56:43Z

@myhsia 应该是有五分钟的倒计时，作为「后悔药」。万一有啥问题可以终止它。

myhsia · 2026-06-28T15:58:00Z

@myhsia 应该是有五分钟的倒计时，作为「后悔药」。万一有啥问题可以终止它。

是的刚刚已经结束发上去了

Liam0205 · 2026-06-28T16:01:49Z

@myhsia 希望感觉流程足够丝滑。恭喜发版。记得关注邮箱，祝顺利！

author(zhlineskip): Add co-author info

e462f09

RuixiZhang42 reviewed Jun 28, 2026

View reviewed changes

author(zhlineskip): 修正 typo 和 README.md

fcc62b3

author(zhlineskip):Maintainer -> Maintainer

6de1b6c

author(zhlineskip): Update README.md

73766cb

Liam0205 mentioned this pull request Jun 28, 2026

feat(ci): release-ctan-upload 支持 CTAN note 内部备注作为手动 input #913

Merged

4 tasks

Liam0205 merged commit 380fa8c into CTeX-org:master Jun 28, 2026
14 checks passed

Uh oh!

Conversation

myhsia commented Jun 28, 2026

Uh oh!

github-actions Bot commented Jun 28, 2026

🔍 PR 审查

🟢 小问题 (1)

Uh oh!

RuixiZhang42 commented Jun 28, 2026

Uh oh!

myhsia commented Jun 28, 2026

Uh oh!

github-actions Bot commented Jun 28, 2026

🔍 PR 增量审查

🔴 阻塞问题 (1)

🟠 重要建议 (1)

Uh oh!

RuixiZhang42 Jun 28, 2026

Choose a reason for hiding this comment

Uh oh!

RuixiZhang42 Jun 28, 2026

Choose a reason for hiding this comment

Uh oh!

RuixiZhang42 Jun 28, 2026

Choose a reason for hiding this comment

Uh oh!

RuixiZhang42 Jun 28, 2026

Choose a reason for hiding this comment

Uh oh!

github-actions Bot commented Jun 28, 2026

🔍 PR 增量审查

🟢 小问题 (1)

Uh oh!

github-actions Bot commented Jun 28, 2026

🔍 PR 增量审查

🟢 小问题 (1)

Uh oh!

Liam0205 commented Jun 28, 2026

Uh oh!

github-actions Bot commented Jun 28, 2026

🔍 PR 增量审查

Uh oh!

Liam0205 commented Jun 28, 2026

Uh oh!

myhsia commented Jun 28, 2026

Done.

Uh oh!

Liam0205 commented Jun 28, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

myhsia commented Jun 28, 2026

Uh oh!

Liam0205 commented Jun 28, 2026

Uh oh!

Liam0205 commented Jun 28, 2026

Uh oh!

Uh oh!

myhsia commented Jun 28, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Liam0205 commented Jun 28, 2026

Uh oh!

myhsia commented Jun 28, 2026

Uh oh!

Liam0205 commented Jun 28, 2026

Uh oh!

Liam0205 commented Jun 28, 2026

Uh oh!

myhsia commented Jun 28, 2026

Uh oh!

Liam0205 commented Jun 28, 2026

Uh oh!

myhsia commented Jun 28, 2026

Uh oh!

Liam0205 commented Jun 28, 2026

Uh oh!

myhsia commented Jun 28, 2026

Uh oh!

Liam0205 commented Jun 28, 2026

Uh oh!

Liam0205 commented Jun 28, 2026 •

edited

Loading

myhsia commented Jun 28, 2026 •

edited

Loading