tenthe commented on issue #880: URL: https://github.com/apache/streampipes/issues/880#issuecomment-1425351537
Yes, you can do this in your IDE. We also have a short description in our wiki how to do it [1]. You can find the checkstyle file in [2]. My suggestion would be to add checkstyle in your local development environment first. To test that everythinf is setup correctly, you can fix the checkstyle violations in #1192. To ensure that there are no checkstyle violations we use a GitHub action. This is also the reason why the build in #1192 failed. To give the developer an even quicker feedback we want to integrate this check as a git pre-commit hook. That means a developer can only perform a git commit when there are no checkstyle violations. I didn't do something like this before, but I guess we can also use the library https://pre-commit.com/ for that. @bossenti: Do you think we should harmonize the tools we use for the hooks? Is it possible to use pre-commit for all the different languages? (For Typescript we use [3]) [1] https://cwiki.apache.org/confluence/display/STREAMPIPES/Code+Style+-+Java [2] https://github.com/apache/streampipes/blob/dev/tools/maven/checkstyle.xml [3] https://github.com/typicode/husky -- 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]
