[Rpm-maint] [rpm-software-management/rpm] Extend dbus-announce plugin (PR #3532)

Michal Domonkos notifications at github.com
Tue Feb 25 11:19:43 UTC 2025


dmnks left a comment (rpm-software-management/rpm#3532)

Hmm, the splitting commit seems to do more refactoring than really needed :sweat_smile: It replaces the `err` exit label with if/else contructs. Was it needed in order to make the split easier/cleaner? If not, I think we should just do the minimum change necessary (i.e. the split).

-- 
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/3532#issuecomment-2681621635
You are receiving this because you are subscribed to this thread.

Message ID: <rpm-software-management/rpm/pull/3532/c2681621635 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.rpm.org/pipermail/rpm-maint/attachments/20250225/1be98dcc/attachment-0001.htm>


More information about the Rpm-maint mailing list