pingtimeout commented on code in PR #1673:
URL: https://github.com/apache/polaris/pull/1673#discussion_r2108450527
##########
site/content/in-dev/unreleased/getting-started/deploying-polaris/quickstart-deploy-aws.md:
##########
@@ -38,16 +38,15 @@ The requirements to run the script below are:
```shell
chmod +x getting-started/assets/cloud_providers/deploy-aws.sh
export ASSETS_PATH=$(pwd)/getting-started/assets/
-./getting-started/assets/cloud_providers/deploy-aws.sh
-```
-
-Also, set the following static credentials for interacting with the Polaris
server in the following exercises:
-
-```shell
export CLIENT_ID=root
export CLIENT_SECRET=s3cr3t
+./getting-started/assets/cloud_providers/deploy-aws.sh
```
+## Next Steps
+Congrats, you now have a running instance of Polaris! For further information
regarding how to use Polaris, check out the [Using Polaris]({{% ref
"using-polaris" %}}) page.
Review Comment:
+1 to what Eric said. The guideline is that, in a given document, the first
occurrence of an Apache project should be "_Apache XYZ_", and the subsequent
occurrences can be just "_XYZ_".
Considering that this is for the documentation website that contains "Apache
Polaris" in the header, and that "Apache Polaris (incubating)" is mentioned in
the very first page (Overview), we can just use "Polaris" in the rest of the
documentation.
--
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.
To unsubscribe, e-mail: issues-unsubscr...@polaris.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
