sramazzina commented on pull request #1215: URL: https://github.com/apache/incubator-hop/pull/1215#issuecomment-989874345
Ciao Hans, Ciao Hans, good approach to tighten the screws on quality. I totally agree and also I think it's good idea to prepare a checklist that must be followed by everybody. I will add the integration tests and update the documentation and that, from my side, it will be done for every PR from now on. If you agree, at this point, I can also update the Meta class by using HopMetadataProperty annotation and remove getXml and load Xml methods. It will require a bit more time but, after tha,t everything will be cleaned up definitely. Let me know what you think about. -- 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]
