Repository navigation
docs(agents): 约定 PR 截图分支 pr-assets/* 的用法并禁止删除 - #36
Merged
Merged
Conversation
已合并 PR 的描述仍引用这些分支上的图片,误删会让截图永久失效。
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.
#31、#33、#34 的 PR 描述都引用了
pr-assets/*分支上的截图。这些分支一旦被当成"没用的旧分支"删掉,已合并 PR 里的截图会永久挂掉,所以写进 AGENTS.md 的「提交与 PR」:pr-assets/<PR 主题>(只含图片),用 raw 链接引用,不提交进功能分支pr-assets/*分支,合并后也不删;清理时只删功能分支另外确认了一下:仓库开着「合并后自动删除源分支」,所以
feat/login-audit、chore/readme-screenshots这类功能分支合并后已经自动删掉了;pr-assets/*从来不是 PR 的源分支,不受影响。只改了 AGENTS.md 两行。