GTG

Code review comment for lp:~nimit-svnit/gtg/dates

Revision history for this message
Nimit Shah (nimit-svnit) wrote :

hey!
   sorry I must have overlooked that line in the make lint output. I have
updated the code now. Have also removed those trailing whitespaces.

Nimit Shah,
B Tech 3rd year,
Computer Engineering Department,
SVNIT Surat
Secretary ACM-NIT Surat
www.dude-says.blogspot.com

On Sat, Jun 9, 2012 at 6:01 PM, Nimit Shah <email address hidden> wrote:

> Bertrand: Is it okay now?
> Nimit Shah,
> B Tech 3rd year,
> Computer Engineering Department,
> SVNIT Surat
> Secretary ACM-NIT Surat
> www.dude-says.blogspot.com
>
>
>
> On Sat, Jun 9, 2012 at 5:21 PM, Nimit Shah <email address hidden> wrote:
>
> > Hi!
> > I'll make the required amendments.
> > Your observation was the very reason I had modified get_due_date
> method.
> >
> > Nimit Shah,
> > B Tech 3rd year,
> > Computer Engineering Department,
> > SVNIT Surat
> > Secretary ACM-NIT Surat
> > www.dude-says.blogspot.com
> >
> >
> >
> > On Sat, Jun 9, 2012 at 5:10 PM, Bertrand Rousseau <
> > <email address hidden>> wrote:
> >
> >> Review: Needs Fixing code, run
> >>
> >> Sorry, I wrongly chose "Approve", it's actually "Need fixing".. (you're
> >> very close, though, don't despair ;-) )
> >> --
> >> https://code.launchpad.net/~nimit-svnit/gtg/dates/+merge/108785<
> https://code.launchpad.net/%7Enimit-svnit/gtg/dates/+merge/108785>
> >> You are the owner of lp:~nimit-svnit/gtg/dates.
> >>
> >
> >
>
> --
> https://code.launchpad.net/~nimit-svnit/gtg/dates/+merge/108785
> You are the owner of lp:~nimit-svnit/gtg/dates.
>

« Back to merge proposal