rossturk commented on PR #623:
URL: https://github.com/apache/airflow-site/pull/623#issuecomment-1196844281

   > > I'm sorry, what are you suggesting as an alternative?
   > 
   > Remove Bash. Completely. I think this whole task is about removing bash 
and replacing it with Python :). I think the problem @Bowrna has (correct me 
@Bowrna if I am wrong) that you cannot really run the OLD script corrrectly. 
This can be fixed locally by updating bash with brew and setting temporary PATH 
to get it first on path. Once the script is replaced with Bash, there is no 
need to mention it in CONTRIBUTING - because ..... there will be no bash to run 
any more.
   > 
   > Or am I missing something :)?
   
   Yes, you are missing that this PR title is "replace site.sh" and the script 
we are talking about is `landing-pages/check-links.sh`. Are you suggesting that 
the scope of this PR is expanded to include *all* shell scripts?
   


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to