lonerzzz commented on a change in pull request #6181: Changes to improve readability URL: https://github.com/apache/pulsar/pull/6181#discussion_r380928343
########## File path: site2/docs/deploy-bare-metal.md ########## @@ -447,13 +445,13 @@ bin/pulsar-admin functions create \ --name exclamation ``` -Check if the function runs as expected by [triggering](functions-deploying.md#triggering-pulsar-functions) the function. +Check whether the function runs as expected by [triggering](functions-deploying.md#triggering-pulsar-functions) the function. ```bash bin/pulsar-admin functions trigger --name exclamation --trigger-value "hello world" ``` -You can see the output as below: +You should see the following output: Review comment: 'Can' implies the ability to do something. 'Should' is used to indicate what is probable or what is intended. When a person runs the command, they should see something, but if something was incorrect they would not. That is the intent I was conveying. ---------------------------------------------------------------- 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
