Skip to content

[core] Consolidate n.s.pmd.lang.rule package#4799

Merged
adangel merged 6 commits into
pmd:masterfrom
adangel:issue-3917-rule-package
Feb 9, 2024
Merged

[core] Consolidate n.s.pmd.lang.rule package#4799
adangel merged 6 commits into
pmd:masterfrom
adangel:issue-3917-rule-package

Conversation

@adangel

@adangel adangel commented Jan 13, 2024

Copy link
Copy Markdown
Member

Describe the PR

  • moved the classes, moved some of them into internal and removed the annotations
  • updated release notes
  • fixed compatibility layer for maven-pmd-plugin (pmd-compat6)

Related issues

Ready?

  • Added unit tests for fixed bug/feature
  • Passing all unit tests
  • Complete build ./mvnw clean verify passes (checked automatically by github actions)
  • Added (in-code) documentation (if needed)

@adangel adangel added this to the 7.0.0 milestone Jan 13, 2024
@adangel adangel mentioned this pull request Jan 13, 2024
55 tasks
@ghost

ghost commented Jan 13, 2024

Copy link
Copy Markdown
1 Message
📖 Compared to master:
This changeset changes 0 violations,
introduces 0 new violations, 0 new errors and 0 new configuration errors,
removes 0 violations, 0 errors and 0 configuration errors.
Download full report as build artifact

Generated by 🚫 Danger

@adangel adangel merged commit 2a3128f into pmd:master Feb 9, 2024
@adangel adangel deleted the issue-3917-rule-package branch February 9, 2024 16:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[core] Consolidate n.s.pmd.lang.rule package

1 participant