Skip to content

Support Resilience4J Configuration Properties#88

Merged
ryanjbaxter merged 4 commits intospring-cloud:masterfrom
Ferioney:issue-61
Feb 18, 2021
Merged

Support Resilience4J Configuration Properties#88
ryanjbaxter merged 4 commits intospring-cloud:masterfrom
Ferioney:issue-61

Conversation

@Ferioney
Copy link
Copy Markdown

included resilience4J spring boot 2 module
Fixes gh-61

included resilience4J spring boot 2 module
Fixes gh-61
Copy link
Copy Markdown
Contributor

@ryanjbaxter ryanjbaxter left a comment

Choose a reason for hiding this comment

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

This includes changes from #87, please separate the two.

@Ferioney Ferioney requested a review from ryanjbaxter February 17, 2021 14:01
Comment thread README.adoc Outdated

==== Properties Configuration

You can configure CircuitBreaker and TimeLimiter instances in Spring Boot’s `application.yml` config file. Property configuration has higher priority that Java `Customizer` configuration.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

"that" -> "than"

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

"in Spring Boot’s application.yml" -> "in your application's configuration properties"

fixed spell mistakes
reverted old constructors
@Ferioney Ferioney requested a review from ryanjbaxter February 17, 2021 17:07
Comment thread README.adoc Outdated
@Ferioney Ferioney requested a review from ryanjbaxter February 18, 2021 08:56
@ryanjbaxter ryanjbaxter changed the title Issue-61 Support Resilience4J Configuration Properties Feb 18, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Resilience4J Configurations from application.yml aren't Supported

3 participants