Skip to content

Commit

Permalink
code quality
Browse files Browse the repository at this point in the history
  • Loading branch information
bjlittle committed Feb 21, 2023
1 parent a95c226 commit d43f0e1
Show file tree
Hide file tree
Showing 3 changed files with 11 additions and 5 deletions.
4 changes: 3 additions & 1 deletion .pylintrc
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,9 @@ fail-on=
fail-under=10.0

# Files or directories to be skipped. They should be base names, not paths.
ignore=CVS,_version.py
ignore=CVS,
_version.py,
CODE_OF_CONDUCT.md

# Add files or directories matching the regex patterns to the ignore-list. The
# regex matches against paths and can be in Posix or Windows format.
Expand Down
1 change: 1 addition & 0 deletions src/geovista/qt.py
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,7 @@
"""
try:
# pylint: disable-next=import-outside-toplevel
import pyvistaqt as pvqt
except ImportError as e:
e.msg = f'{e.msg} - please install the "pyvistaqt" and "pyqt" packages.'
Expand Down
11 changes: 7 additions & 4 deletions tests/test_qt.py
Original file line number Diff line number Diff line change
Expand Up @@ -6,10 +6,13 @@


def test_import():
# the test environment may or may not contain the "pyvistaqt" package.
# this test only checks that the expected exception is raised when it
# detects that the package is not already available in the test
# environment
"""
The test environment may or may not contain the "pyvistaqt" package.
This test only checks that the expected exception is raised when it
detects that the package is not already available in the test
environment.
"""
try:
import pyvistaqt
except ImportError:
Expand Down

0 comments on commit d43f0e1

Please sign in to comment.