Skip to content

[java] Make allowExceptionNameRegex a Regex#4724

Merged
adangel merged 1 commit into
pmd:masterfrom
jsotuyod:pattern-properties
Nov 10, 2023
Merged

[java] Make allowExceptionNameRegex a Regex#4724
adangel merged 1 commit into
pmd:masterfrom
jsotuyod:pattern-properties

Conversation

@jsotuyod

Copy link
Copy Markdown
Member

Describe the PR

This is the last regex property not using PropertyDescriptor<Pattern> underneath.

Related issues

Fixes #1027

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)

@jsotuyod jsotuyod added this to the 7.0.0 milestone Oct 20, 2023
@jsotuyod jsotuyod mentioned this pull request Oct 20, 2023
55 tasks
@ghost

ghost commented Oct 20, 2023

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 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!

adangel added a commit that referenced this pull request Nov 10, 2023
adangel added a commit that referenced this pull request Nov 10, 2023
[java] Make allowExceptionNameRegex a Regex #4724
@adangel adangel merged commit 93f5a68 into pmd:master Nov 10, 2023
@jsotuyod jsotuyod deleted the pattern-properties branch November 10, 2023 10:42
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] Apply the new PropertyDescriptor<Pattern> type where applicable

2 participants