-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
ExtensionDefaultConfigFunc should be ExtensionCreateDefaultConfigFunc #5451
Conversation
ExtensionDefaultConfigFunc should be ExtensionCreateDefaultConfigFunc
Please sign the CLA |
Add a "changelog" entry under "Deprecation" |
Codecov Report
@@ Coverage Diff @@
## main #5451 +/- ##
==========================================
- Coverage 90.89% 90.85% -0.04%
==========================================
Files 191 191
Lines 11419 11421 +2
==========================================
- Hits 10379 10377 -2
- Misses 819 822 +3
- Partials 221 222 +1
Continue to review full report at Codecov.
|
Thanks with this |
This resolved the issue#5437, and better align with the names of following types from package component :
ExporterCreateDefaultConfigFunc
ProcessorCreateDefaultConfigFunc
ReceiverCreateDefaultConfigFunc
ExtensionDefaultConfigFunc