lonerzzz commented on a change in pull request #6181: Changes to improve 
readability
URL: https://github.com/apache/pulsar/pull/6181#discussion_r380920902
 
 

 ##########
 File path: site2/docs/deploy-bare-metal.md
 ##########
 @@ -103,7 +103,7 @@ Directory | Contains
 
 ## [Install Builtin Connectors (optional)]( 
https://pulsar.apache.org/docs/en/next/standalone/#install-builtin-connectors-optional)
 
-> Since Pulsar releases `2.1.0-incubating`, Pulsar releases a separate binary 
distribution, containing all the `builtin` connectors.
+> Since Pulsar release `2.1.0-incubating`, Pulsar provides a separate binary 
distribution, containing all the `builtin` connectors.
 
 Review comment:
   The first 'releases' should be a singular noun as it is describing a single 
release. If that is not the intent, then the '2.1.0-incubating' should have 
wild cards added to indicate the set of releases. For the second 'releases', I 
used a different term since using words with the same name and different 
meanings makes reading the sentence more complex.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

Reply via email to