Attention is currently required from: laforge, pespin, fixeria, lynxis lazus. osmith has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-ci/+/26393 )

Change subject: lint: annotate lines in gerrit ......................................................................

Patch Set 2:

(2 comments)

File lint/checkpatch/checkpatch_json.py:

https://gerrit.osmocom.org/c/osmo-ci/+/26393/comment/0b3473cb_b0a9c0e2 PS1, Line 3: # from coreboot rev 9cae17d028d4bd3b278fc89ada8e06287917e213

I'd note the exact commit in the commit message, but not in the file. […]

Done

File lint/lint_diff.sh:

https://gerrit.osmocom.org/c/osmo-ci/+/26393/comment/1336d844_247fdcf6 PS1, Line 34: if ! git diff -U0 "$COMMIT" | "$SCRIPT_DIR/checkpatch/checkpatch_osmo.sh" > gerrit_report ; then

This condition doesn't need to check if errors were found or not, because above it does "exit 0" if […]

Done