Skip to content

Conversation

thompson-tomo
Copy link

Closes #889

@devOpsHazelcast
Copy link
Contributor

PR closed by Hazelcast automation as no activity (>3 months). Please reopen with comments, if necessary. Thank you for using Hazelcast and your valuable contributions

@thompson-tomo
Copy link
Author

Reopen am awaiting feedback

@emreyigit
Copy link
Collaborator

emreyigit commented Aug 12, 2025

I can't accept this PR since it depends on a specific library.

@thompson-tomo
Copy link
Author

@emreyigit can you explain what you mean.

System.Memory is natively included in the framework as per https://apisof.net/catalog/fd22bb9adc9f2e905e80fbc8eb34b37b starting with net core 2.1 hence it is not required to be added an explicit dependency on the newer frameworks.

@emreyigit
Copy link
Collaborator

Sorry @thompson-tomo. Ignore my previous comment please. I just got your point. I need to verify the changes doesn't break anything. Currently, I have limited time but I've added to my to-do.

@emreyigit emreyigit reopened this Aug 12, 2025
@thompson-tomo
Copy link
Author

No worries, let me know if you have any questions.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Optimise dependencies to leverage framework

3 participants