dexter has posted comments on this change. ( https://gerrit.osmocom.org/c/pysim/+/32719 )
Change subject: WIP: apply pylinth on cmd2 1.5 and later versions seperately ......................................................................
Patch Set 4:
(2 comments)
This change is ready for review.
Patchset:
PS4: My approach seems to be problematic. Not sure if it makes sense to take the effort to debug this.
File contrib/jenkins.sh:
https://gerrit.osmocom.org/c/pysim/+/32719/comment/dcff107e_728daba0 PS4, Line 36: pip uninstall -y cmd2 I am getting error messages around the pip install/uninstall commandlines. In particular:
+ [ ! -d ./pysim-testdata/ ] + pip uninstall -y cmd2 Skipping cmd2 as it is not installed.
This does not make sense. Pylint should need a cmd2 version to be installed in order to do the linting. How can it even complain about things like constructor arguments when cmd2 is not installed?