Skip to content

Fix Kotlin code formatting for Inline Maps section of ref docs #26836

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

Conversation

hrybs
Copy link
Contributor

@hrybs hrybs commented Apr 21, 2021

Official documentation contains incorrect Kotlin code example (page 292 of pdf).
There are Map<*, > instead of Map<*, *> and Map<, *> instead of Map<*, *>.
The cause of this issue is incorrect formatting.

@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged or decided on label Apr 21, 2021
@hrybs
Copy link
Contributor Author

hrybs commented Apr 21, 2021

I see that validation failed because of some internal error. Please check it.

@sbrannen
Copy link
Member

Official documentation contains incorrect Kotlin code example (page 292 of pdf).

In the future, please provide a link to the affected section of the reference manual, since the page numbers of the PDF can change -- for example, "in the Inline Maps section". If you don't want to provide a link, stating something like "in the Inline Maps section of the SpEL chapter" is also sufficient.

Thanks

@sbrannen sbrannen self-assigned this Apr 21, 2021
@sbrannen sbrannen added type: task A general task and removed status: waiting-for-triage An issue we've not yet triaged or decided on labels Apr 21, 2021
@sbrannen sbrannen added this to the 5.3.7 milestone Apr 21, 2021
@sbrannen sbrannen changed the title Fix inline map Kotlin code formatting Fix Kotlin code formatting for Inline Maps section of ref docs Apr 21, 2021
@sbrannen sbrannen merged commit b153b5e into spring-projects:master Apr 21, 2021
@sbrannen
Copy link
Member

This has been merged into master.

Thanks

lxbzmy pushed a commit to lxbzmy/spring-framework that referenced this pull request Mar 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: task A general task
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants