Skip to content

chore: integrate poe for task management in CI/CD workflows#110

Merged
twangodev merged 1 commit intomainfrom
feat/poe
Feb 13, 2026
Merged

chore: integrate poe for task management in CI/CD workflows#110
twangodev merged 1 commit intomainfrom
feat/poe

Conversation

@twangodev
Copy link
Collaborator

@twangodev twangodev commented Feb 13, 2026

Summary by CodeRabbit

  • Chores
    • Updated CI/CD workflows to use Poe task runner for consistent project automation and simplified command management across formatting, linting, type checking, testing, documentation generation, and builds.

Copilot AI review requested due to automatic review settings February 13, 2026 20:55
@codecov
Copy link

codecov bot commented Feb 13, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@coderabbitai
Copy link

coderabbitai bot commented Feb 13, 2026

📝 Walkthrough

Walkthrough

This PR introduces Poe task runner as a unified command interface, replacing direct tool invocations in CI workflows with uv run poe commands, and configuring task definitions in pyproject.toml for linting, formatting, type-checking, testing, documentation, and building.

Changes

Cohort / File(s) Summary
CI Workflow Commands
.github/workflows/docs.yml, .github/workflows/python.yml
Replaces direct tool invocations (ruff format, ruff check, pytest, uv build, etc.) with equivalent uv run poe task commands across all workflow steps.
Project Configuration
pyproject.toml
Adds poethepoet>=0.32.0 to dev dependencies and introduces [tool.poe.tasks] configuration block defining lint, format, typecheck, test, test-integration, and build tasks.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Poem

🐰 Hop and bound, the tasks align,
Poe's commands now brightly shine,
From scattered tools to one command true,
Workflows dance with structured brew! ✨

🚥 Pre-merge checks | ✅ 4
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and specifically describes the main change: integrating Poe for task management in CI/CD workflows, which aligns perfectly with the modifications across workflow files and pyproject.toml.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Merge Conflict Detection ✅ Passed ✅ No merge conflicts detected when merging into main

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch feat/poe

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

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

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR integrates poethepoet (poe) as a task runner to standardize and centralize command execution in CI/CD workflows. It replaces direct tool invocations with poe task references, making the build commands more maintainable and consistent across different environments.

Changes:

  • Added poethepoet dependency (>=0.32.0) to dev dependencies with version-specific installations (0.37.0 for Python <3.10, 0.41.0 for Python >=3.10)
  • Defined poe tasks in pyproject.toml for common operations (lint, format, typecheck, test, docs, build)
  • Updated GitHub Actions workflows to use poe commands instead of direct tool invocations

Reviewed changes

Copilot reviewed 3 out of 4 changed files in this pull request and generated 1 comment.

File Description
uv.lock Added poethepoet package entries with version-specific markers and its pastel dependency
pyproject.toml Added poethepoet to dev dependencies and configured poe tasks for linting, formatting, testing, and building
.github/workflows/python.yml Replaced direct tool commands with poe task references in CI workflow
.github/workflows/docs.yml Replaced pydoc-markdown command with poe docs task

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

🤖 Fix all issues with AI agents
In `@pyproject.toml`:
- Around line 116-134: The test POE task in pyproject.toml only collects
coverage for fishaudio; update the "test" task definition to include coverage
for the second package by adding the missing flag --cov=src/fish_audio_sdk to
the test command so both --cov=src/fishaudio and --cov=src/fish_audio_sdk are
passed to pytest; locate the "test" task entry under [tool.poe.tasks] and append
the additional --cov flag to the existing command string.

@twangodev twangodev merged commit daf00fe into main Feb 13, 2026
32 checks passed
@twangodev twangodev deleted the feat/poe branch February 13, 2026 21:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant