jedcunningham opened a new pull request #21702:
URL: https://github.com/apache/airflow/pull/21702


   Especially given the audience of this script, we don't need to have special 
handling of the error message when an import fails.
   
   Instead of:
   ```
   Could not find the click library. Run 'sudo pip install click' to install.
   ```
   
   Allow the built in failure to bubble up instead:
   ```
   Traceback (most recent call last):                                           
                                                                                
                     
     File "./dev/airflow-github", line 29, in <module>                          
                                                                                
                     
       import rich_click as click                                               
                                                                                
                     
   ModuleNotFoundError: No module named 'rich_click'
   ``` 
   
   Of course I hit this because of the new `rich_click` vs `click` thing and 
was confused since I had `click`, but either way, we don't need the extra hand 
holding I don't think.


-- 
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