+1 to add Windows tests to the PR pipeline. It may take longer time to run (up to 4 hours). But consider the time wasted on reverting, fixing and resubmitting, if there is a failure after merging to the develop branch. It is better to add the Windows tests to the PR pipeline. We can reevaluate and optimize the pipeline if the long running time is truly a concern.
On Thu, Jun 25, 2020 at 9:29 AM Kirk Lund <kl...@apache.org> wrote: > I merged some new AcceptanceTests to develop after having my PR go GREEN. > But now these tests are failing in Windows. > > I'd like to propose that we add the Windows jobs to our PR checks if we > plan to keep testing on Windows in CI. > > Please vote or discuss. > > Thanks, > Kirk >