Sorry, something went wrong.
There was a problem hiding this comment.
Adds in-script documentation clarifying an unintuitive git checkout master -- || git checkout -b master edge case when multiple remotes provide a master branch, so future contributors understand why master’s reflog may end up reflecting HEAD history during test initialization (Closes #2145).
Changes:
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Sorry, something went wrong.
There was a problem hiding this comment.
I didn't think of a better way to deal with this besides adding a comment. Also, I had originally thought of including this as part of some larger PR, but it doesn't really fit nicely in with any of the other forthcoming GitPython PRs I'm working on. So I'm doing it by itself. I think this at least better than not expanding the comment, and I plan to merge this once CI passes.
Sorry, something went wrong.
|
Incidentally, I decided to try to have Claude Code proceed in its default way, rather than articulating a plan first. Overall, this is faster but kind of rough: for example, the branch name is meaningless, which is something I try to avoid. I don't think the branch name is worth the noise and churn of remaking the PR, though. |
Sorry, something went wrong.
Summary
Closes #2145.
Test plan
🤖 Generated with Claude Code