Re: [PATCH] Add notifications.el

[Stefan Monnier]

>> +(defun notifications-notify (&rest params)
>> +  "Send notification via D-Bus using the Freedesktop notification protocol.
>> +Various PARAMS can be set:
>> +
>> + :title          The notification title.
>> + :body           The notification body text.

> You might mention, that these are mandatory parameters. Check the effect of
> (notifications-notify :urgency "low")

If these are really fundamentally mandatory, then it would be better to
turn them into normal parameters, like

  (notifications-notify title body &rest params)


-- Stefan