Refactored by Sourcery (#1)

Co-authored-by: Sourcery AI <bot@sourcery.ai>
ignition-api · Jul 1, 2020 · eaaac39 · eaaac39
1 parent bbebead
commit eaaac39
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 5 deletions.
diff --git a/python/incendium/gui.py b/python/incendium/gui.py
@@ -72,7 +72,7 @@ def confirm(message, title, show_cancel=False):
 
     return (
         not bool(choice)
-        if choice == JOptionPane.YES_OPTION or choice == JOptionPane.NO_OPTION
+        if choice in [JOptionPane.YES_OPTION, JOptionPane.NO_OPTION]
         else None
     )
 

diff --git a/python/incendium/nav.py b/python/incendium/nav.py
@@ -43,9 +43,7 @@ def _get_full_path(from_path, to_path):
     for path_part in path_parts:
         if path_part == PARENT_DIRECTORY and len(full_path):
             full_path = full_path[:-1]
-        elif path_part == CURRENT_DIRECTORY:
-            pass
-        else:
+        elif path_part != CURRENT_DIRECTORY:
             full_path.append(path_part)
 
     return '/'.join(full_path)

diff --git a/python/system/tag.py b/python/system/tag.py
@@ -376,7 +376,6 @@ def queryTagCalculations(paths, calculations,
     print (paths, calculations, startDate, endDate, rangeHours,
            rangeMinutes, aliases, includeBoundingValues,
            validatesSCExec, noInterpolation, ignoreBadQuality)
-    pass
 
 
 def read(tagPath):