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

Removed obsolete options #2841

Merged
merged 3 commits into from
Nov 16, 2023
Merged

Conversation

bitsandfoxes
Copy link
Contributor

@bitsandfoxes bitsandfoxes commented Nov 15, 2023

Removed options marked as [Obsolete].

@@ -270,7 +270,7 @@ public void AddInAppExclude(string prefix)
public bool EnableAndroidSdkTracing { get; set; } = false;

/// <summary>
/// Gets or sets a value that indicates if the <see cref="BeforeSend"/> callback will be invoked for
/// Gets or sets a value that indicates if the <see cref="o:SetBeforeSend"/> callback will be invoked for
Copy link
Collaborator

@vaind vaind Nov 15, 2023

Choose a reason for hiding this comment

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

IMO, It's still "the BeforeSend callback", but it's set with SetBeforeSend.

Suggested change
/// Gets or sets a value that indicates if the <see cref="o:SetBeforeSend"/> callback will be invoked for
/// Gets or sets a value that indicates if the <see cref="o:BeforeSend"/> callback will be invoked for

Copy link
Collaborator

@jamescrosswell jamescrosswell left a comment

Choose a reason for hiding this comment

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

Nice! So good to delete code 😁

@bitsandfoxes bitsandfoxes merged commit 2a60c09 into feat/4.0.0 Nov 16, 2023
6 of 7 checks passed
@bitsandfoxes bitsandfoxes deleted the fix/remove-obsolete-options branch November 16, 2023 09:41
Copy link
Contributor

Fails
🚫 Please consider adding a changelog entry for the next release.

Instructions and example for changelog

Please add an entry to CHANGELOG.md to the "Unreleased" section. Make sure the entry includes this PR's number.

Example:

## Unreleased

- Removed obsolete options ([#2841](https://github.com/getsentry/sentry-dotnet/pull/2841))

If none of the above apply, you can opt out of this check by adding #skip-changelog to the PR description.

Generated by 🚫 dangerJS against 7accb28

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.

3 participants