junior-jl wrote:

> Opinions differ on what goes in a commit message but this seems fine to me, 
> I'd just remove the sub-headings. My usual rule is that folks can choose not 
> to read it if they don't want to, include as much information as you would 
> want to see if you yourself had to come back to fix this change.
> (which happens to me more than I'd like to admit :) )

That's a good rule, thanks! Removed the sub-headings.

> (if you don't have the permissions to do that, I can do it for you, let me 
> know when you're ready)

I don't have write permissions here. Also, I tried using the `--fixup` flag on 
the second commit, does that mean the squashing is automatic or I need to 
squash the commits and push again? 

https://github.com/llvm/llvm-project/pull/73173
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to