Skip to content

[java] SimplifyStartsWith: update description and example, fixes #1868#2723

Merged
oowekyala merged 1 commit into
pmd:masterfrom
mateusz-stefanski:master
Aug 25, 2020
Merged

[java] SimplifyStartsWith: update description and example, fixes #1868#2723
oowekyala merged 1 commit into
pmd:masterfrom
mateusz-stefanski:master

Conversation

@mateusz-stefanski

Copy link
Copy Markdown
Contributor

Describe the PR

Updates Java SimplifyStartsWith rule description and example

Related issues

Ready?

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

@ghost

ghost commented Aug 23, 2020

Copy link
Copy Markdown
1 Message
📖 This changeset introduces 0 new violations, 0 new errors and 0 new configuration errors,
removes 0 violations, 2 errors and 2 configuration errors.
Full report
This changeset introduces 0 new violations, 0 new errors and 0 new configuration errors,
removes 0 violations, 2 errors and 2 configuration errors.
Full report

Generated by 🚫 Danger

@oowekyala oowekyala left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks!

@oowekyala oowekyala changed the title [doc] [java] SimplifyStartsWith: update description and example, fixes #1868 [java] SimplifyStartsWith: update description and example, fixes #1868 Aug 25, 2020
oowekyala added a commit that referenced this pull request Aug 25, 2020
@oowekyala oowekyala merged commit 4d888ee into pmd:master Aug 25, 2020
@oowekyala oowekyala added this to the 6.27.0 milestone Aug 28, 2020
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.

[java] false-positive for SimplifyStartsWith if string is empty

2 participants