kaxil commented on a change in pull request #7207: [AIRFLOW-XXXX] Consistency fixes in new documentation URL: https://github.com/apache/airflow/pull/7207#discussion_r368283262
########## File path: CONTRIBUTING.rst ########## @@ -341,16 +341,16 @@ itself comes bundled with jQuery and bootstrap. While they may be phased out over time, these packages are currently not managed with yarn. Make sure you are using recent versions of node and yarn. No problems have been -found with node\>=8.11.3 and yarn\>=1.19.1 +found with node\>=8.11.3 and yarn\>=1.19.1. -Installing yarn and its packages -------------------------------- +Install yarn and Its Packages +----------------------------- Make sure yarn is available in your environment. -To install it on macOS: +To install yarn on macOS: -1. Run the following commands (taken from `this source <https://gist.github.com/DanHerbert/9520689>`__): +1. Run the install commands (taken from `this source <https://gist.github.com/DanHerbert/9520689>`__): Review comment: ```suggestion 1. Run the following commands (taken from `this source <https://gist.github.com/DanHerbert/9520689>`__): ``` Let's keep "following" as it has a command to set config too ---------------------------------------------------------------- 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
