sruehl commented on pull request #201:
URL: https://github.com/apache/plc4x/pull/201#issuecomment-817078332


   2 Points reading your comment @splatch:
   
   1. @chrisdutz has a valid point with moving that to the SPI
   2. Regarding the "does it compile" comment: Maybe bind/use the fields in 
unit tests, then you would have a compile issue building your module on 
breaking changes and you don't pollute your productive code (You also could 
depend on internal artifacts with the test scope)


-- 
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.

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to