One of the reasons I might be tempted to merge a pull request before the tests is completed is because there’s no mechanism to merge a pull request after the tests have completed (when there’s no reviewer).
I imagine it’s allowed because there still is the case where tests can fail for some reason unrelated to the pull request at hand. In short, the system isn’t reliable or fast or flexible enough to depend solely on automated systems, so it still relies on human judgment. At least, that’s my suspicion. > On 31 Mar, 2021, at 06:52, Mark Shannon <[email protected]> wrote: > > Hi, > > My PR tests are failing seemingly due to some bug in enum.rst > Which might be caused by https://github.com/python/cpython/pull/22392 > (that's the superficial cause, it might not be the root cause). > > Example failure: > https://github.com/python/cpython/pull/25069 > https://github.com/python/cpython/pull/25069/checks?check_run_id=2235991560 > > Why are we merging PRs that fail the CI tests? > Why is that even allowed? > > Cheers, > Mark. > _______________________________________________ > python-committers mailing list -- [email protected] > To unsubscribe send an email to [email protected] > https://mail.python.org/mailman3/lists/python-committers.python.org/ > Message archived at > https://mail.python.org/archives/list/[email protected]/message/322BYODJ5ETK557Q62NW6GP64V2F4TQD/ > Code of Conduct: https://www.python.org/psf/codeofconduct/ _______________________________________________ python-committers mailing list -- [email protected] To unsubscribe send an email to [email protected] https://mail.python.org/mailman3/lists/python-committers.python.org/ Message archived at https://mail.python.org/archives/list/[email protected]/message/LQHG2M6N4HRRXVHZYPXSKSEZEQ76HCNQ/ Code of Conduct: https://www.python.org/psf/codeofconduct/
