story 230 · bugs
A report that git log --graph --all draws an orphan branch's root with parents has been traced to the visual-root indentation added in a34c00d96. A patch was posted, but Pablo Sabater says it was malformed and asked for git send-email. He also says the graph should render the same with or without indentation here.
→ full story→ thread (9)
story 231 · patches
Julia Evans proposed a rewritten EXAMPLES section for git-diff based on what users say they use. Junio Hamano objected to changing one example to --staged without matching changes elsewhere, and to wrapped sentences in tables. Evans offered to revert that and suggested horizontal definition lists.
→ full story→ thread (7)
story 232 · discussion
Kevin Lyles found that a pre-commit hook that modifies staged content does not take effect with git commit <pathspec>. Chris Torek explained the index handling involved. Junio Hamano said hooks are meant for validation and floated documenting that and possibly detecting mutation.
→ full story→ thread (5)
story 233 · bugs
Patrick Peroša reported that bare --force-with-lease has no effect when pushing to a remote group. Tyler Cipriani posted a patch for push_multiple() that fixes this and a second case in which --force-with-lease=<ref>: could overwrite a remote branch.
→ full story→ thread (2)
story 234 · patches
Okhuomon Ajayi's series makes notes-utils.c use the repository it is passed and, in v2, drops USE_THE_REPOSITORY_VARIABLE from the file. Patrick Steinhardt and Karthik Nayak found it good, while Junio Hamano cautioned that the notes code has a long way to go before it is free of the_repository.
→ full story→ thread (10)
story 235 · discussion
Johannes Welsch proposed git branch -s <name> to create a branch and switch to it. Junio Hamano asked why git checkout -b is not enough, and Pushkar Singh said it would overlap with git switch -c and blur the split that motivated git switch.
→ full story→ thread (3)
story 236 · project
Tuomas Ahola asked whether anyone still uses .cirrus.yml now that Cirrus CI appears to have shut down. Carlo Arenas sees no reason to keep it. Patrick Steinhardt agrees it should go but wants to give Arenas a few days before a removal patch.
→ full story→ thread (8)