Skip to content

Fix MultipleDiagnosticsLoggers.Parallel deadlock - #17208

Merged
vzarytovskii merged 6 commits into
dotnet:mainfrom
majocha:fix-parallel
May 23, 2024
Merged

Fix MultipleDiagnosticsLoggers.Parallel deadlock#17208
vzarytovskii merged 6 commits into
dotnet:mainfrom
majocha:fix-parallel

Conversation

@majocha

Copy link
Copy Markdown
Contributor

Description

Fixes#17205

Checklist

  • Test case added

@majocha
majocha requested a review from a team as a code ownerMay 21, 2024 22:32
@github-actions

github-actionsBot commented May 21, 2024

Copy link
Copy Markdown
Contributor

⚠️ Release notes required, but author opted out

Warning

Author opted out of release notes, check is disabled for this pull request.
cc @dotnet/fsharp-team-msft

@majocha

Copy link
Copy Markdown
ContributorAuthor

I think this is ready.

@psfinaki

Copy link
Copy Markdown
Contributor

@ForNeVeR maybe take a look if you have a chance :)

@ForNeVeRForNeVeR left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. I cannot imagine a scenario when this breaks (so far!)

@vzarytovskii

Copy link
Copy Markdown
Member

LGTM. I cannot imagine a scenario when this breaks (so far!)

Famous last words, but lgtm too

@vzarytovskiivzarytovskii added the NO_RELEASE_NOTES Label for pull requests which signals, that user opted-out of providing release notes label May 22, 2024
@vzarytovskii

Copy link
Copy Markdown
Member

/run fantomas

@dotnetdotnet deleted a comment from github-actionsBotMay 22, 2024
@dotnetdotnet deleted a comment from psfinakiMay 22, 2024
@psfinaki

Copy link
Copy Markdown
Contributor

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 2 pipeline(s).

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

NO_RELEASE_NOTESLabel for pull requests which signals, that user opted-out of providing release notes

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

MultipleDiagnosticsLoggers.Parallel deadlock

5 participants

@majocha@psfinaki@vzarytovskii@ForNeVeR@T-Gro