[Merge] ~xypron/ubuntu/+source/rsyslog:merge-lp2045033-noble-8.2312.0-2 into ubuntu/+source/rsyslog:debian/sid
Nick Rosbrook
mp+457913 at code.launchpad.net
Fri Jan 5 19:28:53 UTC 2024
Review: Needs Fixing
The new changes look good to me. My only comments are about the changelog. I'm not sure if it really matters, but I think it would be best to follow the usual conventions, e.g.
* Merge with Debian unstable (LP: #2045033). Remaining changes:
instead of:
Merge with Debian unstable (LP: #2045033)
* Remaining changes:
Also, since you dropped a change, it would be good to elaborate briefly on why it was dropped. In this case it looks like Debian made the same change, so I would say something like:
* Dropped changes, included in Debian:
...
Finally, you have two of each "merge-changelogs" and "reconstruct-changelog" commits which is odd. If you make manual changes to the changelog, I think the convention is to name the commit "update changelog" or simply "changelog".
--
https://code.launchpad.net/~xypron/ubuntu/+source/rsyslog/+git/rsyslog/+merge/457913
Your team Ubuntu Sponsors is requested to review the proposed merge of ~xypron/ubuntu/+source/rsyslog:merge-lp2045033-noble-8.2312.0-2 into ubuntu/+source/rsyslog:debian/sid.
More information about the Ubuntu-sponsors
mailing list