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

[Instrumentation.Hangfire] drop default parameter from registration method #1129

Merged

Conversation

Kielek
Copy link
Contributor

@Kielek Kielek commented Apr 4, 2023

Changes

drop default parameter from registration method

For significant contributions please make sure you have completed the following items:

  • Appropriate CHANGELOG.md updated for non-trivial changes
  • [ ] Design discussion issue #
  • Changes in public API reviewed

@Kielek Kielek added the comp:instrumentation.hangfire Things related to OpenTelemetry.Instrumentation.Hangfire label Apr 4, 2023
@Kielek Kielek force-pushed the i-hangfire-remove-default-parameters branch from 6c7dcd7 to 807b226 Compare April 4, 2023 08:08
@Kielek
Copy link
Contributor Author

Kielek commented Apr 4, 2023

@fred2u. could you please check?

@Kielek Kielek marked this pull request as ready for review April 4, 2023 18:26
@Kielek Kielek requested a review from a team April 4, 2023 18:26
@Kielek Kielek merged commit a50e1cd into open-telemetry:main Apr 6, 2023
@Kielek Kielek deleted the i-hangfire-remove-default-parameters branch April 6, 2023 04:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
comp:instrumentation.hangfire Things related to OpenTelemetry.Instrumentation.Hangfire
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants