fix: stabilize backend ci dependency install#835
Merged
Conversation
🤖 自动审查报告
📁 修改的文件
🧠 AI 代码审查意见审查结论Not Ready 存在关键阻断项，需修复后方可合入。 审查结果必要性
关联性
类型
描述完整性
风险级别
必改项 (Blocking Issues)
建议项 (Suggestions)
|
Contributor
There was a problem hiding this comment.
Pull request overview
This PR stabilizes the backend CI dependency installation by consolidating backend runtime + CI tooling installs into a single requirements entrypoint, avoiding a duplicated pip install path that was intermittently terminated in GitHub Actions.
Changes:
- Add
requirements-ci.txtto centralize backend CI dependencies (includesrequirements.txt+flake8/pytest). - Update
.github/workflows/ci.ymlto install fromrequirements-ci.txtin a singlepipinvocation. - Expand
actions/setup-pythonpip cache key inputs to include bothrequirements.txtandrequirements-ci.txt.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| requirements-ci.txt | New CI requirements entrypoint that includes runtime deps plus backend-gate lint/test tooling. |
| .github/workflows/ci.yml | Switches backend-gate to a single install step and updates pip cache dependency paths. |
This was referenced Mar 24, 2026
Closed
zhouruikevin
pushed a commit
to zhouruikevin/daily_stock_analysis
that referenced
this pull request
Mar 25, 2026
* fix: stabilize backend ci dependency install * fix: split backend gate workflow phases * fix: retry backend ci dependency install * fix: install litellm from github source
Copilot AI
pushed a commit
to qingdaoqin/daily_stock_analysis
that referenced
this pull request
Apr 10, 2026
* fix: stabilize backend ci dependency install * fix: split backend gate workflow phases * fix: retry backend ci dependency install * fix: install litellm from github source Co-authored-by: qingdaoqin <43225158+qingdaoqin@users.noreply.github.com>
yangshiyan
pushed a commit
to yangshiyan/daily_stock_analysis
that referenced
this pull request
Apr 20, 2026
* fix: stabilize backend ci dependency install * fix: split backend gate workflow phases * fix: retry backend ci dependency install * fix: install litellm from github source
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.
PR Type
Background And Problem
backend-gaterepeatedly failed in GitHub Actions before any repo checks ran. Recent attempts showedpip install -r requirements.txtcompleting successfully, then the workflow started a secondpip install flake8 pytestinvocation and the shell was terminated with exit code 143 during that second install phase.Impact:
./scripts/ci_gate.sheven startedTrigger:
.github/workflows/ci.ymlinstalled backend runtime deps and CI tooling in separate pip commandsScope Of Change
.github/workflows/ci.ymlrequirements-ci.txtIssue Link
No linked issue.
Motivation: unblock PR CI by removing the duplicated pip install path that repeatedly died with exit code 143.
Acceptance criteria:
./scripts/ci_gate.shstill passes end to endVerification Commands And Results
Key output & conclusion:
1139 passed, 104 subtests passedon the exploratory PR branch before extraction1123 passed, 104 subtests passedon this standalone branch based on currentmainbackend-gate: all checks passedCompatibility And Risk
Low risk.
requirements-ci.txtsimply centralizes backend CI tooling into the same install transactionrequirements-ci.txtRollback Plan
Revert commit
1cf2189to restore the previous backend CI install flow.EXTRACT_PROMPT Change (if applicable)
Not applicable.
Checklist
docs/CHANGELOG.md；若未更新README.md，已说明原因与文档落点 / If user-visible changes are included, the relevant docs anddocs/CHANGELOG.mdare updated; ifREADME.mdwas not updated, the reason and documentation location are explainedReason for not updating docs/README: