Hi Nicolas, Thank you for the reviewing my suggestions. I've fixed issues you've listed. I've attached new patch to this email. Let me know what you think. Thanks, Mikhail On Sun, 2021-12-26 at 22:26 +0100, Nicolas Goaziou wrote: > Hello, > > Thanks. Some comments follow. > > Mikhail Skorzhinskii writes: > > > * lisp/ox-icalendar.el (org-icalendar-scheduled-summary-prepend): > > configurable prefix for the scheduled headlines > > "New variable" is enough. > > > * lisp/ox-icalendar.el (org-icalendar-deadline-summary-prepend): > > configurable prefix for the headlines with a deadline > > Ditto. > > > +(defcustom org-icalendar-scheduled-summary-prepend "S: " > > +  "String used for prepending summary in exported scheduled > > headlines." > > +  :group 'org-export-icalendar > > +  :type 'string) > > + > > + > > +(defcustom org-icalendar-deadline-summary-prepend "DL: " > > +  "String used for prepending summary in exported deadlines." > > +  :group 'org-export-icalendar > > +  :type 'string) > > + > > Could you add missing :safe and :package-version keywords? > > Regards,