chibenwa commented on PR #3073: URL: https://github.com/apache/james-project/pull/3073#issuecomment-4916632385
> @chibenwa Do you think these checks are too strict? It's IMO good to have a fixed header ordering. I'm happy that we don't have any longer to push "reorder header" and `add missing licenses" commits. FYI intellij has a checkstyle integration that has the potential to make your life easier. > And what do you think about splitting the checks for the existence of the mailbox and the existence of a valid RRT entry into two methods? Agree -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
