OmairK commented on pull request #11179:
URL: https://github.com/apache/airflow/pull/11179#issuecomment-700580202


   > Pre-commit works best if it acts on the files that got changed, but this 
case is a bit different. I think eventually we want to run all the relevant 
.bat tests any time any of the relevant .sh or .bat tests change (because we 
have no direct relation .sh -> .bat).
   
   I was thinking of adding another pre-commit hook to look for relevant bats 
file for a .sh, but we will need a consistent naming scheme for that to happen 
<test_relevant_file.bats>. Do you think this is something that needs to be 
there or am I having too much fun with bash and regex and should leave it for 
now :smile:. 


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


Reply via email to