Code review comment for lp:~barry/quickly/585617-userdirs

Revision history for this message
Didier Roche-Tolomelli (didrocks) wrote :

For a more generic approach (in any case, I'll merge it next week as I want to branch 0.4.3 first and this will land in 0.6), can you remove my two lines:
     if os.path.exists(os.path.expanduser('~/quickly-templates')):
          template_directories.append(os.path.expanduser('~/quickly-templates'))

and prepend QUICKLY_TEMPLATE by ~/quickly-template please? (just to have a common template path detection code). If you don't have the time to do it, I'll do it when merging :)

Thanks a lot Barry!

review: Needs Fixing

« Back to merge proposal