Code review comment for lp:~edgar-b-dsouza/acire/snippet_dependency_res_1

Revision history for this message
Jono Bacon (jonobacon) wrote :

Hi Ed,

I just wanted to let you know that I am finding more and more bug fixes that I need to fix in Acire, so I expect lp:acire to change more frequently. I just wanted to make sure I gave you a heads up this to keep syncing your branch with lp:acire.

We discussed this before, but I would still recommend you keep the vast majority of your logic in a separate source file (e.g. depchecker.py) and then just hook it into bin/acire to integrate it into the app. This will mean that the regular changes in bin/acire will not screw up your branch. :-)

Thanks again for all your hard work, Ed!

« Back to merge proposal