Code review comment for ~morphis/snappy-hwe-snaps/+git/jenkins-jobs:f/kernel-cross-build-support

Revision history for this message
Alfonso Sanchez-Beato (alfonsosanchezbeato) wrote :

A couple of inline comments. Also:

1. Please add to the header of "tools/shyaml" the external repo/commit from which it was taken so we cant track the origin an update easily in the future.

2. It would be a good opportunity to clean scripts of shellcheck warnings, see
https://pastebin.canonical.com/191399/
and
https://pastebin.canonical.com/191400/

review: Needs Fixing

« Back to merge proposal