Yiyiyimu commented on a change in pull request #3980: URL: https://github.com/apache/apisix/pull/3980#discussion_r607116559
########## File path: docs/en/latest/getting-started.md ########## @@ -216,7 +216,7 @@ $ curl http://127.0.0.1:9080/apisix/admin/consumers -H 'X-API-KEY: edd1c9f034335 Next, let's bind our `Consumer(John)` to that `Route`, we only need to **Enable** the [key-auth](./plugins/key-auth.md) plugin for that `Route`: ```bash -curl http://127.0.0.1:9080/apisix/admin/routes/5 -H 'X-API-KEY: edd1c9f034335f136f87ad84b625c8f1' -X PUT -d ' +$ curl http://127.0.0.1:9080/apisix/admin/routes/5 -H 'X-API-KEY: edd1c9f034335f136f87ad84b625c8f1' -X PUT -d ' Review comment: Actually I think it would be a good idea to remove all the `$`, so users could directly use the copy button on the website to get the 'runnable' command. cc @juzhiyuan Another reason could be, the dollar sign is often used when trying to separate commands and outputs. Since in this doc, we only list the command, so there seems no need to use `$` -- 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]
