Attention is currently required from: msuraev.

View Change

2 comments:

• File jobs/gerrit-pipeline-result.yml:

  • Patch Set #1, Line 12: node: osmocom-gerrit-debian10 || osmocom-gerrit-debian11

    What's the reason for using Debian 10? I though the plan is to migrate everything we can to Debian 1 […]

    see discussion here: https://gerrit.osmocom.org/c/osmo-ci/+/29604/4/jobs/gerrit-binpkgs.yml#16

• File scripts/jenkins-gerrit/pipeline_summary.py:

  • Patch Set #1, Line 174: def main():

    Why extra function? You can just move it directly below where main() is called.

    It's a convention used often in python code, and personally I think it looks better than just writing the code below the if. But functionally it would be the same.

To view, visit change 29671. To unsubscribe, or for help writing mail filters, visit settings.