Currently we have zero Windows tests in the PR pipeline, which has led to more 
than a handful of times this year where a PR passed all its checks then failed 
the main pipeline once merged.

Adding these 3 jobs to the PR pipeline would have caught the majority of them:
WindowsUnitTestOpenJDK11
WindowsIntegrationTestOpenJDK11
WindowsAcceptanceTestOpenJDK11

The longest of these 3 jobs is about 1/2 hour longer than the current longest 
PR check.

This is a discussion, not a vote thread.  This might also be a good chance to 
reflect on whether more or fewer PR checks should be “required”, whether there 
are other checks we do not have currently but would be nice, the balance 
between JDK8 and JDK11 checks, or even brainstorm ideas to create additional 
checks that might not apply to all PRs but could be requested (or attached 
automatically based on area of code touched).

Reply via email to