Code review comment for lp:~ampelbein/ubuntu/oneiric/libqtbamf/lp-765915

Didier Roche (didrocks) wrote :

Thanks for your work there! Merged :)
Please, in the future, ensure two things:
- check the output of debcheckout or apt-get source which should tell you that there is a Vcs-Bzr tag in debian/control pointing to the branch where the packaging is (I had to put your commit manually to lp:bamf-qt).
- when request a change, let the changelog use "UNRELEASED" instead of distribution, in case we are pending updates to add other changes. Then, the sponsor will change UNRELEASED to the version.

I've pushed your changed to lp:bamf-qt. Thanks a lot for your contribution there! :-)

review: Approve

« Back to merge proposal