Repository navigation
fix(ci): 修复 Markdown 解析器安装失败 - #337
Merged
Merged
Conversation
Contributor
There was a problem hiding this comment.
审查结论
按完整 pr_diff.txt 审查:Markdown 解析器已改为按需导入,评论相关 workflow 在解析前配置 Python 3.12,非评论事件路径不再安装解析器;CI 测试环境也补齐了 setup-python。未发现需要阻止合并的正确性、安全性、测试或工作流触发问题。
验证
聚焦运行 test_prepare_issue_spec_context.py(29 项)和 test_resolve_pr_event.py(24 项)均通过;test_review_workflow_dispatch.py 因本地审查环境缺少 PyYAML 未能导入,相关 workflow 变更已通过静态检查。
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.
Summary
venv依赖问题:需要安装 Markdown 解析器的 job 先通过actions/setup-python@v6配置 Python 3.12;CI 的 Python 测试依赖安装步骤同样配置。Testing
uv run --no-project --with PyYAML==6.0.3 --with markdown-it-py==4.2.0 --with mdurl==0.1.2 python -m unittest discover -s .github/aicodingflow-tests:526 项通过。actionlint .github/workflows/review-pr.yml .github/workflows/respond-to-pr-comment.yml .github/workflows/create-spec-from-issue.yml .github/workflows/ci.yml:通过。git diff --check:通过。Risk
setup-python在 runner 上提供 Python 3.12;目标仓库同步后仍需用实际 Actions 运行验证该 runner 的工具缓存或下载能力。How to revert
git revert 5e5a4055201343421516053d46d9fc42d2581cfa,恢复原有安装步骤和导入方式。