Refactor deployment checks and update dependencies

- Refactor `wait_for_deploy` to accept `expected_version` argument
- Update callers in `bot.py` to pass the version.
- Update tests in `wait_for_deploy_test.py` and `bot_test.py`.
- Replace deprecated `pkg_resources` with `packaging` for version parsing.
- Improve Python package publishing in `publish.py`:
  - Use `python -m build .` for building sdist and wheel.
  - Use `check_call` to ensure build errors are raised.
  - Fix virtual environment handling to ensure `build` module is found.
- Update GitHub Actions CI workflow to use `uv` instead of `pip`.
- Address various Pylint errors in `publish.py` and `wait_for_deploy.py`.

Author: blarghmatey

.github/workflows/ci.yml

@@ -2,38 +2,28 @@ name: CI
 on: [push]
 jobs:
   python-tests:
-    runs-on: ubuntu-20.04
+    runs-on: ubuntu-latest
 
     steps:
       - uses: actions/checkout@v2
 
       - name: Set up JS requirements
         run: npm install
 
-      - name: Set up Python
-        uses: actions/setup-python@v2
-        with:
-          python-version: "3.9"
+      - name: Install uv
+        run: curl -LsSf https://astral.sh/uv/install.sh | sh
 
-      - id: cache
-        uses: actions/cache@v1
-        with:
-          path: ~/.cache/pip
-          key: ${{ runner.os }}-pip-${{ hashFiles('**/requirements.txt', '**/test_requirements.txt') }}
-          restore-keys: |
-            ${{ runner.os }}-pip-
-
-      - name: Install dependencies
-        run: pip install -r requirements.txt -r test_requirements.txt
+      - name: Install Python dependencies with uv
+        run: uv sync --locked
 
       - name: Lint
-        run: pylint *.py
+        run: uv run pylint *.py
 
       - name: Black
-        run: black --version && black . --check
+        run: uv run black --version && uv run black . --check
 
       - name: Tests
-        run: pytest . && coverage xml
+        run: uv run pytest . && uv run coverage xml
 
       # - name: Upload coverage to CodeCov
       #   uses: codecov/codecov-action@v1

bot_test.py

@@ -443,6 +443,7 @@ async def test_release(
         repo_url=test_repo.repo_url,
         hash_url=test_repo.rc_hash_url,
         watch_branch="release-candidate",
+        expected_version=pr.version,
         timeout_seconds=3600,
     )
     assert doof.said("Now deploying to RC...")
@@ -518,6 +519,7 @@ async def test_hotfix_release(
         repo_url=test_repo.repo_url,
         hash_url=test_repo.rc_hash_url,
         watch_branch="release-candidate",
+        expected_version=pr.version,
         timeout_seconds=3600,
     )
     assert doof.said("Now deploying to RC...")
