Skip to content

remove unnecessary todos#8842

Merged
aibrahim-oai merged 2 commits intomainfrom
remove-unnecessary-todos
Jan 7, 2026
Merged

remove unnecessary todos#8842
aibrahim-oai merged 2 commits intomainfrom
remove-unnecessary-todos

Conversation

@aibrahim-oai
Copy link
Collaborator

// todo(aibrahim): why are we passing model here while it can change?

we update it on each turn with .with_model

//TODO(aibrahim): run CI in release mode.

although it's good to have, release builds take double the time tests take.

// todo(aibrahim): make this async function

we figured out another way of doing this sync

@aibrahim-oai aibrahim-oai merged commit c31960b into main Jan 7, 2026
24 of 26 checks passed
@aibrahim-oai aibrahim-oai deleted the remove-unnecessary-todos branch January 7, 2026 18:43
@github-actions github-actions bot locked and limited conversation to collaborators Jan 7, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants