Skip to content

Conversation

@knjiang
Copy link
Contributor

@knjiang knjiang commented Dec 21, 2025

No description provided.

Copy link
Contributor Author

knjiang commented Dec 21, 2025

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

This stack of pull requests is managed by Graphite. Learn more about stacking.

@knjiang knjiang force-pushed the 12-21-add_cross-transformation-coverage_test_and_google_bedrock_payloads_ branch 2 times, most recently from bed5032 to 6bc7926 Compare December 21, 2025 16:16
Comment on lines 19 to 40
/// Provider definitions - ADD NEW PROVIDERS HERE
/// (directory_name, display_name, provider_format)
const PROVIDERS: &[(&str, &str, ProviderFormat)] = &[
(
"chat-completions",
"ChatCompletions",
ProviderFormat::OpenAI,
),
("anthropic", "Anthropic", ProviderFormat::Anthropic),
("responses", "Responses", ProviderFormat::OpenAI),
("google", "Google", ProviderFormat::Google),
("bedrock", "Bedrock", ProviderFormat::Converse),
];
Copy link
Contributor Author

Choose a reason for hiding this comment

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

the main place for adding new providers to our compatability matrix

@knjiang knjiang marked this pull request as ready for review December 21, 2025 16:33
@knjiang knjiang force-pushed the 12-21-add_cross-transformation-coverage_test_and_google_bedrock_payloads_ branch 5 times, most recently from f5ace44 to c9b537f Compare December 22, 2025 16:10
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