Skip to content

ci: add PR workflow for build, test, and Codecov coverage upload#40

Merged
minorcell merged 10 commits into1024XEngineer:mainfrom
Yumiue:main
Mar 24, 2026
Merged

ci: add PR workflow for build, test, and Codecov coverage upload#40
minorcell merged 10 commits into1024XEngineer:mainfrom
Yumiue:main

Conversation

@Yumiue
Copy link
Copy Markdown
Collaborator

@Yumiue Yumiue commented Mar 24, 2026

概要

这个 PR 主要补充了仓库的 PR 自动化校验流程,并清理了本地覆盖率产物相关文件。

变更内容

  • 新增 GitHub Actions CI 工作流
  • 在 PR 创建和更新时自动执行 go build ./...
  • 在 PR 创建和更新时自动执行 go test ./... 并生成覆盖率
  • 将覆盖率结果上传到 Codecov
  • .gitignore 中补充覆盖率产物忽略规则
  • 删除误提交的 cover 文件
  • 补充少量注释,帮助理解相关逻辑

目的

  • 让 PR 在合并前自动完成基础校验
  • 更早发现构建和测试问题
  • 持续跟踪覆盖率结果
  • 避免本地生成文件再次被误提交

说明

  • 需要在 GitHub Actions Secrets 中配置 CODECOV_TOKEN
  • 当前工作流默认仅在非 Draft 的 PR 上运行

@chatgpt-codex-connector
Copy link
Copy Markdown

Codex usage limits have been reached for code reviews. Please check with the admins of this repo to increase the limits by adding credits.
Credits must be used to enable repository wide code reviews.

@Yumiue Yumiue changed the title 删除cover文件,加入适当注释 删除cover文件,加入适当注释,加上action组件 Mar 24, 2026
@Yumiue Yumiue changed the title 删除cover文件,加入适当注释,加上action组件 docs|feat(remove,add)排除并删除cover文件,添加CI workflow Mar 24, 2026
@Yumiue Yumiue changed the title docs|feat(remove,add)排除并删除cover文件,添加CI workflow ci: add PR workflow for build, test, and Codecov coverage upload Mar 24, 2026
Comment thread .github/workflows/ci.yml
flags: unittests
fail_ci_if_error: true
env:
CODECOV_TOKEN: ${{ secrets.CODECOV_TOKEN }}
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.

这个变量“CODECOV_TOKEN”仓库里面配置了吗?

@minorcell minorcell merged commit 5f2ba10 into 1024XEngineer:main Mar 24, 2026
phantom5099 added a commit that referenced this pull request Apr 19, 2026
test: improve custom provider loader coverage
pionxe pushed a commit to pionxe/neo-code that referenced this pull request Apr 24, 2026
…7016845

fix(verify): harden verifier security checks
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