Skip to content

ci(cli): run cli tests in windows - #213

Merged
ryanking13 merged 17 commits into
mainfrom
gyeongjae/windows-ci
Aug 21, 2026
Merged

ci(cli): run cli tests in windows#213
ryanking13 merged 17 commits into
mainfrom
gyeongjae/windows-ci

Conversation

@ryanking13

@ryanking13 ryanking13 commented Aug 18, 2026

Copy link
Copy Markdown
Contributor

Run unittest with Windows machine in CI. This is on top of #214

@ask-bonk

ask-bonk Bot commented Aug 18, 2026

Copy link
Copy Markdown

ProviderModelNotFoundError

github run

@ask-bonk

ask-bonk Bot commented Aug 18, 2026

Copy link
Copy Markdown

@ryanking13 Bonk workflow failed. Check the logs for details.

View workflow run · To retry, trigger Bonk again.

@ryanking13
ryanking13 changed the base branch from main to gyeongjae/windows-ci-stack1 August 18, 2026 12:30
@ryanking13
ryanking13 marked this pull request as ready for review August 18, 2026 12:33
@ryanking13
ryanking13 requested review from dom96, hoodmane and joesepi and removed request for hoodmane August 18, 2026 12:33

@dom96 dom96 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.

Looks like it needs a rebase: I'm seeing changes from the PR lower in the stack.

@ryanking13

Copy link
Copy Markdown
Contributor Author

Looks like it needs a rebase: I'm seeing changes from the PR lower in the stack.

Right, I created this PR first and cherry-picked to make a base branch, but it seems like github is not smart enough to understand it.

@ryanking13
ryanking13 force-pushed the gyeongjae/windows-ci branch from 8c3bed5 to 46a03b3 Compare August 19, 2026 04:29
@ryanking13
ryanking13 requested a review from dom96 August 19, 2026 04:34
Base automatically changed from gyeongjae/windows-ci-stack1 to main August 19, 2026 04:36
@ryanking13
ryanking13 force-pushed the gyeongjae/windows-ci branch from 46a03b3 to d5df4a8 Compare August 19, 2026 04:36
@ryanking13

Copy link
Copy Markdown
Contributor Author

Done rebasing.

@dom96 dom96 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.

Nice!

Comment thread packages/cli/tests/test_cli.py
@ryanking13
ryanking13 merged commit 47fbc48 into main Aug 21, 2026
19 checks passed
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