ephraimbuddy commented on PR #37152:
URL: https://github.com/apache/airflow/pull/37152#issuecomment-1968889328

   I like the suggestions. From the suggestions, we are going to do two things, 
the 3rd one will be pending.
   1. separate job for the coverage analysis
   Given what we did in warnings text job, I believe it's possible but will 
prefer to do it in a separate PR. For now, I will make this non-fatal and think 
about how to collect the analysis and implement it like the warnings job in a 
separate PR.
   
   2. Make the coverage failure understandable by providing URLs for the user 
to look at and understand where the problem is coming from.
   This can also happen in a separate PR, WDYT?
   


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