[Rpm-maint] [rpm-software-management/rpm] Extend dbus-announce plugin (PR #3532)
Michal Domonkos
notifications at github.com
Mon Feb 24 15:40:31 UTC 2025
dmnks left a comment (rpm-software-management/rpm#3532)
Please do the split of `send_ts_message()` into `send_ts_message_simple()` in a separate refactoring commit that precedes the addition of `send_ts_message_details()`. That'll make the resulting diffs easier to read and reason about.
There seems to be a memory leak (see the failed CI test) which will be easier to spot that way, too.
--
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/3532#issuecomment-2678851329
You are receiving this because you are subscribed to this thread.
Message ID: <rpm-software-management/rpm/pull/3532/c2678851329 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.rpm.org/pipermail/rpm-maint/attachments/20250224/37a88ec1/attachment.htm>
More information about the Rpm-maint
mailing list