openshift-eng / elliott

Apache License 2.0
15 stars 28 forks source link

input value check for elliott go cli #510

Closed Ximinhan closed 1 year ago

Ximinhan commented 1 year ago

as we are tying to integrate elliott go cli into art-bot, we need to check the input value and return clear output

$ ./elliott go -a 00000000
The input value is not valid.
$ ./elliott go -a ""
Usage: elliott go [OPTIONS]
Try 'elliott go -h' for help.

Error: Invalid value for '--advisory' / '-a': '' is not a valid integer.
$ ./elliott go -a 111
there is no builds related to golang
openshift-bot commented 1 year ago

Build #1

GLOB sdist-make: /mnt/workspace/jenkins/working/art-tools_elliott_PR-510/setup.py
py38 create: /mnt/workspace/jenkins/working/art-tools_elliott_PR-510/.tox/py38
py38 installdeps: -rrequirements-dev.txt
ERROR: invocation failed (exit code 1), logfile: /mnt/workspace/jenkins/working/art-tools_elliott_PR-510/.tox/py38/log/py38-1.log
================================== log start ===================================
ERROR: Double requirement given: setuptools>=65.5.1 (from -r requirements.txt (line 17)) (already in setuptools>=41.6.0 (from -r requirements-dev.txt (line 2)), name='setuptools')
WARNING: You are using pip version 20.1.1; however, version 23.0.1 is available.
You should consider upgrading via the '/mnt/workspace/jenkins/working/art-tools_elliott_PR-510/.tox/py38/bin/python -m pip install --upgrade pip' command.

=================================== log end ====================================
ERROR: could not install deps [-rrequirements-dev.txt]; v = InvocationError('/mnt/workspace/jenkins/working/art-tools_elliott_PR-510/.tox/py38/bin/python -m pip install -rrequirements-dev.txt', 1)
___________________________________ summary ____________________________________
ERROR:   py38: could not install deps [-rrequirements-dev.txt]; v = InvocationError('/mnt/workspace/jenkins/working/art-tools_elliott_PR-510/.tox/py38/bin/python -m pip install -rrequirements-dev.txt', 1)
Ximinhan commented 1 year ago

the requirements-dev.txt check failure is not related to the pr, will merge the pr and left the requirements.txt mismatch to synk