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

OF-2647: Remove deprecated code #2243

Merged
merged 2 commits into from
Sep 4, 2023

Conversation

guusdk
Copy link
Member

@guusdk guusdk commented Aug 31, 2023

This removes all code that was marked as being deprecated, and to be removed in or after release 4.8.0

Most of the removal revolves around making references to files more type-safe, by moving from String to Path.

@guusdk
Copy link
Member Author

guusdk commented Aug 31, 2023

I have dismissed the four CodeQL check failures. They all relate to uncontrolled (depending on user-provided value) data being used. However, a check exists (it was added as part of OF-1886) that should prevent abuse.

@guusdk guusdk force-pushed the 2647_Remove-4.8-deprecation branch 2 times, most recently from 452c5f9 to 53965bc Compare September 2, 2023 08:42
This removes all code that was marked as being deprecated, and to be removed in or after release 4.8.0

Most of the removal revolves around making references to files more type-safe, by moving from String to Path.
@guusdk guusdk merged commit e360dc9 into igniterealtime:main Sep 4, 2023
12 checks passed
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.

1 participant