Skip to content

Fix manipulating property sources example in Javadoc for ConfigurableEnvironment #29693

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

Merged
merged 1 commit into from
Dec 16, 2022

Conversation

quim3ra
Copy link
Contributor

@quim3ra quim3ra commented Dec 15, 2022

The Manipulating property sources example is stating the MutablePropertySources expect a Map<String,String> whereas is a Map<String,Object>

The Manipulating property sources example is stating the MutablePropertySources expect a `Map<String,String>` whereas is a `Map<String,Object>`
@pivotal-cla
Copy link

@quim3ra Please sign the Contributor License Agreement!

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

See the FAQ for frequently asked questions.

@pivotal-cla
Copy link

@quim3ra Thank you for signing the Contributor License Agreement!

@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged or decided on label Dec 15, 2022
@quim3ra quim3ra changed the title Manipulating property sources example is wrong Javadoc: Manipulating property sources example is wrong Dec 16, 2022
@sbrannen sbrannen self-assigned this Dec 16, 2022
@sbrannen sbrannen changed the title Javadoc: Manipulating property sources example is wrong Fix manipulating property sources example in Javadoc for ConfigurableEnvironment Dec 16, 2022
@sbrannen sbrannen added type: documentation A documentation task and removed status: waiting-for-triage An issue we've not yet triaged or decided on labels Dec 16, 2022
@sbrannen sbrannen added this to the 6.0.4 milestone Dec 16, 2022
@sbrannen sbrannen merged commit 5d2ca11 into spring-projects:main Dec 16, 2022
@sbrannen
Copy link
Member

Good catch.

This has been merged into main.

Thanks

@sbrannen sbrannen added the in: core Issues in core modules (aop, beans, core, context, expression) label Dec 16, 2022
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.

4 participants