Skip to content
New issue

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

Sign up for GitHub

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

Already on GitHub? Sign in to your account

Jump to bottom

[ant] Split off Ant integration into a new submodule #4080

Closed
Tracked by #2524
oowekyala opened this issue Aug 4, 2022 · 1 comment · Fixed by #4089
Closed
Tracked by #2524

[ant] Split off Ant integration into a new submodule #4080

oowekyala opened this issue Aug 4, 2022 · 1 comment · Fixed by #4089
Labels
an:enhancement An improvement on existing features / rules dependencies Pull requests that update a dependency file
Projects
PMD 7
Milestone
7.0.0

Comments

@oowekyala
Copy link
Member

Is your feature request related to a problem? Please describe.
Currently the PMD and CPD Ant tasks are implemented in pmd-core. This adds dependencies to pmd-core, which could essentially be a library on which PMD clients could rely (the ant integration being only one such client).

Describe the solution you'd like
Restrict pmd-core to provide only a programmatic API. Move Ant-related classes and dependencies to a new pmd-ant module.

Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.

Additional context Related to #4079
@oowekyala oowekyala added the an:enhancement An improvement on existing features / rules label Aug 4, 2022
@oowekyala oowekyala added this to the 7.0.0 milestone Aug 4, 2022
@oowekyala oowekyala added the dependencies Pull requests that update a dependency file label Aug 4, 2022
@oowekyala oowekyala added this to To do in PMD 7 Aug 4, 2022
This was referenced Aug 9, 2022
Merged
Closed
@adangel adangel mentioned this issue Oct 4, 2022
Closed
6 tasks
@adangel adangel moved this from To do to In progress in PMD 7 Oct 4, 2022
@adangel adangel linked a pull request Oct 4, 2022 that will close this issue
[ant] Move ant integration to a new pmd-ant module #4089
Merged
4 tasks
adangel added a commit that referenced this issue Oct 4, 2022
@adangel
[doc] Update release notes (#4080, #4089)
98a4686
@adangel
Copy link
Member

adangel commented Oct 4, 2022

Finished for PMD 7 via #4089

@adangel adangel closed this as completed Oct 4, 2022
PMD 7 automation moved this from In progress to Done Oct 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
an:enhancement An improvement on existing features / rules dependencies Pull requests that update a dependency file
Projects
No open projects
PMD 7
  
Done
Milestone
7.0.0
Development

Successfully merging a pull request may close this issue.

[ant] Move ant integration to a new pmd-ant module oowekyala/pmd
2 participants
@adangel @oowekyala