Skip to content

Conversation

wener-tiobe
Copy link
Contributor

@wener-tiobe wener-tiobe commented Sep 6, 2023

Describe the PR

Fixes similar issue to #4423, but then for CPD.

Related issues

#4423

  • Fixes #

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)

@ghost
Copy link

ghost commented Sep 6, 2023

1 Message
📖 No regression tested rules have been changed.

Generated by 🚫 Danger

@adangel adangel added this to the 7.0.0 milestone Sep 7, 2023
@adangel adangel changed the title Update CpdCommand.java [cli] CPD: Fix NPE when only --file-list is specified Sep 7, 2023
@adangel adangel added the a:bug PMD crashes or fails to analyse a file. label Sep 7, 2023
@adangel adangel self-assigned this Sep 9, 2023
Copy link
Member

@adangel adangel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

I'll add a unit test...

adangel added a commit to adangel/pmd that referenced this pull request Sep 9, 2023
adangel added a commit that referenced this pull request Sep 9, 2023
[cli] CPD: Fix NPE when only --file-list is specified #4664
@adangel adangel merged commit d79df96 into pmd:master Sep 9, 2023
@wener-tiobe wener-tiobe deleted the patch-1 branch September 12, 2023 08:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

a:bug PMD crashes or fails to analyse a file.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants