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

Console-related cleanup #4076

Merged
merged 1 commit into from
Jun 14, 2023
Merged

Console-related cleanup #4076

merged 1 commit into from
Jun 14, 2023

Conversation

geeknoid
Copy link
Member

@geeknoid geeknoid commented Jun 14, 2023

  • Deprecate the logging console exporter. It's features will eventually be surfaced by Microsoft.Extensions.Logging.Console.

  • Move the latency console code into the main Microsoft.Extensions.Telemetry package which let me delete the Microsoft.Extensions.Telemetry.Console package.

Microsoft Reviewers: Open in CodeFlow

@ghost ghost assigned geeknoid Jun 14, 2023
@geeknoid geeknoid assigned noahfalk and unassigned noahfalk Jun 14, 2023
@geeknoid geeknoid requested a review from noahfalk June 14, 2023 18:32
- Deprecate the logging console exporter. It's features will eventually
be surfaced by Microsoft.Extensions.Logging.Console.

- Move the latency console code into the main
Microsoft.Extensions.Telemetry package which let me delete the
Microsoft.Extensions.Telemetry.Console package.
@geeknoid geeknoid merged commit b92c366 into main Jun 14, 2023
@geeknoid geeknoid deleted the geeknoid/console branch June 14, 2023 20:58
@ghost ghost added this to the 8.0 Preview6 milestone Jun 14, 2023
@RussKie
Copy link
Member

RussKie commented Jul 12, 2023

@geeknoid @rafal-mz I missed this pull request, but essentially any changes in our package offering - i.e., an addition, a removal or name change of a package must be reflected in https://github.com/dotnet/api-catalog-infra/ so that the Docs team has the right content for the API docs. You can use https://github.com/dotnet/api-catalog-infra/pull/67 as a reference.

@ghost ghost locked as resolved and limited conversation to collaborators Aug 13, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants