matrei commented on PR #15686: URL: https://github.com/apache/grails-core/pull/15686#issuecomment-4593460293
> > > I don't think we can merge hibernate 7 without this change. This is a critical component of that merge. We only have it configured to run in hibernate 7. > > > > > > @jdaugherty Why is it a critical component of merging an implementation of Hibernate 7? > > Also, a general observation, there seems to be a lot of commits with functional and formatting changes combined. > > It is helpful if we separate these into different commits. > > We used this plugin only in the hibernate 7 dev to ensure security best practices. That is interesting. I have not used PMD before. Looking at the [security](https://docs.pmd-code.org/latest/pmd_rules_java.html#security) rule set, there only seems to be two rules, and none of them seems to apply to Hibernate 7 code. Are there other rules that helps ensure security best practices? -- 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]
