-
Notifications
You must be signed in to change notification settings - Fork 869
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
Fixes duplicate/bad confirmations retry logic - 1.7.x #5262
Conversation
38f5b9b
to
e13914b
Compare
macOS actually ran great here 🎉 there was a "Publish xUnit Test" failure which is not a problem. iOS failed though, timing out during fetch |
Passed on all except iOS; marking skip for others and re-running |
Looks like |
Just an FYI, #5260 hasn't landed in |
e13914b
to
720ef1c
Compare
720ef1c
to
29ef95e
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Uplift into 1.7.x
approved after deliberating with @brave/uplift-approvers. Please make sure that all the correct labels are being used and the associated issue is moved into the correct milestone.
Uplift of https://github.com/brave/brave-core/pull/5260/files
Resolves brave/brave-browser#9242
After you merge: