avikganguly01 commented on pull request #1894: URL: https://github.com/apache/fineract/pull/1894#issuecomment-947587901
> @avikganguly01 @francisguchie @ankita10r @IOhacker > > This has now broken the build, and all the other builds are failing. We should _never_ merge failing builds please. I will try to revert this change - please can you fix the failing tests before merging again. @IOhacker : There are 10+ tests like testSkippingRepaymentOnFirstDayOfMonth() which fails randomly and has nothing to do with this PR. @ptuomola : Can you be more specific with regards to which integration test failed because of this PR? Do not revert this change unless proven otherwise. The author of this PR is not responsible for fixing all the integration tests which fail due to no fault of hers. -- 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
