bitflicker64 commented on code in PR #366:
URL: https://github.com/apache/hugegraph-ai/pull/366#discussion_r3469873213
##########
.github/workflows/ruff.yml:
##########
@@ -40,7 +40,7 @@ jobs:
- name: Install dev dependencies
run: |
- uv sync --extra dev
+ uv sync --extra dev --extra llm --extra python-client
Review Comment:
thanks for the catch. Fixed in b9286d4:
- Moved continue-on-error: true from the job level to the step level on the
ty check step — so the ty-check job reports green in PR checks and won't block
merge, while the ty diagnostics still surface in CI logs
- The ty-check job matrix stays on 3.10/3.11 only (matching hugegraph-llm's
requires-python)
- Build job reverted to uv sync --extra dev only — no llm deps, no 3.12
conflict
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
