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

[core] Remove old CLI from pmd core #4523

Merged
merged 9 commits into from
May 2, 2023
Merged

[core] Remove old CLI from pmd core #4523

merged 9 commits into from
May 2, 2023

Conversation

oowekyala
Copy link
Member

Describe the PR

Remove the old CLI from PMD core. Instead of updating the code samples on the website I improved the javadocs and linked to them.

Note this contains #4522

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)

@oowekyala oowekyala added in:documentation Affects the documentation is:feature-removal Remove an unneeded unused feature labels Apr 29, 2023
@oowekyala oowekyala added this to the 7.0.0 milestone Apr 29, 2023
@pmd-test
Copy link

1 Message
📖 Compared to master:
This changeset changes 5 violations,
introduces 0 new violations, 0 new errors and 0 new configuration errors,
removes 0 violations, 0 errors and 0 configuration errors.
Full report

Generated by 🚫 Danger

@oowekyala oowekyala marked this pull request as ready for review April 29, 2023 16:59
@jsotuyod jsotuyod merged commit cb62fab into pmd:master May 2, 2023
3 checks passed
@oowekyala oowekyala deleted the pmd7.remove-core-cli branch May 2, 2023 14:57
adangel
adangel reviewed May 4, 2023
View reviewed changes
pom.xml Show resolved Hide resolved
adangel added a commit that referenced this pull request May 4, 2023
@adangel
[doc] Update release notes (#4523)
dc61b11
adangel added a commit to adangel/pmd that referenced this pull request May 4, 2023
@adangel
[java] PMDCoverageTest - assert report (pmd#4523)
d5a44da
@oowekyala oowekyala mentioned this pull request May 26, 2023
Closed
4 tasks
@adangel adangel mentioned this pull request May 30, 2023
Closed
@oowekyala oowekyala mentioned this pull request Jun 6, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@adangel adangel adangel left review comments

@jsotuyod jsotuyod jsotuyod approved these changes

Assignees
No one assigned
Labels
in:documentation Affects the documentation is:feature-removal Remove an unneeded unused feature
Projects
None yet
Milestone
7.0.0
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@oowekyala @pmd-test @jsotuyod @adangel