Code review comment for lp:~psivaa/charms/trusty/jenkaas/example-job

Revision history for this message
Francis Ginther (fginther) wrote :

Looks good. I would change the missing "#!/bin/sh" though. I agree it isn't strictly necessary, but it will more then likely lead to confusion if it's not there.

review: Approve

« Back to merge proposal