Code review comment for lp:~mvo/webbrowser-app/multiarch-fixes

Revision history for this message
Olivier Tilloy (osomon) wrote :

The change looks good, thanks for taking care of this Michael.

I wonder if the new entry in debian/changelog is really necessary? And if the CI train will deal gracefully with it?

Usually when I modify files under debian/, I don’t add a new changelog entry, but I make sure they are documented in the commit message, which is what the CI train picks up to build its own changelog entry. But you probably know better than me how it all works.

review: Approve

« Back to merge proposal