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

Simplify Http *TelemetryBuilder generic signatures #12858

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

trask
Copy link
Member

@trask trask commented Dec 9, 2024

I'm not sure that these complex generic signatures are really unlocking anything useful for users...

@github-actions github-actions bot added the test native This label can be applied to PRs to trigger them to run native tests label Dec 9, 2024
@github-actions github-actions bot requested a review from theletterf December 9, 2024 02:08
@trask trask force-pushed the simplify-signatures branch 5 times, most recently from 9b82090 to d8019d4 Compare December 9, 2024 05:50
@trask trask changed the title [DRAFT] Simplify signatures [DRAFT] Implement common HttpClientTelemetryBuilder and HttpServerTelemetryBuilder Dec 9, 2024
@trask trask force-pushed the simplify-signatures branch 5 times, most recently from 96f4e96 to 54cd5fb Compare December 10, 2024 04:01
@trask trask changed the title [DRAFT] Implement common HttpClientTelemetryBuilder and HttpServerTelemetryBuilder Simplify Http *TelemetryBuilder generic signatures Dec 10, 2024
@trask trask force-pushed the simplify-signatures branch 4 times, most recently from 2770c88 to 7795dec Compare December 10, 2024 04:24
@trask trask force-pushed the simplify-signatures branch 6 times, most recently from 5796077 to 330b147 Compare December 17, 2024 03:05
@trask trask force-pushed the simplify-signatures branch from 330b147 to 6465c74 Compare December 17, 2024 04:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
test native This label can be applied to PRs to trigger them to run native tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant