Skip to content

Improvements + fixes#880

Merged
shawncal merged 1 commit intomicrosoft:feature-serviceregistryfrom
SergeyMenshykh:pass-logger-to-services
May 9, 2023
Merged

Improvements + fixes#880
shawncal merged 1 commit intomicrosoft:feature-serviceregistryfrom
SergeyMenshykh:pass-logger-to-services

Conversation

@SergeyMenshykh
Copy link
Member

Description

This PR does following:

  1. Fixes not compiling code that got broken after the merge with latest bits from the main branch.
  2. Pasing the logger registered in kernel builder to all service registered by With* kernel extension methods.
  3. Removes unnecessary logger parameter from signatures of all With* kernel extension methods.

Contribution Checklist

…m main branch.

2. Pasing default logger that is registered in kernel builder to all service registered with With* KernelExtension methods.
3. Changing signatures of the With* methods so that they don't accept unnessesary logger.
@github-actions github-actions bot added .NET Issue or Pull requests regarding .NET code kernel Issues or pull requests impacting the core kernel kernel.core labels May 9, 2023
@SergeyMenshykh SergeyMenshykh requested a review from shawncal May 9, 2023 15:34
@shawncal shawncal merged commit c8a9a70 into microsoft:feature-serviceregistry May 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

kernel Issues or pull requests impacting the core kernel .NET Issue or Pull requests regarding .NET code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants