Skip to content

Comments

Complete poetry to uv migration: fix test runner and workflow#41

Merged
KyleKing merged 2 commits intomainfrom
copilot/fix-ci-failures-and-migrate
Jan 30, 2026
Merged

Complete poetry to uv migration: fix test runner and workflow#41
KyleKing merged 2 commits intomainfrom
copilot/fix-ci-failures-and-migrate

Conversation

Copy link
Contributor

Copilot AI commented Jan 29, 2026

Incomplete migration from poetry to uv caused CI test failures (FileNotFoundError: 'poetry') and a workflow warning about an invalid os input parameter.

Changes

Test infrastructure:

  • tests/helpers.py: Changed CTT_CMD from ['poetry', 'run', 'ctt'] to ['uv', 'run', 'ctt']

CI workflow (.github/workflows/usage_test.yml):

  • Removed invalid os input parameter (not accepted by setup action)
  • Changed command from poetry run ctt to uv run ctt
  • Updated trigger paths from poetry.lock to uv.lock

Documentation:

  • Updated example in docs/README.md from poetry run pytest to uv run pytest

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • astral.sh
    • Triggering command: /usr/bin/curl curl -LsSf REDACTED (dns block)

If you need me to access, download, or install something from one of these locations, you can either:


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

- Replace 'poetry run' with 'uv run' in tests/helpers.py
- Update .github/workflows/usage_test.yml to use uv and remove invalid 'os' input
- Update documentation to use 'uv run' instead of 'poetry run'
- Change poetry.lock reference to uv.lock in workflow trigger paths

Co-authored-by: KyleKing <3784339+KyleKing@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix CI failures and migrate usage test to uv Complete poetry to uv migration: fix test runner and workflow Jan 29, 2026
Copilot AI requested a review from KyleKing January 29, 2026 14:03
@KyleKing KyleKing marked this pull request as ready for review January 29, 2026 16:38
@coderabbitai
Copy link

coderabbitai bot commented Jan 29, 2026

Important

Review skipped

Bot user detected.

To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

  • 🔍 Trigger a full review

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

@KyleKing KyleKing merged commit 9579a46 into main Jan 30, 2026
4 of 6 checks passed
@KyleKing KyleKing deleted the copilot/fix-ci-failures-and-migrate branch January 30, 2026 14:16
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.

2 participants