Philipp Hörist pushed to branch master at gajim / gajim
Commits: 69770417 by Philipp Hörist at 2019-12-02T21:26:20Z Pylint: Don't check for optional module
- - - - - 0b11fb4b by Philipp Hörist at 2019-12-02T21:27:32Z Update pylintrc
pylint throws false positive no-member errors because the class is overridden in pygobject and pylint seem not to pick up on this
- - - - - 713a95fb by Philipp Hörist at 2019-12-02T21:42:29Z CI: Update pylint script
- - - - -
3 changed files:
- gajim/common/dbus/location.py - pylintrc - scripts/dev/pylint-ci.sh
Changes:
===================================== gajim/common/dbus/location.py ===================================== @@ -23,7 +23,7 @@ from nbxmpp.structs import LocationData from gajim.common import app
if app.is_installed('GEOCLUE'): - from gi.repository import Geoclue # pylint: disable=ungrouped-imports + from gi.repository import Geoclue # pylint: disable=ungrouped-imports,no-name-in-module
log = logging.getLogger('gajim.c.dbus.location')
===================================== pylintrc ===================================== @@ -140,7 +140,7 @@ ignore-mixin-members=yes # List of class names for which member attributes should not be checked (useful # for classes with dynamically set attributes). This supports the use of # qualified names. -ignored-classes=optparse.Values,thread._local,_thread._local +ignored-classes=optparse.Values,thread._local,_thread._local,gi.repository.GLib.Error
# List of module names for which member attributes should not be checked # (useful for modules/projects where namespaces are manipulated during runtime
===================================== scripts/dev/pylint-ci.sh ===================================== @@ -11,9 +11,10 @@ IGNORE_GTK_MODULE_ERRORS=C0103,C0301,C0330,C0415,E0401,E0611,E1101,E1133,E1136,R
"$PYLINT" --version
-"$PYLINT" --disable=$IGNORE_ERRORS --ignore=modules,gtk "$@" +"$PYLINT" --disable=$IGNORE_ERRORS --ignore=modules,dbus,gtk "$@" "$PYLINT" --disable=$IGNORE_GTK_MODULE_ERRORS "$@/gtk" "$PYLINT" --disable=E0401,C0415 "$@/common/modules" +"$PYLINT" "$@/common/dbus"
# C0103 invalid-name # C0301 line-too-long
View it on GitLab: https://dev.gajim.org/gajim/gajim/compare/2499d4dc20bf30b1bc701b8962c4d8e6d0...