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...