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.

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

#4360 - Clean up code #4361

Merged
merged 6 commits into from
Dec 7, 2023
Merged

#4360 - Clean up code #4361

merged 6 commits into from
Dec 7, 2023

Conversation

reckart
Copy link
Member

@reckart reckart commented Dec 6, 2023

What's in the PR

  • Remove unused class
  • Rename enum classes and persistent enum type classes to have more sensible names

How to test manually

  • No specific test procedure

Automatic testing

  • PR includes unit tests

Documentation

  • PR updates documentation

- Remove unused class
- Rename enum classes and persistent enum type classes to have more sensible names
@reckart reckart added this to the 31.0 milestone Dec 6, 2023
@reckart reckart self-assigned this Dec 6, 2023
- Remove deprecated unused code
- Consolidate duplicate functionality
- Use AnnotationPredicates instead of locally implementing them
- Use modern Java API
- Update string reference to renamed persistent enum type
- Use modern Java API
@reckart reckart merged commit 0956c64 into main Dec 7, 2023
@reckart reckart deleted the refactoring/4360-Clean-up-code branch December 7, 2023 17:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

1 participant