Code review comment for lp:~stub/charm-helpers/fix-configure_sources

Revision history for this message
David Britton (dpb) wrote :

+1, tested and works great. Thanks for adding this.

Agreed with @tribaal on the docstring. Now that the functionality is expanding, I think it sort of needs the docstring, and it's a good chance to add it.

Marking as approved since this is a minor non-functional change suggested.

review: Approve

« Back to merge proposal