-
Notifications
You must be signed in to change notification settings - Fork 504
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
GH-395: make the retry context aware of the policies that have a maxi… #396
GH-395: make the retry context aware of the policies that have a maxi… #396
Conversation
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.
I have no objection on this.
I like the code as it is right now in the PR without any further changes.
Thanks
Glad to hear that @artembilan, thank you |
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.
Add your name to the @author
list of the affected classes, please.
Thanks, done. Have added myself as an author to all classes I modified that already had an author list. |
d309296
to
37300f9
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.
Looks like there is a conflict in your branch against the latest main
:
Conflicting files
src/test/java/org/springframework/retry/annotation/EnableRetryTests.java
Please, consider to rebase, fix it and push.
Thanks
…hat have a maximum number of attempts set
…t already had an author list
37300f9
to
5d87091
Compare
Oh yes, I noticed immediately after pushing the update, rebased and fixed the conflicts, thank you. |
thank you for contribution; looking forward for more! |
No description provided.