On Wed, 24 Jan 2018 at 10:12 Victor Stinner <victor.stinner@gmail.com> wrote:
Hi,

AppVeyor usually takes between 30 min and 1 hour to check a PR,
whereas Travis CI takes between 10 and 20 minutes (in average,
ignoring rare cases when it's broken). AppVeyor queue is regulary
busy.

Sometimes, I know that my PR is right, because the fix is obvious.
Sometimes, the PR has no impact on Windows. In these cases, the slow
AppVeyor CI can be annoying since it prevents me to merge a PR.

What do you think of making AppVeyor optional again? Careful core
developers should wait for AppVeyor, except if they know that Windows
doesn't matter for a specific PR.

No one has spoken up so I guess everyone else is okay with the delay ATM. My worry with flipping it off and trusting ourselves this close to the beta is the last time I did that people didn't wait and we landed breaking changes. So unless other people speak up that the delay is a hindrance I'm inclined to leave it until we at least get past the beta.

-Brett
 

Victor

2018-01-10 21:45 GMT+01:00 Brett Cannon <brett@python.org>:
> I just switched it on to help make sure we don't break on Windows just
> before hitting beta. If it turns out AppVeyor isn't stable enough I will
> switch it back off.
>
> _______________________________________________
> python-committers mailing list
> python-committers@python.org
> https://mail.python.org/mailman/listinfo/python-committers
> Code of Conduct: https://www.python.org/psf/codeofconduct/
>