-
Notifications
You must be signed in to change notification settings - Fork 41.5k
Polish README.adoc #19817
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
Polish README.adoc #19817
Conversation
Updated minor grammer mistakes
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the PR. I've made some suggestions, can you have a look?
README.adoc
Outdated
@@ -233,5 +233,5 @@ Boot step-by-step: | |||
|
|||
|
|||
== License | |||
Spring Boot is Open Source software released under the | |||
Spring Boot is an Open Source software released under the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I believe the current phrasing is correct.
README.adoc
Outdated
@@ -102,7 +102,7 @@ need JDK 1.8. | |||
$ ./gradlew publishToMavenLocal | |||
---- | |||
|
|||
This will build all of the jars and documentation and publish every to your local | |||
This will build all of the jars and documentation and publish it to your local |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
isn't them
more appropriate here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@snicoll true
Updated the files as suggested by @snicoll
Thanks for the follow-up @Abdullah8006 and for making your first contribution to Spring Boot. |
Updated minor grammar mistakes in the README.adoc file.