[GitHub] [airflow] potiuk commented on issue #6415: [AIRFLOW-5745] Breeze complete has licence nown

[GitBox <gi...@apache.org>]

potiuk commented on issue #6415: [AIRFLOW-5745] Breeze complete has licence nown
URL: https://github.com/apache/airflow/pull/6415#issuecomment-546128744
 
 
   Yep. That was the thing. It was excluded via .dockerignore. I will make a small modification to the scripts to mount all source files in CI (even those .dockerignored) specifically for RAT checker. 
   It will make it different that local checks - mounting all sources on Mac  make the checks a loooot slower (combined node_modules and slow osxfs). 
   
   There is only one question why it was not detected by insert-license pre-commit hook (it was configured to include that file as well and should be detected and added automatically but for some reason it was not). I am looking into it.

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


With regards,
Apache Git Services