I never noticed this one.... but on the topic of useless CI checks Up-to-date checker seems like a strong candidate for removal for similar reasons.
On Sat, Jun 25, 2022 at 2:26 AM Jarek Potiuk <[email protected]> wrote: > Hey all, > > I think this workflow we have in CI is rather useless now. > It has been somewhat useful at some point in the past when we tried to > automate CI checks and see if we can get more info if the PR needs > more thorough check (i.e. whether it touches core airflow, or whether > it requires "full tests" but there are few problems: > > * it introduces a lot of noise > * it's not 100% foolproof - there are both false positives and false > negatives > * we can (and sometimes do) manually set the "full test needed" label > for PRs that need more "thorough" CI check before merge > > I personally see that as mostly noise for quite some time now and I > see no value in it. Anyone disagree? > > Should we remove it ? > > J. >
