alb3rtobr commented on pull request #6116: URL: https://github.com/apache/geode/pull/6116#issuecomment-828250179
> > Tests were executed again today and one test failed unexpectedly. Last commit was done 21 days ago > > It's always a risk to merge based on tests that passed long time ago (sometimes even short time ago -- a lot can change on develop in a short time). We recently had a commit merged that immediately broke develop, and found that PR checks had last been run 4 weeks earlier. To reduce this risk, I will be re-executing tests weekly to improve confidence that PRs have been tested against something more closely resembling the latest develop. Agree, but then PRs should be rebased against develop branch, right? In my case, I have seen there are conflicts I have to solve, so Im going to rebase anyway... -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected]
