[GitHub] [streampipes] tenthe commented on pull request #850: [#820] enable checkstyle for some extension modules

[GitBox <gi...@apache.org>]

tenthe commented on PR #850:
URL: https://github.com/apache/streampipes/pull/850#issuecomment-1344140867

   @flomickl, ah ok, sorry, I didn't know you were working on this.
   We try to capture all the modules we are working on in #820. If you're working on a module, you can tag it there.
   
   As for `streampipes-processors-geo-jvm`, this PR is pretty big and includes changes for all modules in streampipes-extensions. I think it's not that easy to take those changes out. My goal would be to merge it as soon as possible to reduce the risk of further merge conflicts.
   My suggestion would be if you merge your branch, you can apply all your changes for the module `streampipes-processors-geo-jvm` so you don't have to resolve all the conflicts manually. My changes only affect the ceckstyle, we don't lose any logic if you override them with your changes.


-- 
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: dev-unsubscribe@streampipes.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org