Too many __version__ implementations, picked one.

Fixes issue #158 which was a result of trying to stuff a module
type into a string, specifically `busylight.__version__`. That
version was a duplicate of the code in `busylight.__init__` and
a residual from how versioning was handled in the project
Makefile. I think. I've slept since then. Keeping the __version__
in __init__ and removing __version__. Fixed the imports in
__main__, busylight_api and test_cli to reflect the new reality.

Author: JnyJny

busylight/__main__.py

@@ -14,7 +14,7 @@
 from .color import ColorLookupError, ColorTuple, parse_color_string
 from .effects import Effects
 from .manager import LightManager
-from .__version__ import __version__
+from . import __version__
 
 
 cli = typer.Typer()
@@ -368,7 +368,7 @@ def serve_http_api(
         help="Host name to bind the server to.",
     ),
     port: int = typer.Option(
-        21169,
+        8000,
         "--port",
         "-p",
         help="Network port number to listen on.",

busylight/api/busylight_api.py

@@ -141,7 +141,10 @@ async def startup():
 
 @busylightapi.on_event("shutdown")
 async def shutdown():
-    await busylightapi.off()
+    try:
+        await busylightapi.off()
+    except Exception as error:
+        logger.debug("problem during shutdown: {error}")
 
 
 ## Exception Handlers

tests/functional/test_cli.py

@@ -6,7 +6,7 @@
 from typer.testing import CliRunner
 
 from busylight.__main__ import cli
-from busylight.__version__ import __version__
+from busylight import __version__
 from busylight.lights import USBLight
 
 runner = CliRunner()