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 <m...@hotpy.org> 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 -- python-committers@python.org > To unsubscribe send an email to python-committers-le...@python.org > https://mail.python.org/mailman3/lists/python-committers.python.org/ > Message archived at > https://mail.python.org/archives/list/python-committers@python.org/message/322BYODJ5ETK557Q62NW6GP64V2F4TQD/ > Code of Conduct: https://www.python.org/psf/codeofconduct/ _______________________________________________ python-committers mailing list -- python-committers@python.org To unsubscribe send an email to python-committers-le...@python.org https://mail.python.org/mailman3/lists/python-committers.python.org/ Message archived at https://mail.python.org/archives/list/python-committers@python.org/message/LQHG2M6N4HRRXVHZYPXSKSEZEQ76HCNQ/ Code of Conduct: https://www.python.org/psf/codeofconduct/