@@ -1216,6 +1218,7 @@ async def test_wait_for_deploy_rc(
         repo_url=test_repo.repo_url,
         hash_url=test_repo.rc_hash_url,
         watch_branch="release-candidate",
+        expected_version=release_pr.version,
         timeout_seconds=3600,
     )
     get_unchecked.assert_called_once_with(
@@ -1236,26 +1239,17 @@ async def test_wait_for_deploy_prod(
 ):  # pylint: disable=unused-argument
     """Bot._wait_for_deploy_prod should wait until repo has been deployed to production"""
     wait_for_deploy_mock = mocker.async_patch("bot.wait_for_deploy")
-    version = "1.2.345"
-    get_version_tag_mock = mocker.async_patch(
-        "bot.get_version_tag", return_value=f"v{version}"
-    )
     channel_id = test_repo.channel_id
     release_pr = ReleasePR(
-        "version", "https://github.com/org/repo/pulls/123456", "body", 123456, False
+        "1.2.345", "https://github.com/org/repo/pulls/123456", "body", 123456, False
     )
 
     await doof._wait_for_deploy_prod(  # pylint: disable=protected-access
         repo_info=test_repo, manager="me", release_pr=release_pr
     )
 
-    get_version_tag_mock.assert_called_once_with(
-        github_access_token=GITHUB_ACCESS,
-        repo_url=test_repo.repo_url,
-        commit_hash="origin/release",
-    )
     assert doof.said(
-        f"My evil scheme v{version} for {test_repo.name} has been released "
+        f"My evil scheme {release_pr.version} for {test_repo.name} has been released "  # Use release_pr.version
         f"to production at {remove_path_from_url(test_repo.prod_hash_url)}. "
         "And by 'released', I mean completely...um...leased.",
         channel_id=channel_id,
@@ -1265,6 +1259,7 @@ async def test_wait_for_deploy_prod(
         repo_url=test_repo.repo_url,
         hash_url=test_repo.prod_hash_url,
         watch_branch="release",
+        expected_version=release_pr.version,  # Pass expected_version from PR
         timeout_seconds=3600,
     )
 

publish.py

@@ -3,7 +3,6 @@
 from pathlib import Path
 
 from async_subprocess import (
-    call,
     check_call,
 )
 from constants import (
@@ -27,13 +26,6 @@ async def upload_to_pypi(project_dir):
         # Heroku has both Python 2 and 3 installed but the system libraries aren't configured for our use,
         # so make a virtualenv.
         async with virtualenv("python3", outer_environ) as (virtualenv_dir, environ):
-            # Use the virtualenv binaries to act within that environment
-            pip_path = os.path.join(virtualenv_dir, "bin", "pip")
-
-            # Install dependencies. wheel is needed for Python 2. twine uploads the package.
-            await check_call(
-                [pip_path, "install", "twine"], env=environ, cwd=project_dir
-            )
             await upload_with_twine(
                 project_dir=project_dir, virtualenv_dir=virtualenv_dir, environ=environ
             )
@@ -55,13 +47,24 @@ async def upload_with_twine(
         "TWINE_USERNAME": os.environ["PYPI_USERNAME"],
         "TWINE_PASSWORD": os.environ["PYPI_PASSWORD"],
     }
-
+    # Use the virtualenv binaries to act within that environment
+    pip_path = os.path.join(virtualenv_dir, "bin", "pip")
     python_path = os.path.join(virtualenv_dir, "bin", "python")
+    # Install dependencies. wheel is needed for Python 2. twine uploads the package.
+    await check_call(
+        [pip_path, "install", "setuptools"], env=environ, cwd=project_dir
+    )
+    await check_call(
+        [pip_path, "install", "build"], env=environ, cwd=project_dir
+    )
+    await check_call(
+        [pip_path, "install", "twine"], env=environ, cwd=project_dir
+    )
     twine_path = os.path.join(virtualenv_dir, "bin", "twine")
 
-    # Create source distribution and wheel.
-    await call([python_path, "setup.py", "sdist"], env=environ, cwd=project_dir)
-    await call([python_path, "setup.py", "bdist_wheel"], env=environ, cwd=project_dir)
+    # Create source distribution and wheel using the virtualenv's python executable
+    # and explicitly passing the virtualenv's environment.
+    await check_call([python_path, "-m", "build", "."], env=environ, cwd=project_dir)
     dist_files = os.listdir(os.path.join(project_dir, "dist"))
     if len(dist_files) != 2:
         raise Exception("Expected to find one tarball and one wheel in directory")

publish_test.py

@@ -54,7 +54,7 @@ def _call(command, *args, **kwargs):
             virtualenv_dir=virtualenv_dir,
             environ=environ,
         )
-    assert call_mock.call_count == 1
+    assert call_mock.call_count == 5
 
 
 async def test_upload_to_npm(mocker, test_repo_directory, library_test_repo):

pyproject.toml

@@ -0,0 +1,28 @@
+[project]
+name = "doof"
+requires-python = "~=3.13"
+version = "0.0.1"
+dependencies = [
+    "python-dateutil",
+    "pytz",
+    "requests",
+    "sentry-sdk",
+    "setuptools>=78.1.0",
+    "tornado",
+    "virtualenv",
+]
+
+[dependency-groups]
+dev = [
+    "black==22.3.0",
+    "codecov",
+    "pdbpp",
+    "pylint",
+    "pytest",
+    "pytest-asyncio",
+    "pytest-cov",
+    "pytest-mock",
+]
+
+[tool.uv]
+package = false

release.py

@@ -4,7 +4,7 @@
 import os
 from subprocess import CalledProcessError
 
-from pkg_resources import parse_version
+from packaging.version import parse as parse_version
 
 from async_subprocess import (
     call,