Error is thrown when using PMD 7.3.0

Issue #2588 resolved
Jason van Beukering created an issue

Hi, PMD 7.3.0 was recently released. Using this with IC is throwing an exception

Comments (6)

  1. Scott Wells repo owner
    • changed status to open

    PMD 7.x is changing pretty significantly in each update as they iterate on changes from the major PMD 7 update. My guess is that I probably just need to change the jars that are pulled in for that distribution, but in general it's best to stay on the latest supported version until I give the all-clear for the absolute latest version, at least until things stabilize a bit with PMD 7.

    I'm planning to release a build of IC2 on Monday. I'll see if I can work a fix for this into that build.

  2. Jason van Beukering reporter

    I'm doing brew updates regularly and this is at the moment auto-purging old PMD libraries, hence this issue. I'll look to switch this off, but at least I'm giving you advanced warning :)

    Regards Jason

  3. Scott Wells repo owner

    Yeah, I just returned to say “Thanks for letting me know!” because that’s the important part. Apologies if my previous response came off gruff or defensive. That wasn’t the intention at all. I was more just stating the reality of the version-to-version changes with PMD 7 at present. Having said that, it’s extremely useful to have folks let me know when updates are out because I only check about one every week or two.

  4. Scott Wells repo owner

    Yep, it's the addition of apex-ls_2.13-5.2.0.jar.classes to 7.3.0, presumably extracted from one of the other bundled jars, that causes this issue. I’ve already committed a fix that allows IC2’s inspection to work with all released versions of PMD 7.x again. That should be available on Monday. Thanks again for bringing this to my attention.

  5. Jason van Beukering reporter

    Awesome stuff. Ps. Wasn’t offended at all.

    Kind Regards Jason van Beukering (+1 773-456-1104)

  6. Log in to comment