amoghrajesh commented on PR #36548:
URL: https://github.com/apache/airflow/pull/36548#issuecomment-1875925476

   @ephraimbuddy there was a bug in the code and I managed to fix it now after 
some debugging. The issue was that the non core commits were getting included 
due to faulty else condition.
   
   Managed to test between origin/v2-7-stable and origin/v2-8-stable.
   
   Results:
   1. File sizes:
   ```
   root@8738c81c51bf:/opt/airflow/dev# ls -lh
   total 508K
   -rw-r--r--  1 root root  29K Jan  3 20:15 2-7-to-2-8-newchanges
   -rw-r--r--  1 root root  29K Jan  3 17:20 2-7-to-2-8-original
   ```
   2. Diff:
   ```
   root@8738c81c51bf:/opt/airflow/dev# git diff --no-index 2-7-to-2-8-original 
2-7-to-2-8-newchanges
   diff --git a/2-7-to-2-8-original b/2-7-to-2-8-newchanges
   index 78fda81..1862824 100644
   --- a/2-7-to-2-8-original
   +++ b/2-7-to-2-8-newchanges
   @@ -1,4 +1,4 @@
   -len of commits 1700
   +Number of commits 1700
    Uncategorized
    """""""""""""
    - Update RELEASE_NOTES.rst
   ```
   
![image](https://github.com/apache/airflow/assets/35884252/1969a15b-daec-45d5-9a2e-81d12f29d1a4)
   
   The diff is due to different log message, not any actual diff.


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