Thanks for the MP, Athos.
I'll leave my review notes in this comment (instead of inline) because not all changes are visible in the MP diff.
- d/changelog: LGTM, but the "+" items need to be indented further right (2 spaces).
Otherwise, it LGTM. I've created the following bileto ticket and uploaded the package there:
https://bileto.ubuntu.com/#/ticket/4846
As I mentioned before, bileto was having problems with autopkgtest triggers last time I used it. Hopefully it'll work now.
I'm approving this MP but let's take a look at the autopkgtest results before we upload the package.
« Back to merge proposal
Thanks for the MP, Athos.
I'll leave my review notes in this comment (instead of inline) because not all changes are visible in the MP diff.
- d/changelog: LGTM, but the "+" items need to be indented further right (2 spaces).
Otherwise, it LGTM. I've created the following bileto ticket and uploaded the package there:
https:/ /bileto. ubuntu. com/#/ticket/ 4846
As I mentioned before, bileto was having problems with autopkgtest triggers last time I used it. Hopefully it'll work now.
I'm approving this MP but let's take a look at the autopkgtest results before we upload the package.