Conversation
There was a problem hiding this comment.
Unfortunate that IntelliJ (I'm assuming) rearranged everything in the inspection file. On quick glance, it looks like a couple inspections got removed: e.g., AutoCloseableResource and WeakerAccess. Intentional? Maybe they're on by default so don't need to be specified?
AutoCloseable and WeakerAccess appear to be redundant with current IntelliJ defaults. The Unicode change was unintentional and has been reverted. The JavaDoc ones were intentional. The custom inspections appear to be have been alphabetized by name, by IntelliJ. Hopefully that means they'll be easier to diff going forward. |
Rationale
We want to be using our centrally defined XML parser configuration
Related Pull Requests
Changes