Code review comment for lp:~sergiusens/snapcraft/meta-all-yaml

Revision history for this message
Sergio Schvezov (sergiusens) wrote :

> Cool stuff with tests :)

Thanks for the review
> Some of the branches from the wrap methods are not tested. This is already
> big, so feel free to make a card or a bug for that.

_wrap_tests is not new code, just moved, so I won't be adding tests in this MP for it and leave a TODO.

> Plenty of comments in the diff.

Thanks, I will take a look now

« Back to merge proposal