Andy Parkins [ARCHIVE] on Nostr: š
Original date posted:2013-10-04 š Original message:On Friday 04 October 2013 ...
š
Original date posted:2013-10-04
š Original message:On Friday 04 October 2013 12:30:07 Mike Hearn wrote:
> Git makes it easy to fork peoples work off and create long series of
> commits that achieve some useful goal. That's great for many things.
> Unfortunately, code review is not one of those things.
>
> I'd like to make a small request - when submitting large, complex pieces of
> work for review, please either submit it as one giant squashed change, or
Don't do this. It throws away all of the good stuff that git lets you record.
There is more to a git branch than just the overall difference. Every single
log message and diff is individually valuable. It's easy to make a squashed
diff from many little commits; it's impossible to go the other way.
Command line for you so you don't have to think about it:
git diff $(git merge-base master feature-branch) feature-branch
git-merge-base finds the common ancestor between master and feature-branch,
and then compares feature-branch against that.
Andy
--
Dr Andy Parkins
andyparkins at gmail.com
Published at
2023-06-07 15:07:18Event JSON
{
"id": "944817ab01e65d450dacbbb43695e02dacb0c863df2cb78c142cf891cdc29399",
"pubkey": "99bec497728c848e65549d1a5257d08de97621edcb4b77073269a45dac708d59",
"created_at": 1686150438,
"kind": 1,
"tags": [
[
"e",
"c5a68e6f904af4bb9ba26063f532146fb896046fee8fed7dd28cab691644cf81",
"",
"root"
],
[
"e",
"ac9f7f36ff536513bd02ca65d0cc80be70f9ba37c9270b0fb6a2243828e77b2f",
"",
"reply"
],
[
"p",
"f2c95df3766562e3b96b79a0254881c59e8639f23987846961cf55412a77f6f2"
]
],
"content": "š
Original date posted:2013-10-04\nš Original message:On Friday 04 October 2013 12:30:07 Mike Hearn wrote:\n\u003e Git makes it easy to fork peoples work off and create long series of\n\u003e commits that achieve some useful goal. That's great for many things.\n\u003e Unfortunately, code review is not one of those things.\n\u003e \n\u003e I'd like to make a small request - when submitting large, complex pieces of\n\u003e work for review, please either submit it as one giant squashed change, or\n\nDon't do this. It throws away all of the good stuff that git lets you record. \nThere is more to a git branch than just the overall difference. Every single \nlog message and diff is individually valuable. It's easy to make a squashed \ndiff from many little commits; it's impossible to go the other way.\n\nCommand line for you so you don't have to think about it:\n\n git diff $(git merge-base master feature-branch) feature-branch \n\ngit-merge-base finds the common ancestor between master and feature-branch, \nand then compares feature-branch against that.\n\n\nAndy\n\n-- \nDr Andy Parkins\nandyparkins at gmail.com",
"sig": "c48038241a61d2f7743c600e59679e3dbcbcc9c5ff5a8ede0905f1e01995d5bd2e58cc59a6f9161ba84a44b7ec4ca2583072de5b47b75692df86dd102cd89f68"
}