Skip to content

Comments

fix: resolve pre-existing CI failures#578

Merged
vivekkalyan merged 1 commit intomainfrom
fix/ci-failures
Feb 24, 2026
Merged

fix: resolve pre-existing CI failures#578
vivekkalyan merged 1 commit intomainfrom
fix/ci-failures

Conversation

@vivekkalyan
Copy link
Collaborator

@vivekkalyan vivekkalyan commented Feb 24, 2026

Summary

  • ruff format: src/art/cli.py and src/art/tinker/server.py
  • ty type error in src/art/tinker/server.py:64: The prompt_tokens method had return type list[int] but apply_chat_template returns a union type (str | list[int] | list[str] | list[list[int]] | BatchEncoding). Wrapped the return value with cast(list[int], ...) since the function is always called with parameters that produce list[int] output.

Copy link
Collaborator

@bradhilton bradhilton left a comment

Choose a reason for hiding this comment

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

thank you 🙏 sorry for leaving these ci issues on main 😬

@vivekkalyan vivekkalyan merged commit 524feb0 into main Feb 24, 2026
2 checks passed
@vivekkalyan vivekkalyan deleted the fix/ci-failures branch February 24, 2026 23:03
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