Skip to content

GH-427: Add fix for the issue with multiplierExpression and randomExp…#428

Merged
artembilan merged 10 commits into
spring-projects:mainfrom
aftabshk:fix_expression_issue
May 10, 2024
Merged

GH-427: Add fix for the issue with multiplierExpression and randomExp…#428
artembilan merged 10 commits into
spring-projects:mainfrom
aftabshk:fix_expression_issue

Conversation

@aftabshk

@aftabshk aftabshk commented Apr 27, 2024

Copy link
Copy Markdown
Contributor

This PR fixes the issue explained here: Issue number 427.

Please comment if there is any feedback.

@artembilan artembilan left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thank you for the contribution!
Let me know if my review makes sense for you!

@aftabshk aftabshk requested a review from artembilan May 3, 2024 10:58
Comment thread src/main/java/org/springframework/retry/annotation/Backoff.java Outdated
Comment thread src/main/java/org/springframework/retry/backoff/ExponentialBackOffPolicy.java Outdated
@aftabshk aftabshk requested a review from artembilan May 3, 2024 18:19
Comment thread src/main/java/org/springframework/retry/annotation/Backoff.java
Comment thread src/main/java/org/springframework/retry/annotation/Backoff.java
Comment thread src/test/java/org/springframework/retry/backoff/BackOffPolicyBuilderTests.java Outdated
Comment thread src/test/java/org/springframework/retry/backoff/BackOffPolicyBuilderTests.java Outdated

@artembilan artembilan left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Just a couple minor nit-pick to make the code in the test more cleaner.

Thanks

Comment thread src/test/java/org/springframework/retry/backoff/BackOffPolicyBuilderTests.java Outdated

@artembilan artembilan left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

The build has failed:

 Error:  Failed to execute goal io.spring.javaformat:spring-javaformat-maven-plugin:0.0.40:validate (default) on project spring-retry: Formatting violations found in the following files:
Error:   * /home/runner/work/spring-retry/spring-retry/src/main/java/org/springframework/retry/annotation/Backoff.java
Error:   * /home/runner/work/spring-retry/spring-retry/src/test/java/org/springframework/retry/backoff/BackOffPolicyBuilderTests.java
Error:  
Error:  Run `spring-javaformat:apply` to fix.

When you fix that and build is green, I'll merge.
Thanks

@artembilan artembilan merged commit f94bfff into spring-projects:main May 10, 2024
@artembilan

Copy link
Copy Markdown
Member

Has been merged.

Thank you for contribution; looking forward for more!

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