You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@zeppelin.apache.org by "Reamer (via GitHub)" <gi...@apache.org> on 2024/04/02 07:45:11 UTC
Re: [PR] [ZEPPELIN-6006] Remove command line applications when downloading applications [zeppelin]
Reamer commented on PR #4746:
URL: https://github.com/apache/zeppelin/pull/4746#issuecomment-2031300017
You are right, the build-tools module was set up so that the checkstyle reports work properly and we can distribute the configuration cleanly. (see https://maven.apache.org/plugins/maven-checkstyle-plugin/examples/multi-module-config.html)
I think it is unnecessary to create another module, so I have moved it in there.
It was important to me that the code is contained in a module that is independent of the Zeppelin interpreter modules and the Zeppelin-zengine/zeppelin server.
I can also put the code in a new module. What do you think?
Translated with DeepL.com (free version)
--
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: reviews-unsubscribe@zeppelin.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org