Re: [PATCH v14 7/8] history: create squashed commits without editing

On 20/08/2026 21:10, Junio C Hamano wrote:

"Harald Nordgren via GitGitGadget" [off-list ref] writes:

quoted

+	/*
+	 * Allow "fixup! <hex object id>", but not "fixup! HEAD^" or
+	 * "fixup! main". If the target is not being squshed check the subject
+	 * to allow "fixup! abc123" and "fixup! <subject of abc123>" to be
+	 * squashed together.
+	 */
+	target = lookup_commit_reference_by_name(s);
+	if (target && istarts_with(oid_to_hex(&target->object.oid), s)) {

Why istarts_with()?  "fixup! ABCdef" should not be accepted, should it?

I agree there isn't really a compelling case for mixed case oids, but 
accepting all uppercase, or all lowercase seems reasonable, or are we 
planning to completely ban uppercase oids as brain has suggested? The 
aim here is to accept any oid that rebase would, but not accept ref names.

Thanks

Phillip