potiuk commented on pull request #11179: URL: https://github.com/apache/airflow/pull/11179#issuecomment-700599763
> 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 . I think this is a good idea, to do in the near future. Bash Unit tests are not the fastest ones and as we add more of those, pre-commit will take more and more time. So having 1-1 relationship: script/bats test is a good idea! And not very difficult one. ---------------------------------------------------------------- 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]
