potiuk commented on PR #36537:
URL: https://github.com/apache/airflow/pull/36537#issuecomment-1885343909

   All right I verified the contents of generated files and added a few 
modifications. Generally speaking - we have few more files added:
   * README files
   * openapi.yml
   * .pyi files
   * .mako file (generated ffo )
   * provider.yaml.schema.json
   
   All of those are fine to be part of the packages.
   
   Yarn errror.log was present in the prevrious package, so it's good to be 
excluded
   
   Also in the new package, licesnse files structure is a bit diferent - I 
renamed "licenses" directory to 3rd-party-licenses to better show the 
difference between Airflow license and 3rd-party ones (I needed to add 
not-yet-approved PEP-639 support for multiple licences (and hatchling nicely 
supports it).
   
   I also compared the generated files and the differences there are 
refliecting the actual differences (removed devel_extras etc.).
   
   Below is comparision (left side: old - right side: new):
   
   ![Screenshot from 2024-01-10 
18-47-30](https://github.com/apache/airflow/assets/595491/13caf66f-0cd9-4f40-b969-a85b7ed51149)
   
   
   ![Screenshot from 2024-01-10 
18-43-40](https://github.com/apache/airflow/assets/595491/41ec469a-0764-49a1-9723-2bdcaa85c041)
   
   
   


-- 
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: commits-unsubscr...@airflow.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to