Code review comment for lp:~dhavlicek/awn-extras/calendar-evolution

Revision history for this message
Michael Rooney (mrooney) wrote :

Looks pretty good on an initial review. One comment is that you shouldn't be doing print statements for normal things like on line 50 (printing errors are fine).

I also liked your change in the previous branch where you used the calendar module to get the length of the month; I wouldn't mind seeing that in there as well as a bonus cleanup.

review: Needs Fixing

« Back to merge proposal