laforge has submitted this change. ( https://gerrit.osmocom.org/c/pysim/+/37966?usp=email ) Change subject: contrib/jenkins.sh: Install dependencies before calling pylint ...................................................................... contrib/jenkins.sh: Install dependencies before calling pylint This is the only way we can make sure pylint has all required information about imports from packages we depend upon. Change-Id: I29582aa3d7f9ace9ce832d5b907420aaf14881fb --- M contrib/jenkins.sh 1 file changed, 6 insertions(+), 0 deletions(-) Approvals: fixeria: Looks good to me, but someone else must approve laforge: Looks good to me, approved Jenkins Builder: Verified diff --git a/contrib/jenkins.sh b/contrib/jenkins.sh index b3d83dd..5356fcb 100755 --- a/contrib/jenkins.sh +++ b/contrib/jenkins.sh @@ -51,6 +51,12 @@ "pylint") # Print pylint version pip3 freeze | grep pylint + + virtualenv -p python3 venv --system-site-packages + . venv/bin/activate + + pip install . + # Run pylint to find potential errors # Ignore E1102: not-callable # pySim/filesystem.py: E1102: method is not callable (not-callable) -- To view, visit https://gerrit.osmocom.org/c/pysim/+/37966?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: merged Gerrit-Project: pysim Gerrit-Branch: master Gerrit-Change-Id: I29582aa3d7f9ace9ce832d5b907420aaf14881fb Gerrit-Change-Number: 37966 Gerrit-PatchSet: 3 Gerrit-Owner: laforge <laforge@osmocom.org> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: dexter <pmaier@sysmocom.de> Gerrit-Reviewer: fixeria <vyanitskiy@sysmocom.de> Gerrit-Reviewer: laforge <laforge@osmocom.org>