docs: clarify fork_point is merge-base#9142
Conversation
|
Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA). View this failed invocation of the CLA check for more information. For the most up to date status, view the checks section at the bottom of the pull request. |
|
Why use a hyphen and quotes? |
|
@joyously It also comes up in discussion and writing about Git, e.g. this question "How to find the most recent common ancestor of two Git branches?" suggests using the |
|
Perhaps that should go in the Git comparison page, as I am an example of a user that knew nothing about it. |
|
We have this on this page under the
So perhaps we can use similar language here, i.e. saying that it's what |
I was a bit surprised that there wasn't a `merge_base` function, and more surprised that searching this page for "merge-base" didn't find anything. Let's add a note here.
I was a bit surprised that there wasn't a
merge_basefunction, and more surprised that searching this page for "merge-base" didn't find anything.Let's add a note here.
Checklist
If applicable:
CHANGELOG.mdREADME.md,docs/,demos/)cli/src/config-schema.json)how it works, how it's organized), including any code drafted by an LLM.
an eye towards deleting anything that is irrelevant, clarifying anything
that is confusing, and adding details that are relevant. This includes,
for example, commit descriptions, PR descriptions, and code comments.