Apply suggestions from code review

Style fixes from PR Co-authored-by: Ben Hoyt <[email protected]>
canonical · May 24, 2024 · 05e1339 · 05e1339
1 parent 045266a
commit 05e1339
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/lib/charms/operator_libs_linux/v2/snap.py b/lib/charms/operator_libs_linux/v2/snap.py
@@ -585,15 +585,15 @@ def ensure(
                 )
                 self._install(channel, cohort, revision)
                 logger.info("The snap installation completed successfully")
-            elif (revision is None) or (revision != self._revision):
+            elif revision is None or revision != self._revision:
                 # The snap is installed, but we are changing it (e.g., switching channels).
                 logger.info(
                     "Refreshing snap %s, revision %s, tracking %s", self._name, revision, channel
                 )
                 self._refresh(channel=channel, cohort=cohort, revision=revision, devmode=devmode)
                 logger.info("The snap refresh completed successfully")
             else:
-                logger.info("Snap refresh was unnecessary %s", self._name)
+                logger.info("Refresh of snap %s was unnecessary", self._name)
 
         self._update_snap_apps()
         self._state = state