-
Notifications
You must be signed in to change notification settings - Fork 41.6k
Editing pass for production-ready-features.adoc #10830
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
Editing pass for production-ready-features.adoc #10830
Conversation
I made it more formal and corrected sentence errors.
wilkinsona
left a comment
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.
Spotted three things that I don't think are particularly contentious. I'll take care of them as part of merging this.
| TIP: Generated passwords are logged as the application starts. Search for '`Using default | ||
| security password`'. | ||
| TIP: Generated passwords are logged as the application starts. To find the password in | ||
| the console, search it for '`Using default security password`'. |
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.
"it for" -> "for"?
|
|
||
| Metrics will, by default, be generated with the name `http.client.requests`. The name | ||
| can be customized using the `spring.metrics.web.client.requests-metrics-name` property. | ||
| By deault, metrics are generated with the name, `http.client.requests`. The name |
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.
"deault" -> "default"
| the application directory with the file name, `application.pid`). | ||
| * `EmbeddedServerPortFileWriter` creates a file (or files) containing the ports of the | ||
| embedded server (by default in the application directory with the file name | ||
| embedded server (by default, in the application directory with the file name, |
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.
Drop the second comma?
* gh-10830: Wrap production-ready-features.adoc at 90 characters Make editorial changes to production-ready-features.adoc
I made it more formal and corrected sentence errors.