Merged
Conversation
* 合并 (#8) * chore: add comprehensive contributing guide (#10) * chore: add comprehensive contributing guide * Update CONTRIBUTING.md Co-authored-by: greptile-apps[bot] <165735046+greptile-apps[bot]@users.noreply.github.com> * Update CONTRIBUTING.md Co-authored-by: greptile-apps[bot] <165735046+greptile-apps[bot]@users.noreply.github.com> --------- Co-authored-by: greptile-apps[bot] <165735046+greptile-apps[bot]@users.noreply.github.com> * ci: restore pr-check.yml workflow for dev branch - Restore backend and frontend PR checks - Enable Python syntax check and import test for backend - Enable pnpm build check for frontend - Fix 'Expected — Waiting for status to be reported' issue * feat(ci): upgrade PR review + add CI auto-fix & review responder - Upgrade codex-pr-review.md to 6-perspective review with confidence scoring - Rewrite claude-pr-review.yml with Codex-first fallback pattern - Add claude-ci-autofix.yml for auto-fixing CI failures + dev sync - Add claude-review-responder.yml for auto-responding to PR reviews - Update CI_CD_SETUP.md with new workflow documentation * fix(ci): address Codex review security findings - [Critical] auto-fix: restrict to same-repo PRs only (fork protection) - [High] review-responder: require explicit @claude mention, restrict to same-repo - [Medium] auto-fix: use `gh run view --log-failed` instead of broken API call - [Medium] pr-review: fix Codex matching to use head_sha + pull_requests - [Low] auto-fix: correct workflow name "Tests" → "Test Suite" * fix(ci): restore workflow name to match branch protection rules Rename "Claude PR Review (Fallback)" back to "Claude PR Review" so the check name matches what branch protection expects. * revert: restore workflow name to "Claude PR Review (Fallback)" * ci: fix invalid workflow syntax * ci: fix pnpm cache setup * ci: fix test workflow step order * ci: harden Claude workflows * ci: pin review responder checkout to SHA --------- Co-authored-by: greptile-apps[bot] <165735046+greptile-apps[bot]@users.noreply.github.com>
* 合并 (#8) * chore: add comprehensive contributing guide (#10) * chore: add comprehensive contributing guide * Update CONTRIBUTING.md Co-authored-by: greptile-apps[bot] <165735046+greptile-apps[bot]@users.noreply.github.com> * Update CONTRIBUTING.md Co-authored-by: greptile-apps[bot] <165735046+greptile-apps[bot]@users.noreply.github.com> --------- Co-authored-by: greptile-apps[bot] <165735046+greptile-apps[bot]@users.noreply.github.com> * ci: restore pr-check.yml workflow for dev branch - Restore backend and frontend PR checks - Enable Python syntax check and import test for backend - Enable pnpm build check for frontend - Fix 'Expected — Waiting for status to be reported' issue * feat(ci): upgrade PR review + add CI auto-fix & review responder - Upgrade codex-pr-review.md to 6-perspective review with confidence scoring - Rewrite claude-pr-review.yml with Codex-first fallback pattern - Add claude-ci-autofix.yml for auto-fixing CI failures + dev sync - Add claude-review-responder.yml for auto-responding to PR reviews - Update CI_CD_SETUP.md with new workflow documentation * fix(ci): address Codex review security findings - [Critical] auto-fix: restrict to same-repo PRs only (fork protection) - [High] review-responder: require explicit @claude mention, restrict to same-repo - [Medium] auto-fix: use `gh run view --log-failed` instead of broken API call - [Medium] pr-review: fix Codex matching to use head_sha + pull_requests - [Low] auto-fix: correct workflow name "Tests" → "Test Suite" * fix(ci): restore workflow name to match branch protection rules Rename "Claude PR Review (Fallback)" back to "Claude PR Review" so the check name matches what branch protection expects. * revert: restore workflow name to "Claude PR Review (Fallback)" * ci: fix invalid workflow syntax * ci: fix pnpm cache setup * ci: fix test workflow step order * ci: harden Claude workflows * ci: pin review responder checkout to SHA * docs: add CONTRIBUTING.md reference in README Direct contributors to read the contributing guide before starting. --------- Co-authored-by: greptile-apps[bot] <165735046+greptile-apps[bot]@users.noreply.github.com>
* 合并 (#8) * chore: add comprehensive contributing guide (#10) * chore: add comprehensive contributing guide * Update CONTRIBUTING.md Co-authored-by: greptile-apps[bot] <165735046+greptile-apps[bot]@users.noreply.github.com> * Update CONTRIBUTING.md Co-authored-by: greptile-apps[bot] <165735046+greptile-apps[bot]@users.noreply.github.com> --------- Co-authored-by: greptile-apps[bot] <165735046+greptile-apps[bot]@users.noreply.github.com> * ci: restore pr-check.yml workflow for dev branch - Restore backend and frontend PR checks - Enable Python syntax check and import test for backend - Enable pnpm build check for frontend - Fix 'Expected — Waiting for status to be reported' issue * feat(ci): upgrade PR review + add CI auto-fix & review responder - Upgrade codex-pr-review.md to 6-perspective review with confidence scoring - Rewrite claude-pr-review.yml with Codex-first fallback pattern - Add claude-ci-autofix.yml for auto-fixing CI failures + dev sync - Add claude-review-responder.yml for auto-responding to PR reviews - Update CI_CD_SETUP.md with new workflow documentation * fix(ci): address Codex review security findings - [Critical] auto-fix: restrict to same-repo PRs only (fork protection) - [High] review-responder: require explicit @claude mention, restrict to same-repo - [Medium] auto-fix: use `gh run view --log-failed` instead of broken API call - [Medium] pr-review: fix Codex matching to use head_sha + pull_requests - [Low] auto-fix: correct workflow name "Tests" → "Test Suite" * fix(ci): restore workflow name to match branch protection rules Rename "Claude PR Review (Fallback)" back to "Claude PR Review" so the check name matches what branch protection expects. * revert: restore workflow name to "Claude PR Review (Fallback)" * ci: fix invalid workflow syntax * ci: fix pnpm cache setup * ci: fix test workflow step order * ci: harden Claude workflows * ci: pin review responder checkout to SHA * docs: add CONTRIBUTING.md reference in README Direct contributors to read the contributing guide before starting. * update --------- Co-authored-by: greptile-apps[bot] <165735046+greptile-apps[bot]@users.noreply.github.com>
update test
fix: use env vars instead of secrets in step-level if conditions
Review Summary by QodoRemove GitHub workflows and CI/CD configuration
WalkthroughsDescription• Removes entire .github directory containing CI/CD workflows and configuration files • Deletes GitHub Actions workflow files for PR reviews, issue triage, release automation, and testing • Removes prompt templates used by automated workflows (Claude and Codex integrations) • Deletes CI/CD setup documentation and configuration files (CI_CD_SETUP.md, CI_TRIGGER.md, cliff.toml) • Removes funding configuration (FUNDING.yml) Diagramflowchart LR
A[".github directory"] -- "contains" --> B["Workflows"]
A -- "contains" --> C["Prompts"]
A -- "contains" --> D["Config files"]
B -- "deleted" --> E["PR reviews, CI/CD, testing"]
C -- "deleted" --> F["Claude & Codex prompts"]
D -- "deleted" --> G["Setup docs & configs"]
File Changes |
Code Review by Qodo🐞 Bugs (0) 📘 Rule violations (0) 📎 Requirement gaps (0)
Great, no issues found!Qodo reviewed your code and found no material issues that require reviewⓘ The new review experience is currently in Beta. Learn more |
Contributor
Greptile OverviewGreptile SummaryRemoved the
Confidence Score: 5/5
|
| Filename | Overview |
|---|---|
| .gitignore | Added /cankao/ to gitignore to prevent reference directory from being tracked |
| cankao/.github/prompts/pr-review-comprehensive.md | Deleted reference prompt template for comprehensive PR review (761 lines removed) |
| cankao/.github/workflows/claude-ci-autofix.yml | Deleted reference workflow for Claude CI autofix (542 lines removed) |
| cankao/.github/workflows/claude-pr-review.yml | Deleted reference workflow for PR reviews (514 lines removed) |
| cankao/.github/workflows/release.yml | Deleted reference release workflow (538 lines removed) |
Sequence Diagram
sequenceDiagram
participant Dev as Developer
participant Git as Git Repository
participant GH as GitHub
Dev->>Git: Delete cankao/ directory
Note over Git: Removes 6,671 lines across<br/>29 reference files
Dev->>Git: Update .gitignore
Note over Git: Add /cankao/ entry
Dev->>Git: git commit -m "update"
Dev->>GH: Push to remote (dev branch)
GH->>GH: Create PR #29
Note over GH: PR title: "dele"<br/>Target: dev branch
# Size label (XL: 6672 lines changed, 30 files changed)
gh pr edit 29 --repo datawhalechina/whale-whisper --add-label "size/XL"# Summary review body (includes required XL split suggestion)
gh pr review 29 --repo datawhalechina/whale-whisper --comment --body "$(cat <<'EOF'
## 🤖 Codex PR Review
No significant issues identified in this PR.
### PR Size: XL
- **Lines changed**: 6672
- **Files changed**: 30
- **Split suggestions**: Consider splitting into (1) deleting `cankao/.github/**` and (2) adding `/cankao/` to `.gitignore` to keep review atomic.
### Review Coverage
- [x] Logic and correctness - Clean
- [x] Security (OWASP Top 10) - Clean
- [x] Error handling - Clean
- [x] Type safety - Clean
- [x] Documentation accuracy - Clean
- [x] Test coverage - Adequate
- [x] Code clarity - Good
---
*Automated review by Codex AI*
EOF
)" |
6 tasks
FutureUnreal
added a commit
that referenced
this pull request
Feb 8, 2026
Merge pull request #29 from datawhalechina/main
6 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
概要
删除仓库中的
cankao/参考资料目录(包含 GitHub Actions 工作流模板、CI/CD 配置文档、Prompt 模板等),并将该目录加入.gitignore防止再次误提交。问题
cankao/(参考)目录包含大量 GitHub Actions 工作流模板、CI/CD 配置说明、PR Review / Issue 自动响应等 Prompt 模板文件。这些文件属于开发参考资料,不应纳入版本控制,占用仓库空间且可能造成混淆。关联 PR:
解决方案
cankao/目录下全部 29 个文件(共 -6,671 行).gitignore中添加/cankao/规则,防止该目录被再次跟踪变更内容
核心变更
cankao/.github/下的全部参考文件,包括:CI_CD_SETUP.md、CI_TRIGGER.md、FUNDING.yml)cliff.toml)changelog-update.md、codex-issue-auto-response.md、codex-pr-review.md、docs-update.md、pr-review-comprehensive.md、release-analysis.md、release-notes.md)claude-ci-autofix.yml、claude-issue-*.yml、claude-mention-responder.yml、claude-pr-*.yml、claude-review-responder.yml、claude-unified-docs.yml、codex-*.yml、dev.yml、pr-check.yml、release.yml、test.yml)辅助变更
.gitignore:新增/cankao/忽略规则(+1 行)测试
自测方式
.github/workflows/.gitignore规则正确Checklist
由 Claude AI 自动生成