dstandish commented on issue #5399: [AIRFLOW-4756] fix gantt view error after 
clearing failed task
URL: https://github.com/apache/airflow/pull/5399#issuecomment-500672644
 
 
   I mean the answer is always yes, right?  
   
   But probably a bit of a refactor would be in order.  You would want to 
establish what the `data` object, which is passed to the view, needs to look 
like.  Then wrap in a function the logic that produces that data object.  Then 
write tests for that.  Or something like that, no?  
   
   But, this does fix a bug!

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