[GitHub] [airflow] potiuk commented on a change in pull request #21346: Update docs/readme.rst

[GitBox <gi...@apache.org>]

potiuk commented on a change in pull request #21346:
URL: https://github.com/apache/airflow/pull/21346#discussion_r800551393



##########
File path: docs/README.rst
##########
@@ -74,7 +74,24 @@ Once you have built the documentation run the following command from the root di
 
     docs/start_doc_server.sh
 
-Then, view your docs at ``localhost:8000``
+You need to have python installed to run the command, if you get an error message -- python: command not found
+It is possible that you don't have python installed or added to PATH.
+Visit link <https://python.land/installing-python> to install python and add to PATH

Review comment:
       One more thing. I think it will be better if you remove the whole chapter about "replacing python with python3" - but instead you should simply **actually replace** it as part of this PR. 
   
   I believe while "python" might not be available on the PATH, if python3 is available, using `python3` there is the **right** thing to do . 




-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org