splatch commented on pull request #263: URL: https://github.com/apache/plc4x/pull/263#issuecomment-889249872
> bit stringly typed so maybe we can add some sanity checks that protocolName, and output flavor is present on getMessageIOType but other than that LGTM I agree, it is indeed yet another string trap. I'd wish we could migrate to typed configuration and minimize amount of arguments passed over test framework (language, protocolName, driverName). Yet for that I believe we need to book some exclusive time reserved only for test framework. -- 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]
