Change in pysim[master]: contrib/jenkins.sh: run pylint to find potential errors

This is merely a historical archive of years 2008-2021, before the migration to mailman3.

A maintained and still updated list archive can be found at https://lists.osmocom.org/hyperkitty/list/gerrit-log@lists.osmocom.org/.

laforge gerrit-no-reply at lists.osmocom.org
Sun May 2 20:22:49 UTC 2021


laforge has submitted this change. ( https://gerrit.osmocom.org/c/pysim/+/24033 )

Change subject: contrib/jenkins.sh: run pylint to find potential errors
......................................................................

contrib/jenkins.sh: run pylint to find potential errors

Change-Id: Ib4cdd0fa824329569f973925ba3a46656a8c8296
---
M contrib/jenkins.sh
1 file changed, 12 insertions(+), 0 deletions(-)

Approvals:
  laforge: Looks good to me, approved
  Jenkins Builder: Verified



diff --git a/contrib/jenkins.sh b/contrib/jenkins.sh
index 2d9653a..cd9629c 100755
--- a/contrib/jenkins.sh
+++ b/contrib/jenkins.sh
@@ -26,6 +26,18 @@
 # Execute automatically discovered unit tests first
 python -m unittest discover -v -s tests/
 
+# Run pylint to find potential errors
+# Ignore E1102: not-callable
+#   pySim/filesystem.py: E1102: method is not callable (not-callable)
+# Ignore E0401: import-error
+#   pySim/utils.py:276: E0401: Unable to import 'Crypto.Cipher' (import-error)
+#   pySim/utils.py:277: E0401: Unable to import 'Crypto.Util.strxor' (import-error)
+pip install pylint
+python -m pylint --errors-only \
+	--disable E1102 \
+	--disable E0401 \
+	pySim *.py
+
 # attempt to build documentation
 pip install sphinx
 pip install sphinxcontrib-napoleon

-- 
To view, visit https://gerrit.osmocom.org/c/pysim/+/24033
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings

Gerrit-Project: pysim
Gerrit-Branch: master
Gerrit-Change-Id: Ib4cdd0fa824329569f973925ba3a46656a8c8296
Gerrit-Change-Number: 24033
Gerrit-PatchSet: 4
Gerrit-Owner: fixeria <vyanitskiy at sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge at osmocom.org>
Gerrit-MessageType: merged
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20210502/a722519f/attachment.htm>


More information about the gerrit-log mailing list