Thanks to visit codestin.com
Credit goes to github.com

Skip to content

fix: defaultValue in additional-spring-configuration-metadata.json#1041

Merged
OlgaMaciaszek merged 1 commit intospring-cloud:4.1.xfrom
dowenliu-xyz:main
Sep 4, 2024
Merged

fix: defaultValue in additional-spring-configuration-metadata.json#1041
OlgaMaciaszek merged 1 commit intospring-cloud:4.1.xfrom
dowenliu-xyz:main

Conversation

@dowenliu-xyz
Copy link
Contributor

defaultValue of spring.cloud.openfeign.circuitbreaker.alphanumeric-ids.enabled is true since version 4.0.0

…eric-ids.enabled` is true since version 4.0.0
Copy link
Collaborator

@OlgaMaciaszek OlgaMaciaszek left a comment

Choose a reason for hiding this comment

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

Hi @dowenliu-xyz. Thanks for submitting the PR. Looks good, however, could you please submit it against 4.1.x instead of main? Like this, the change will also be included in the 2023.0.x release train.

@dowenliu-xyz
Copy link
Contributor Author

Ok, it's fine. And I've noticed that I have make this change base on main branch. Is it ok?

@dowenliu-xyz dowenliu-xyz changed the base branch from main to 4.1.x September 4, 2024 10:18
@OlgaMaciaszek
Copy link
Collaborator

Ok, it's fine. And I've noticed that I have make this change base on main branch. Is it ok?

Yes, looks fine.

Copy link
Collaborator

@OlgaMaciaszek OlgaMaciaszek left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks.

@OlgaMaciaszek OlgaMaciaszek merged commit 027a7b7 into spring-cloud:4.1.x Sep 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

No open projects
Status: Done
Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants