`land` should exit gracefully when there are non-submitted changes, now it crashes
还没有人认领这个 Issue。
评估
调研方向
从 stack_pr/cli.py 中的 command_land 和 set_base_branches 开始,跟踪 traceback,并复现列出的 stack-pr export、view 和 land 序列。完成的标准是 stack-pr land 能够妥善处理未提交的更改,而不是引发 ValueError: head is not set。
由索引模型根据 Issue 内容生成。
描述
Repro:
- make change
zzz stack-pr export- make change
ppp stack-pr view:
VIEW
Stack:
* 3344c73c (no PR, 'ZolotukhinM/stack/133' -> 'ZolotukhinM/stack/132'): ppp
* a7883442 (#139, 'ZolotukhinM/stack/132' -> 'main'): zzz
stack-pr land
LAND
Traceback (most recent call last):
File "/Users/mvz/.local/bin/stack-pr", line 7, in <module>
sys.exit(main())
~~~~^^
File "/Users/mvz/.local/pipx/venvs/stack-pr/lib/python3.13/site-packages/stack_pr/cli.py", line 1659, in main
command_land(common_args)
~~~~~~~~~~~~^^^^^^^^^^^^^
File "/Users/mvz/.local/pipx/venvs/stack-pr/lib/python3.13/site-packages/stack_pr/cli.py", line 1241, in command_land
set_base_branches(st, args.target)
~~~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^^
File "/Users/mvz/.local/pipx/venvs/stack-pr/lib/python3.13/site-packages/stack_pr/cli.py", line 483, in set_base_branches
e.base, prev_branch = prev_branch, e.head
^^^^^^
File "/Users/mvz/.local/pipx/venvs/stack-pr/lib/python3.13/site-packages/stack_pr/cli.py", line 300, in head
raise ValueError("head is not set")
ValueError: head is not set
- 主要语言
- Python
- 星标
- 568
- 派生
- 43
- PR 合并指标
- 30 天内没有已合并 PR
环境准备
- 没有 Dockerfile 或 Docker Compose 文件
- 没有 Pull Request 模板
- 阅读贡献指南
从这里开始
- 先读完整个 Issue,再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库,在一个分支上完成修改。
- 提交 Pull Request,并在描述里引用这个 Issue 编号。
modular/stack-pr 的其他 Issue
-
难度 3/5 1-2 天 新手友好度 30/100
-
难度 3/5 1-2 天 新手友好度 48/100
-
难度 3/5 1-2 天 新手友好度 42/100
-
难度 3/5 1-2 天 新手友好度 48/100
-
难度 3/5 1-2 天 新手友好度 45/100
相似的 Issue
-
难度 2/5 1-3 小时 新手友好度 85/100
维护者通常 1 天内回复
-
SR_SECURITY_DESCRIPTOR.fromString drops the SACL when no DACL is present可能已有人在做 @paul7436 今天认领。 未关闭
难度 2/5 1-3 小时 新手友好度 88/100
维护者通常 2 天内回复
-
难度 2/5 1-3 小时 新手友好度 85/100
equinor/fmu-sumo-uploader#302 ·
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 75/100
modelscope/evalscope#1821 ·
维护者通常 1 天内回复
-
Sanity on ansible-core devel fails: ignore-2.23.txt references the removed import-3.9 test可能已有人在做 @yurnov 今天认领。 未关闭needs_triage
难度 1/5 1 小时以内 新手友好度 91/100
ansible-collections/kubernetes.core#1275 ·
维护者通常 1 天内回复