snuyanzin commented on PR #25022: URL: https://github.com/apache/flink/pull/25022#issuecomment-2208531580
Thanks for driving this In general it looks ok from my side Do you think it would make sense to add a checkstyle rule forbidding usage of `jsonpath` in imports? Like it is already done for all other shaded libs here https://github.com/apache/flink/blob/cc19159b489dafad71c01bf551bc224a38203eae/tools/maven/checkstyle.xml#L192-L207 -- 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]
