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 wording in Javadoc for @Autowired #31235

Merged
merged 1 commit into from
Sep 14, 2023

Conversation

Victory-Canopus
Copy link
Contributor

No description provided.

@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged or decided on label Sep 14, 2023
@sbrannen sbrannen changed the title Fixed Grammatical Errors Improve wording in Javadoc for @Autowired Sep 14, 2023
@@ -30,7 +30,7 @@
* <h3>Autowired Constructors</h3>
* <p>Only one constructor of any given bean class may declare this annotation with the
* {@link #required} attribute set to {@code true}, indicating <i>the</i> constructor
* to autowire when used as a Spring bean. Furthermore, if the {@code required}
* to be autowired when used as a Spring bean. Furthermore, if the {@code required}
Copy link
Member

Choose a reason for hiding this comment

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

It's actually grammatically correct as-is; however, the proposed change may be easier to understand for most readers. So we'll merge the PR to improve the wording, and I've updated the title accordingly.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks! It's my first-ever GitHub contribution.

@sbrannen sbrannen self-assigned this Sep 14, 2023
@sbrannen sbrannen added the type: documentation A documentation task label Sep 14, 2023
@sbrannen sbrannen merged commit 9d7f9b5 into spring-projects:main Sep 14, 2023
@sbrannen
Copy link
Member

This has been merged into main.

Thanks, and congratulations on your first contribution to the Spring Framework! 👍

@sbrannen sbrannen added this to the 6.1.0-RC1 milestone Sep 14, 2023
@snicoll snicoll removed the status: waiting-for-triage An issue we've not yet triaged or decided on label Sep 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: documentation A documentation task
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants