potiuk commented on issue #19967:
URL: https://github.com/apache/airflow/issues/19967#issuecomment-987865031


   Yeah. Just add a pre-commiit to generate the files and save them somewhere 
in `./dev/breeze/autocomplete/` for example.
   
   There is no need to check it. Assuming that precommit generates the same 
script when you run it twice, pre-commit checks on it's own if the re-generated 
file changed or not. So it's ok to regenerate the file again and again only to 
find out that nothing changed (but in pre-commit definition you should specify 
that pre-commit should run only when the `./airflow_breeze.py` changes (because 
that's where the defintions of click commands are). 
   
   This can be done by adding `pass_filenames` to false: 
https://pre-commit.com/#hooks-pass_filenames and adding 
dev/breeze/airflow_breeze.py proper regexp to `files` parameter of pre-commit 
https://pre-commit.com/#hooks-files. 
   


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