Skip to content
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

External configuration documentation uses incorrect placeholder syntax #31941

Closed
wants to merge 1 commit into from

Conversation

vilmos
Copy link
Contributor

@vilmos vilmos commented Aug 1, 2022

Property placeholder format is not "{property}" but "${property}".

Fix property placeholder
@pivotal-cla
Copy link

@vilmos Please sign the Contributor License Agreement!

Click here to manually synchronize the status of this Pull Request.

See the FAQ for frequently asked questions.

@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Aug 1, 2022
@pivotal-cla
Copy link

@vilmos Thank you for signing the Contributor License Agreement!

@philwebb philwebb added type: documentation A documentation update and removed status: waiting-for-triage An issue we've not yet triaged labels Aug 1, 2022
@philwebb philwebb added this to the 2.6.x milestone Aug 1, 2022
@philwebb philwebb changed the title Update external-config.adoc External configuration documentation uses incorrect placeholder syntax Aug 1, 2022
@philwebb philwebb self-assigned this Aug 1, 2022
@philwebb
Copy link
Member

philwebb commented Aug 1, 2022

Thanks very much @vilmos for making your first contribution to Spring Boot!

@philwebb philwebb modified the milestones: 2.6.x, 2.6.11 Aug 1, 2022
philwebb pushed a commit that referenced this pull request Aug 1, 2022
@philwebb philwebb closed this in 66a93a3 Aug 1, 2022
@vilmos vilmos deleted the patch-1 branch September 28, 2022 10:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: documentation A documentation update
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants