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

   I think it's about 10th attempt I remember to fix the codecov support :D. 
   
   I am tempted to make this comment in our CI.yml
   
   ```
   # For those who notice that our code coverage is broken and attempt to fix 
it:
   # Please increase the counter below after the attempt of yours failed
   # Number of attempts: 10
   ```
   
   Joking of course ... But I would be lying if I said that I am looking 
forward to it. Code coverage is a rather misleading number usually, and I never 
had a case where it was either:
   
   a) put as a "rigid target" and inducing behaviours that did not increase the 
quality of the tests but costed a lot of time and (mostly useless) effort.
   b) were completely ignored
   c) were sometimes looked at to see that we are not going in a wrong 
direction (semi-randomly)
   
   I think the c) approach is the healtiest one - and for that I think the 
current codecov  integration is not needed IMHO. 
   
   My 3 cents :). I do not feel very strongly about it, I just learned to be 
pretty indifferent about having/not having codecov support. I would rather 
completely remove it to be honest if we decide it is not REALLY useful and if 
we do not based some real actions on 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.

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