<div dir="auto">Ah, I found a workaround: Firefox on Android has a "[x] See the computer version" option which allows the merge!?<div dir="auto"><br></div><div dir="auto">Victor</div></div><div class="gmail_extra"><br><div class="gmail_quote">Le 22 avr. 2017 12:29 AM, "Victor Stinner" <<a href="mailto:victor.stinner@gmail.com">victor.stinner@gmail.com</a>> a écrit :<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="auto">Hi,<div dir="auto"><br></div><div dir="auto">I tried to merge a pull request on my phone, but I got the error:</div><div dir="auto"><br></div><div dir="auto">"Pull requests that have a failing status can’t be merged on a phone."</div><div dir="auto"><br></div><div dir="auto"><div dir="auto" style="font-family:sans-serif">The GitHub PEP announced that it will be possible to merge a change from the beach. Well, it's doable but only if you bring a laptop, not a phone :-)</div></div><div dir="auto"><br></div><div dir="auto">All tests pass except Codecov which is unstable. On a computer, I can merge such PR.</div><div dir="auto"><br></div><div dir="auto">What is the status of Codecov? Is someone actively working on fixing it to make it more reliable. I dislike code coverage in general, even more when it's run in a CI.</div><div dir="auto"><br></div><div dir="auto">Can we change the merge policy to allow merge on a phone? Or can we fix Codecov?</div><div dir="auto"><br></div><div dir="auto">Note: the PR is</div><div dir="auto"><a href="https://github.com/python/cpython/pull/1237#pullrequestreview-34044762" target="_blank">https://github.com/python/<wbr>cpython/pull/1237#<wbr>pullrequestreview-34044762</a><br></div><div dir="auto"><br></div><div dir="auto">Codecov says "10% of diff hit (target: 100%)". The newly added code is tested on Windows on release build. Maybe Codecov only tests on Windows?</div><div dir="auto"><br></div><div dir="auto">I dislike code coverage because there is a temptation to write artficial tests whereas the code is tested indirectly or the code is not important enough to *require* tests.</div><div dir="auto"><br></div><div dir="auto">Victor</div></div>
</blockquote></div></div>