ParkGyeongTae commented on PR #5308: URL: https://github.com/apache/zeppelin/pull/5308#issuecomment-5010975946
Good catch, thanks for digging into the #4884 history — you're right. I re-checked: the `<version>` tag was removed intentionally in ZEPPELIN-6138 as part of deferring ~10 plugins to the Apache parent pom's managed versions, and `plugin.checkstyle.version` was just leftover cruft from that change, not a missing reference that needed to be restored. I've updated this PR (and the Jira description) to remove the orphaned property instead of re-pinning it, so maven-checkstyle-plugin stays consistent with the other plugins from #4884 that are parent-managed. Verified locally that the build still runs checkstyle (now inherited from the parent pom, currently 3.4.0) and passes. PTAL, thanks again for catching this. -- 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]
