1.10.1 - Refactor GitHub release task

Also fixes a bug with python-build-once.

Author: vkottler

.github/workflows/python-package.yml

@@ -65,7 +65,7 @@ jobs:
 
       - run: |
           mk python-release owner=vkottler \
-                            repo=vmklib version=1.10.0
+                            repo=vmklib version=1.10.1
         if: |
           matrix.python-version == '3.11'
           && matrix.system == 'ubuntu-latest'

.pylintrc

@@ -1,2 +1,5 @@
+[DESIGN]
+max-args=6
+
 [MESSAGES CONTROL]
 disable=duplicate-code

README.md

@@ -2,11 +2,11 @@
     =====================================
     generator=datazen
     version=3.1.2
-    hash=0bae505c06b01979c4276b236a95630b
+    hash=0c42310385e6f37f284556e25b419929
     =====================================
 -->
 
-# vmklib ([1.10.0](https://pypi.org/project/vmklib/))
+# vmklib ([1.10.1](https://pypi.org/project/vmklib/))
 
 [![python](https://img.shields.io/pypi/pyversions/vmklib.svg)](https://pypi.org/project/vmklib/)
 ![Build Status](https://github.com/vkottler/vmklib/workflows/Python%20Package/badge.svg)

local/variables/package.yaml

@@ -1,5 +1,5 @@
 ---
 major: 1
 minor: 10
-patch: 0
+patch: 1
 entry: mk

pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta:__legacy__"
 
 [project]
 name = "vmklib"
-version = "1.10.0"
+version = "1.10.1"
 description = "Simplify project workflows by standardizing use of GNU Make."
 readme = "README.md"
 requires-python = ">=3.8"

vmklib/__init__.py

@@ -1,7 +1,7 @@
 # =====================================
 # generator=datazen
 # version=3.1.2
-# hash=0e50415f07e485c70acde31abb963f05
+# hash=089d813c9238c490a893adefbd7ad945
 # =====================================
 
 """
@@ -10,4 +10,4 @@
 
 DESCRIPTION = "Simplify project workflows by standardizing use of GNU Make."
 PKG_NAME = "vmklib"
-VERSION = "1.10.0"
+VERSION = "1.10.1"

vmklib/tasks/python/build.py

@@ -14,11 +14,11 @@
 
 # internal
 from vmklib.tasks.args import environ_fallback_split
-from vmklib.tasks.mixins.concrete import ConcreteBuilderMixin
+from vmklib.tasks.mixins.concrete import ConcreteOnceMixin
 from vmklib.tasks.python import PREFIX
 
 
-class PythonBuild(ConcreteBuilderMixin, SubprocessLogMixin):
+class PythonBuild(ConcreteOnceMixin, SubprocessLogMixin):
     """Build a Python package."""
 
     async def run(

vmklib/tasks/release.py

@@ -49,14 +49,15 @@ async def upload_release_asset(
         )
         return loads(result.stdout)  # type: ignore
 
-    async def run(
+    async def release(
         self,
-        inbox: Inbox,
-        outbox: Outbox,
-        *args,
-        **kwargs,
+        cwd: Path,
+        owner: str,
+        repo: str,
+        version: str,
+        dist: str = "dist",
     ) -> bool:  # pragma: nocover
-        """Generate ninja configuration files."""
+        """Create a release."""
 
         # Check for API key in environment.
         if "GITHUB_API_TOKEN" not in os.environ:
@@ -68,12 +69,7 @@ async def run(
         # Set token header.
         ensure_api_token(os.environ["GITHUB_API_TOKEN"])
 
-        cwd: Path = args[0]
-
         # Ensure GitHub parameters are set.
-        owner = kwargs["owner"]
-        repo = kwargs["repo"]
-        version = kwargs["version"]
         if not owner or not repo or not version:
             return False
 
@@ -94,10 +90,27 @@ async def run(
 
         # Use 'Upload a release asset' API to upload all files in the 'dist'
         # directory to the new release.
-        for item in cwd.joinpath(kwargs.get("dist", "dist")).iterdir():
+        for item in cwd.joinpath(dist).iterdir():
             result = await self.upload_release_asset(
                 owner, repo, release_id, item
             )
             self.log.info("Uploaded '%s'.", result["browser_download_url"])
 
         return True
+
+    async def run(
+        self,
+        inbox: Inbox,
+        outbox: Outbox,
+        *args,
+        **kwargs,
+    ) -> bool:  # pragma: nocover
+        """Generate ninja configuration files."""
+
+        return await self.release(
+            args[0],
+            kwargs["owner"],
+            kwargs["repo"],
+            kwargs["version"],
+            dist=kwargs.get("dist", "dist"),
+        )