Merge pull request #27 from unixorn/fix-swiftbar-logging-bug

Deal with Swiftbar change

Author: unixorn

.github/workflows/superlinter.yml

@@ -47,7 +47,7 @@ jobs:
       # Run Linter against code base #
       ################################
       - name: Lint Code Base
-        uses: github/super-linter@v3
+        uses: github/super-linter@v4
         env:
           VALIDATE_ALL_CODEBASE: false
           DEFAULT_BRANCH: main

Changelog.md

@@ -20,4 +20,8 @@
 
 ## 1.3.1
 
-- Add `/opt/homebrew/bin` to the plugin's `$PATH` when it exists and is a directory.
+- Add `/opt/homebrew/bin` to the plugin's `$PATH` when it exists and is a directory.
+
+## 1.3.2
+
+- Swiftbar started capturing `stderr` in addition to `stdout`, causing log messages to spam the menu bar. Default to log level critical so we don't spam log messages (that no one was seeing anyway) except when we're testing.

lima-plugin

@@ -5,7 +5,7 @@
 # Copyright 2021, Joe Block <[email protected]>
 #
 # <xbar.title>Lima Control</xbar.title>
-# <xbar.version>v1.0</xbar.version>
+# <xbar.version>v1.3.2</xbar.version>
 # <xbar.author>Joe Block</xbar.author>
 # <xbar.author.github>unixorn</xbar.author.github>
 # <xbar.desc>Control Lima VM</xbar.desc>
@@ -34,7 +34,7 @@ RUNNING_VM_COLOR = "#29cc00"
 # Stopped VM color (default red)
 STOPPED_VM_COLOR = "#ff0033"
 
-VERSION = "1.3.1"
+VERSION = "1.3.2"
 
 
 def logSetup(level: str = "INFO"):
@@ -72,7 +72,7 @@ def parseCLI():
         type=str.upper,
         help="set log level",
         choices=["DEBUG", "INFO", "ERROR", "WARNING", "CRITICAL"],
-        default="INFO",
+        default="CRITICAL",
     )
     parser.add_argument(
         "--vm", "--virtual-machine", type=str, help="Which vm to use", default="default"
@@ -325,7 +325,7 @@ def imageOps(action: str, image: str, vm: str = "default"):
     :param str image: What image to do the action on
     :param str vm: Which VM is the image in?
     """
-    logging.critical("imageOps")
+    logging.info("imageOps")
     logging.info("action: %s" % action)
     logging.info("image: %s" % image)
     logging.info("vm: %s" % vm)
@@ -371,7 +371,7 @@ def vmOps(action: str, vm: str = "default"):
     :param str action: What action to run - should be start or stop
     :param str vm: Name of VM to act on
     """
-    logging.critical("vmOps")
+    logging.info("vmOps")
     logging.debug("action: %s" % action)
     logging.debug("vm: %s" % vm)