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:
[email protected]


With regards,
Apache Git Services

Reply via email to