Skip to content

feat: init uv project for pytest#193

Merged
16bit-ykiko merged 4 commits into
clice-io:mainfrom
Myriad-Dreamin:uv
Aug 25, 2025
Merged

feat: init uv project for pytest#193
16bit-ykiko merged 4 commits into
clice-io:mainfrom
Myriad-Dreamin:uv

Conversation

@Myriad-Dreamin

@Myriad-Dreamin Myriad-Dreamin commented Aug 25, 2025

Copy link
Copy Markdown
Contributor

Then we can run tests using the command:

uv run pytest -s --log-cli-level=INFO tests/integration --executable=./build/bin/clice --resource-dir=".llvm/lib/clang/20" -k "test_clang_tidy"

@star-hengxing star-hengxing requested a review from Copilot August 25, 2025 15:50
@16bit-ykiko

16bit-ykiko commented Aug 25, 2025

Copy link
Copy Markdown
Member

@cursor review

@cursor

cursor Bot commented Aug 25, 2025

Copy link
Copy Markdown

🚨 Bugbot Trial Expired

Your Bugbot trial has expired. Please purchase a license in the Cursor dashboard to continue using Bugbot.

@16bit-ykiko

16bit-ykiko commented Aug 25, 2025

Copy link
Copy Markdown
Member

/gemini review

Copilot AI reviewed Aug 25, 2025

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

gemini-code-assist[bot]
gemini-code-assist Bot reviewed Aug 25, 2025
View reviewed changes

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Code Review

This pull request introduces uv to manage Python dependencies for running tests, which is a great improvement for simplifying the development setup. However, I've found a few critical issues that need to be addressed. The pytest-asyncio dependency is specified with a non-existent version in pyproject.toml, which makes the uv.lock file invalid and will cause dependency installation to fail. Additionally, the updated xmake.lua script contains a bug where a variable is used before it's defined, which will prevent the integration tests from running. Please see the detailed comments for suggestions on how to fix these issues.

Comment thread pyproject.toml
Comment thread uv.lock
Comment thread xmake.lua
@16bit-ykiko 16bit-ykiko merged commit 90d424b into clice-io:main Aug 25, 2025
9 checks passed
@16bit-ykiko 16bit-ykiko changed the title Init uv project for pytest feat: init uv project for pytest Apr 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot Copilot left review comments

+1 more reviewer

@gemini-code-assist gemini-code-assist[bot] gemini-code-assist[bot] left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@Myriad-Dreamin @16bit-ykiko @star-hengxing