Skip to content

Fix/replace pytest anyio with pytest asyncio - #333

Merged
dsarno merged 2 commits into
CoplayDev:mainfrom
dsarno:fix/replace-pytest-anyio-with-pytest-asyncio
Oct 21, 2025
Merged

Fix/replace pytest anyio with pytest asyncio#333
dsarno merged 2 commits into
CoplayDev:mainfrom
dsarno:fix/replace-pytest-anyio-with-pytest-asyncio

Conversation

@dsarno

@dsarno dsarno commented Oct 21, 2025

Copy link
Copy Markdown
Collaborator

fix previous test PR, incorrect pytest dep

Summary by CodeRabbit

  • Bug Fixes

    • Improved branch name display to show actual branch references instead of generic labels.
  • Chores

    • Updated development dependencies for testing framework.
@coderabbitai

coderabbitai Bot commented Oct 21, 2025

Copy link
Copy Markdown
Contributor

Caution

Review failed

The pull request is closed.

Walkthrough

Dependencies updated in the dev group (pytest-anyio replaced with pytest-asyncio). Source configuration enhanced to display dynamic branch names in option labels and adjusted upstream URL scheme from git+https to plain https.

Changes

Cohort / File(s) Summary
Dev Dependencies
MCPForUnity/UnityMcpServer~/src/pyproject.toml
Replaced pytest-anyio>=0.6.0 with pytest-asyncio>=0.23 in optional dev dependencies.
Dynamic Branch Labels & URL Scheme
mcp_source.py
Option labels (2 and 3) now interpolate actual branch names using f-strings instead of static text. Upstream URL changed from git+https scheme to plain https scheme.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Poem

🐰 A rabbit's ode to branches bright,
No more static text in sight!
From git+https to https plain,
pytest-asyncio runs the vein,
Dynamic names make config right! 🌿

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

📜 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 0397887 and eac7c6c.

📒 Files selected for processing (2)
  • MCPForUnity/UnityMcpServer~/src/pyproject.toml (1 hunks)
  • mcp_source.py (2 hunks)

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.

@dsarno
dsarno merged commit e3e5485 into CoplayDev:main Oct 21, 2025
1 check was pending
@dsarno
dsarno deleted the fix/replace-pytest-anyio-with-pytest-asyncio branch October 24, 2025 17:40
choej2303 pushed a commit to choej2303/unity-mcp-ide that referenced this pull request Dec 13, 2025
* mcp_source.py: use MCPForUnity path and show branch in options

* dev(deps): replace pytest-anyio (stub) with pytest-asyncio for tests
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

1 participant