Skip to content

fix(ch390): add missing email field in author to resolve CI KeyError - #2018

Merged
Rbb666 merged 2 commits into
masterfrom
copilot/fix-failing-github-actions-job
Aug 11, 2026
Merged

fix(ch390): add missing email field in author to resolve CI KeyError#2018
Rbb666 merged 2 commits into
masterfrom
copilot/fix-failing-github-actions-job

Conversation

Copilot AI commented Aug 11, 2026

Copy link
Copy Markdown
Contributor

CI was failing with Error message:'email' — a Python KeyError — because peripherals/ch390/package.json declared author with only a name field, and ci.py unconditionally accesses package_info['author']['email'].

Changes

  • peripherals/ch390/package.json: Added email (liucj1228@outlook.com, sourced from upstream commit history) and github (HosakaSu) to the author dict.
 "author": {
-    "name": "Liu Changjie"
+    "name": "Liu Changjie",
+    "email": "liucj1228@outlook.com",
+    "github": "HosakaSu"
 },

Co-authored-by: Rbb666 <64397326+Rbb666@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix failing GitHub Actions job build fix(ch390): add missing email field in author to resolve CI KeyError Aug 11, 2026
Copilot AI requested a review from Rbb666 August 11, 2026 03:48
@Rbb666

Rbb666 commented Aug 11, 2026

Copy link
Copy Markdown
Member

@copilot 我看软件包仓库有类似的:tag不存在/分支填写错误/email字段没有填写的情况,请扫描并修复仓库中存在的类似问题

Copilot AI commented Aug 11, 2026

Copy link
Copy Markdown
Contributor Author

@copilot 我看软件包仓库有类似的:tag不存在/分支填写错误/email字段没有填写的情况,请扫描并修复仓库中存在的类似问题

已对仓库中所有 820 个软件包进行了全面扫描,检查以下三类问题:

  1. email 字段缺失/为空:扫描全部 820 个 package.json,仅发现 peripherals/ch390/package.json 一处(即已在本 PR 修复的问题),现已全部清零。

  2. 分支名称填写错误(不存在的分支):对 28 个使用非标准分支名(非 master/main)的 VER_SHA 条目逐一通过 git ls-remote 验证,全部有效,无需修复。

  3. Tag 不存在(archive zip URL 指向无效 tag):对全仓库 531 条 archive URL 提取 owner/repo/tag 信息,通过 GitHub API 批量核查,所有 tag 均存在,无需修复。

结论:本 PR 修复的 ch390 是唯一存在问题的软件包,仓库其余部分均符合规范。Commit: 7833eee

@Rbb666
Rbb666 marked this pull request as ready for review August 11, 2026 04:32
@Rbb666
Rbb666 merged commit bc326a6 into master Aug 11, 2026
13 checks passed
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.

2 participants