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
