Skip to content

ci: bootstrap GitHub Actions workflow - #15

Merged
yydounai1234 merged 4 commits into
mainfrom
chore/add-cicd
Jul 28, 2026
Merged

ci: bootstrap GitHub Actions workflow#15
yydounai1234 merged 4 commits into
mainfrom
chore/add-cicd

Conversation

@suerzzh

@suerzzh suerzzh commented Jul 21, 2026

Copy link
Copy Markdown

修改内容

  • 添加仓库基础 GitHub Actions CI
  • 添加初始 .gitignore
  • 添加 Pull Request 模板
  • 检查 README、LICENSE 和 .gitignore
  • 阻止明显的敏感文件名进入仓库

当前阶段

仓库目前尚未导入应用代码,因此本次 CI 只负责仓库基础检查。

前端、后端和 WebRTC 服务代码进入仓库后,再根据真实技术栈增加构建和测试任务。

验证方式

创建 Pull Request 后,由 GitHub Actions 自动运行 Repository Baseline

@fennoai fennoai 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.

The workflow structure is otherwise sound. One correctness issue is called out inline; no additional high-confidence performance, security, or documentation findings were identified.

Comment thread .github/workflows/ci.yml Outdated
run: |
set -euo pipefail

if ! grep -q "^# UniSpeaking" README.md; then

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

grep -q "^# UniSpeaking" matches the title anywhere in the file, despite the error message claiming the README must start with it. A README with any preceding content would pass this check. Please validate the first line explicitly (for example, with head -n 1 README.md or an anchored line-by-line comparison).

@suerzzh

suerzzh commented Jul 21, 2026

Copy link
Copy Markdown
Author

已在个人 Fork 中完成 CI 验证:

  1. main push 触发成功
  2. workflow_dispatch 手动触发成功
  3. README 标题错误时 CI 正确失败
  4. 恢复 README 后 CI 重新通过
  5. 已修复 README 第一行校验问题

请有合并权限的负责人审核并合并。

@yydounai1234 yydounai1234 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

@yydounai1234
yydounai1234 merged commit 69feeab into main Jul 28, 2026
1 check passed
@yydounai1234
yydounai1234 deleted the chore/add-cicd branch July 28, 2026 09: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.

2 participants