meonkeys commented on PR #5264: URL: https://github.com/apache/fineract/pull/5264#issuecomment-3725733480
Thanks for the "Testing" section in the original PR description. What does "Verified Liquibase changelog integrity locally" mean? I suggest adding to your acceptance/verification testing: 1. Perform all the usual developer code sanity/smoke checks Ádám Sághy mentioned / make sure build passes / etc. 1. Per FINERACT-2423 description, run the server or a release jar and make sure the warning no longer appears. 1. Test your changes with synthetic data. There might already be tests for migrations and I don't know where to get synthetic data. -- 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]
