Skip to content

feat(clients): allow protocol selection by class constructor#7568

Merged
kuhe merged 10 commits intomainfrom
feat/protocol-selection
Dec 16, 2025
Merged

feat(clients): allow protocol selection by class constructor#7568
kuhe merged 10 commits intomainfrom
feat/protocol-selection

Conversation

@kuhe
Copy link
Contributor

@kuhe kuhe commented Dec 11, 2025

Issue

codegen sync for smithy-lang/smithy-typescript#1812

smithy-lang/smithy-typescript@27f444b...a0318e2

Description

Update to allow protocol selection by constructor reference rather than needing to know specifically how to construct a custom protocol.

Testing

new e2e tests, CI

Checklist

  • add e2e test for CloudWatch with ctor based selection

@kuhe kuhe force-pushed the feat/protocol-selection branch 5 times, most recently from d4de8fd to d34bf29 Compare December 15, 2025 20:08
@kuhe kuhe marked this pull request as ready for review December 15, 2025 20:50
@kuhe kuhe requested a review from a team as a code owner December 15, 2025 20:50
@kuhe kuhe force-pushed the feat/protocol-selection branch from 9deebae to 5a322c1 Compare December 16, 2025 02:05
@kuhe
Copy link
Contributor Author

kuhe commented Dec 16, 2025

opened microsoft/rushstack#5507 when I encountered an error with api-extractor in the course of this PR

@kuhe kuhe merged commit 5c5fd2e into main Dec 16, 2025
7 checks passed
@kuhe kuhe deleted the feat/protocol-selection branch December 16, 2025 14:42
@github-actions
Copy link

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs and link to relevant comments in this thread.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Dec 31, 2025
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