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.

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

Consolidate operation and operation_shape modules #2781

Closed
wants to merge 12 commits into from

Conversation

hlbarber
Copy link
Contributor

Motivation and Context

We have two modules operation and operation_shape in the generated server SDK, these can be consolidated.

@hlbarber hlbarber added breaking-change This will require a breaking change server Rust server SDK labels Jun 15, 2023
@hlbarber hlbarber requested review from a team as code owners June 15, 2023 15:21
@github-actions
Copy link

A new generated diff is ready to view.

A new doc preview is ready to view.

@github-actions
Copy link

A new generated diff is ready to view.

A new doc preview is ready to view.

@hlbarber hlbarber enabled auto-merge June 15, 2023 17:50
@hlbarber hlbarber disabled auto-merge June 15, 2023 17:50
@hlbarber hlbarber enabled auto-merge June 15, 2023 17:53
@hlbarber hlbarber disabled auto-merge June 15, 2023 17:53
@hlbarber hlbarber enabled auto-merge June 15, 2023 17:53
@github-actions
Copy link

A new generated diff is ready to view.

A new doc preview is ready to view.

@github-actions
Copy link

A new generated diff is ready to view.

A new doc preview is ready to view.

@github-actions
Copy link

A new generated diff is ready to view.

A new doc preview is ready to view.

@github-actions
Copy link

A new generated diff is ready to view.

A new doc preview is ready to view.

@hlbarber hlbarber force-pushed the harryb/consolidate-operation branch from faf52d3 to a7b5e63 Compare June 17, 2023 10:18
@github-actions
Copy link

A new generated diff is ready to view.

A new doc preview is ready to view.

@github-actions
Copy link

A new generated diff is ready to view.

A new doc preview is ready to view.

@hlbarber
Copy link
Contributor Author

This is blocked by tokio-rs/tracing#2621

@crisidev
Copy link
Contributor

Wow, this is subtle, good catch!

@hlbarber hlbarber disabled auto-merge July 11, 2023 19:40
@hlbarber hlbarber removed the blocked label Jul 11, 2023
@rcoh rcoh requested review from a team as code owners November 14, 2023 02:21
@jmklix jmklix closed this Apr 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaking-change This will require a breaking change server Rust server SDK
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants