You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2020/08/19 11:46:32 UTC

[GitHub] [airflow] potiuk commented on pull request #10402: Replaced aliases for common tools with functions.

potiuk commented on pull request #10402:
URL: https://github.com/apache/airflow/pull/10402#issuecomment-676211992


   Inspired by @feluelle  I found a much better way - instead of aliases I am using functions with the same name. That required extraction of command binary paths at the time of function definition, but it works much better (for example it works in non-interactive git pre-commits and also it opens up capabilities of automated testing of our bash functions (which I planned for quite some time). That will come next.


----------------------------------------------------------------
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:
users@infra.apache.org