Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Relax bounds on Scoped #2779

Merged
merged 2 commits into from
Jun 15, 2023
Merged

Relax bounds on Scoped #2779

merged 2 commits into from
Jun 15, 2023

Conversation

hlbarber
Copy link
Contributor

Motivation and Context

I accidentally added this bound, it is not only redundant but its absence is one of the key benefits of this approach.

@hlbarber hlbarber added the server Rust server SDK label Jun 15, 2023
@hlbarber hlbarber requested a review from a team as a code owner June 15, 2023 12:12
@hlbarber hlbarber requested a review from a team as a code owner June 15, 2023 12:13
@github-actions
Copy link

A new generated diff is ready to view.

  • No codegen difference in the AWS SDK
  • No codegen difference in the Client Test
  • No codegen difference in the Server Test
  • No codegen difference in the Server Test Python
  • No codegen difference in the Server Test Typescript

A new doc preview is ready to view.

@hlbarber hlbarber added this pull request to the merge queue Jun 15, 2023
Merged via the queue into main with commit 07bd832 Jun 15, 2023
@hlbarber hlbarber deleted the harryb/relax-scoped-bound branch June 15, 2023 13:52
Velfi pushed a commit that referenced this pull request Jun 15, 2023
@hlbarber @Velfi
Relax bounds on Scoped (#2779)
b8844a7 
## Motivation and Context

I accidentally added this bound, it is not only redundant but its
absence is one of the key benefits of this approach.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jjant jjant jjant approved these changes

Assignees
No one assigned
Labels
server Rust server SDK
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@hlbarber @jjant