Skip to content

Upgrade to PMD 7 #174

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 12 commits into from
Apr 10, 2023
Merged

Upgrade to PMD 7 #174

merged 12 commits into from
Apr 10, 2023

Conversation

adangel
Copy link
Member

@adangel adangel commented Mar 24, 2023

No description provided.

@adangel adangel added this to the 7.0.0 milestone Mar 24, 2023
adangel added 11 commits April 10, 2023 12:12
- Set minimum required Java version to 8 (BRRE)
- Use TreeExport XmlTreeRenderer for AST writer
- Remove type resolution option. This is now always enabled.
- Remove dfa option. This doesn't exist anymore.
- Remove PMD Designer buttons
These are ExceptionInInitializerError and NoClassDefFoundError

It basically won't work without changes in eclipse equinox, which
provides uris like "bundleresource://..." from the class loader.
ApexLink uses Class.getResource(...) to find all pseudo Apex
classes on the classpath (all classes under package "com.nawforce.runforce".

See https://github.com/nawforce/apex-link/blob/7688adcb7a2d7f8aa28d0618ffb2a3aa81151858/apexlink/src/main/scala/com/nawforce/apexlink/types/platform/PlatformTypeDeclaration.scala#L260-L273
@adangel adangel marked this pull request as ready for review April 10, 2023 10:35
@adangel adangel merged commit 769326c into pmd:master Apr 10, 2023
@adangel adangel deleted the upgrade-to-pmd7 branch April 10, 2023 12:29
# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant