Skip to content

chore: remove workaround for sentry-dotnet issue #3141 #85

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

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

vaind
Copy link
Collaborator

@vaind vaind commented Jul 11, 2025

Should be fixed now by getsentry/sentry-dotnet#4323

We need to wait for a release to nuget in order to verify.

@vaind vaind force-pushed the chore/remove-on-exit-flush-workaround branch from 4b8222f to 9a1d9ba Compare July 15, 2025 05:59
@vaind vaind marked this pull request as ready for review July 15, 2025 06:04
@vaind vaind mentioned this pull request Jul 15, 2025
@@ -2,6 +2,10 @@

## Unreleased

### Fixes

- Remove workaround for Dotnet SDK issue causing a hang on process exit - automatic flush should work now. ([#85](https://github.com/getsentry/sentry-powershell/pull/85))
Copy link
Member

Choose a reason for hiding this comment

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

suggestion: Swap the two parts of the note so that the actual fix/improvement comes first.

e.g.

Suggested change
- Remove workaround for Dotnet SDK issue causing a hang on process exit - automatic flush should work now. ([#85](https://github.com/getsentry/sentry-powershell/pull/85))
- Automatic flush by removing workaround for Dotnet SDK issue causing a hang on process exit. ([#85](https://github.com/getsentry/sentry-powershell/pull/85))

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants