fix: remove testpypi we don't need it in the workflow

[lukeocodes]

Summary by CodeRabbit

• Chores
  • Simplified the release workflow by removing the TestPyPI publishing step; releases now publish directly to PyPI.
  • Eliminated related TestPyPI environment configuration; PyPI publishing logic and error handling remain unchanged.
  • Expected impact: faster availability of releases on PyPI and a leaner pipeline.
  • No changes to application behavior or public interfaces; end-users will see no functional differences.

[coderabbitai]

Walkthrough

The release workflow in .github/workflows/release-please.yml was updated to remove the TestPyPI publishing step and its associated environment variable. The publish job now proceeds directly to PyPI publishing without interacting with TestPyPI. No other workflow logic or error handling was changed.

Changes

Cohort / File(s)	Summary
Release workflow \.github/workflows/release-please.yml	Removed the "Publish to TestPyPI" step, eliminated use of POETRY_PYPI_TOKEN_TESTPYPI, and kept the PyPI publish step unchanged.

Sequence Diagram(s)

sequenceDiagram
  autonumber
  actor Developer
  participant GitHub Actions as Workflow: release-please
  participant PyPI

  Developer->>GitHub Actions: Tag/release event
  Note over GitHub Actions: Build & prepare artifacts
  GitHub Actions->>PyPI: Publish package (poetry publish)
  PyPI-->>GitHub Actions: Publish result
  GitHub Actions-->>Developer: Job status

Loading

sequenceDiagram
  autonumber
  actor Developer
  participant GitHub Actions as Workflow: release-please
  participant TestPyPI
  participant PyPI

  rect rgba(240,240,255,0.6)
  Note over GitHub Actions: Previous flow (removed)
  Developer->>GitHub Actions: Tag/release event
  GitHub Actions->>TestPyPI: Publish to TestPyPI (removed)
  TestPyPI-->>GitHub Actions: Result
  end

  rect rgba(240,255,240,0.6)
  Note over GitHub Actions: Current flow
  GitHub Actions->>PyPI: Publish to PyPI
  PyPI-->>GitHub Actions: Result
  end

Loading

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

• fix: adding publish step to release-please #581 — Also modifies .github/workflows/release-please.yml around TestPyPI publishing, introducing a flow that includes TestPyPI; closely related to this PR’s removal of that step.

Suggested reviewers

• naomi-lgbt

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check	✅ Passed	The title succinctly conveys the removal of the TestPyPI publishing step from the release workflow, which is the main change in this pull request. It follows conventional commit style with the "fix:" prefix, improving consistency and readability. It is specific enough for a reviewer to understand the impact without additional context.
Docstring Coverage	✅ Passed	No functions found in the changes. Docstring coverage check skipped.

✨ Finishing touches

🧪 Generate unit tests

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch lo/remove-pytest

---

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 3c56e9a and 65865ec.

📒 Files selected for processing (1)

• .github/workflows/release-please.yml (0 hunks)

💤 Files with no reviewable changes (1)

• .github/workflows/release-please.yml

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}