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

[5.2] Port PR 2896 (Allow errors with negative numbers to be considered transient) #3089

Closed

Conversation

edwardneal
Copy link
Contributor

Port of #2896.
Fixes #2890 in v5.2.x.

If possible, could this be rolled into the same 5.2 release as #3076 please?

@mdaigle mdaigle requested a review from a team January 7, 2025 18:30
@cheenamalhotra
Copy link
Member

/azp run

@cheenamalhotra cheenamalhotra changed the title 5.2 | Port PR 2896 (Allow errors with negative numbers to be considered transient) [5.2] Port PR 2896 (Allow errors with negative numbers to be considered transient) Jan 17, 2025
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@cheenamalhotra cheenamalhotra added this to the 5.2.3 milestone Jan 17, 2025
@mdaigle
Copy link
Contributor

mdaigle commented Feb 6, 2025

/azp run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@benrr101
Copy link
Contributor

@edwardneal I must've been out of my mind yesterday b/c I thought I responded to this, but I guess I did not. Same thing applies to this PR as did to the other one we did recently. So I am taking this PR into dotnet repo and re-submitting it, so we can get all the CI to work.

Sorry this is so confusing (or maybe I'm just projecting). I (we) definitely appreciate you help in getting some of these little things off the heap of projects we need to do.

Closing in favor of #3185

@benrr101 benrr101 closed this Feb 27, 2025
@edwardneal
Copy link
Contributor Author

Thanks @benrr101, I was expecting the same thing for this PR so there's no confusion. It'll be good to get this merged.

@edwardneal edwardneal deleted the port/5.2/issue-2890 branch February 28, 2025 06:33
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.

5 participants