bamaer commented on PR #8962: URL: https://github.com/apache/hop/pull/8962#issuecomment-6215461148
LGTM. Built the PR locally and ran the 18 new test classes: all pass. With the changed source files reverted to main, all of them fail except `ActionSetVariablesBlankValueTest`, which is expected now that the Set Variables change is out. Not run against real MySQL/Oracle/AWS/SOAP/SFTP endpoints. -- 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]
