Skip to content

test(tui): include Volcengine in provider picker expectation#2394

Merged
Hmbown merged 1 commit into
mainfrom
codex/fix-provider-picker-volcengine
May 31, 2026
Merged

test(tui): include Volcengine in provider picker expectation#2394
Hmbown merged 1 commit into
mainfrom
codex/fix-provider-picker-volcengine

Conversation

@Hmbown

@Hmbown Hmbown commented May 31, 2026

Copy link
Copy Markdown
Owner

Summary

Verification

  • cargo fmt --all -- --check
  • git diff --check
  • cargo test -p codewhale-tui picker_lists_all_providers -- --nocapture

Copilot AI review requested due to automatic review settings May 31, 2026 07:02

@greptile-apps greptile-apps 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.

Hmbown has reached the 50-review limit for trial accounts. To continue receiving code reviews, upgrade your plan.

@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 adds "Volcengine Ark" to the list of providers in the provider picker tests. There are no review comments, and I have no feedback to provide.

@Hmbown Hmbown merged commit 8ceb67f into main May 31, 2026
13 of 14 checks passed

Copilot AI 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.

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

@Hmbown

Hmbown commented May 31, 2026

Copy link
Copy Markdown
Owner Author

Merged this tiny CI repair to unblock the queue after #1993 added Volcengine Ark. Local targeted proof passed (cargo fmt --all -- --check, git diff --check, and cargo test -p codewhale-tui picker_lists_all_providers -- --nocapture), and the branch checks that had completed were green; this fixes the same provider-picker drift that was causing unrelated macOS PR runs to fail.

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