Oooh, cool!

Looks good to me.

harvest/local_settings.py.sample is new… glad it's there, because for some 
reason I never took the time to think that using it would be a really really 
good idea!

The Readme says to run cp local_settings.py.sample local_settings.py. I know it 
isn't an automated script and only techies are going to be following that, but 
you may want to suggest cp -i or -n so it won't overwrite an existing 
local_settings.py that somebody has lovingly created.
I can see myself making that mistake without the Readme reminding me ;)
-- 
https://code.launchpad.net/~dholbach/harvest/debug/+merge/29193
Your team harvest-dev is requested to review the proposed merge of 
lp:~dholbach/harvest/debug into lp:harvest.

_______________________________________________
Mailing list: https://launchpad.net/~harvest-dev
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~harvest-dev
More help   : https://help.launchpad.net/ListHelp

Reply via email to