On Fri Aug 1, 2025 at 2:52 PM CEST, Thomas Lamprecht wrote:
> Applied, thanks!
>
> But can you please add a cover-letter for any actual patch series (i.e., not
> just a single patch), as that provides some value even if it's empty, as it
> makes it very clear if trailers like T-b or R-b are intended for the full
> series or just a single patch out of it. b4 also uses the same logic to
> determine for which commits to apply the R-b and T-b too, so here I had to
> manually add Maximiliano's R-b and T-b to patch 2 to 5, as from b4's POV they
> where only supplied for patch 1/5.
>
> A diffstat over all patches is a side benefit, a short sentence as overview
> might be nice too, but for things like these here it can be even fine to not
> have that.
>
> Not a big thing, but would make my (and others using b4) life a bit easier and
> better ensure that tests and reviews get encoded into git, providing some
> recognition of that important work.
>

Okay, will keep this in mind for the future. Thanks for the hint and
explanation!


_______________________________________________
pve-devel mailing list
pve-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel

Reply via email to