emacs-orgmode
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Orgmode] [patch] icalendar extended to define alarm triggers for ti


From: Eric S Fraga
Subject: Re: [Orgmode] [patch] icalendar extended to define alarm triggers for timed events
Date: Fri, 02 Jul 2010 13:56:18 +0100
User-agent: Wanderlust/2.15.9 (Almost Unreal) SEMI/1.14.6 (Maruoka) FLIM/1.14.9 (Gojō) APEL/10.7 Emacs/23.1 (i486-pc-linux-gnu) MULE/6.0 (HANACHIRUSATO)

On Fri, 2 Jul 2010 08:53:06 +0200, Carsten Dominik <address@hidden> wrote:
> 
> Hi Eric,
> 
> 
> On Jul 1, 2010, at 12:44 AM, Eric S Fraga wrote:
> 
> > Hello,
> >
> > attached is a rudimentary patch for org-icalendar.el to add the
> > definition of alarm triggers for timed events that are exported to
> > icalendar format.  The default is to behave as it does at the moment
> > however.
> >
> > I hope it proves passable (given my less than brilliant elisp
> > expertise...).
> 
> This looks pretty good!
> But can you please
> 
> - Make sure that the first line of the docstring of the new variable
>   fits into 75 characters (better: 72)?
>   You already have made it a stand-alone sentence, which is also
> needed.
> 
> - It would also be helpful if if can make git commit and
>   include the proper commit message including a ChangeLog-like entry
>   as the second paragraph in that message.
> 
> Great work!
> 
> - Carsten
> 

Thanks Carsten.

I have seen the few messages regarding how to submit patches so I see
now what you would like.  I'll try to do that this weekend (I'm not au
fait with git but I think the instructions are fairly clear...).
-- 
Eric S Fraga
GnuPG: 8F5C 279D 3907 E14A 5C29  570D C891 93D8 FFFC F67D

reply via email to

[Prev in Thread] Current Thread [Next in Thread]