Attention is currently required from: pespin.
msuraev has posted comments on this change. (
https://gerrit.osmocom.org/c/osmo-ci/+/29428
)
Change subject: scripts/obs/*.py: add option to ignore required programs check
......................................................................
Patch Set 1:
(1 comment)
File scripts/obs/lib/__init__.py:
https://gerrit.osmocom.org/c/osmo-ci/+/29428/comment/3d0ab55b_cef408c5
PS1, Line 36: parser.add_argument("-i", "--ignore-req",
action="store_true",
don't you need to se a default to false here?
Nope, argparse is smart enough to figure out that smth which isn't true is
false.
--
To view, visit
https://gerrit.osmocom.org/c/osmo-ci/+/29428
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-ci
Gerrit-Branch: master
Gerrit-Change-Id: I8b5eaedab87b586fb1ca4c22746be4dfbb3c2176
Gerrit-Change-Number: 29428
Gerrit-PatchSet: 1
Gerrit-Owner: msuraev <msuraev(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: osmith <osmith(a)sysmocom.de>
Gerrit-CC: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: pespin <pespin(a)sysmocom.de>
Gerrit-Comment-Date: Wed, 21 Sep 2022 12:00:44 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: pespin <pespin(a)sysmocom.de>
Gerrit-MessageType: comment