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

Improve Javadoc for SpringProperties.getFlag() #34295

Closed
wants to merge 1 commit into from

Conversation

remeio
Copy link
Contributor

@remeio remeio commented Jan 21, 2025

getFlag return true when the property is equal, ignoring case, to the string "true", not just "true", "TrUe" also means true.

See more at Boolean#parseBoolean

getFlag return true when the property is equal, ignoring case, to the string "true", not just "true", "TrUe" also means true.

Signed-off-by: Mengqi Xu <2663479778@qq.com>
@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged or decided on label Jan 21, 2025
@sbrannen sbrannen self-assigned this Jan 21, 2025
@sbrannen sbrannen added type: documentation A documentation task in: core Issues in core modules (aop, beans, core, context, expression) and removed status: waiting-for-triage An issue we've not yet triaged or decided on labels Jan 21, 2025
@sbrannen sbrannen added this to the 6.2.3 milestone Jan 21, 2025
@sbrannen sbrannen changed the title Fix SpringProperties#getFlag document. Improve Javadoc for SpringProperties.getFlag() Jan 21, 2025
sbrannen added a commit that referenced this pull request Jan 21, 2025

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
See gh-34295
@sbrannen sbrannen closed this in 016a4c4 Jan 21, 2025
@sbrannen
Copy link
Member

This has been merged in 016a4c4 and revised in 6863b90.

Thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
in: core Issues in core modules (aop, beans, core, context, expression) type: documentation A documentation task
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants