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

#889 Only include system.memory on framework #890

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

thompson-tomo
Copy link

Closes #889

@devOpsHazelcast
Copy link
Contributor

devOpsHazelcast commented Apr 26, 2024

CLA assistant check
All committers have signed the CLA.

Copy link

netlify bot commented Apr 26, 2024

Deploy Preview for silly-valkyrie-e996d9 canceled.

Name Link
🔨 Latest commit 0575762
🔍 Latest deploy log https://app.netlify.com/sites/silly-valkyrie-e996d9/deploys/6680a04eb37a34000803736d

@emreyigit
Copy link
Collaborator

Hi @thompson-tomo, the client doesn't support framework 4.5. In this case, we can't merge this PR.
https://hazelcast.github.io/hazelcast-csharp-client/versions.html

@thompson-tomo
Copy link
Author

sorry, i think i got confused looking at the conditional compilation hence have gone and cleaned up the conditional compilation and simply dropped the dependency all together.

@thompson-tomo thompson-tomo force-pushed the chore/#889_OptimiseDependencies branch from 50da3ee to a83dbad Compare May 21, 2024 03:03
@thompson-tomo thompson-tomo force-pushed the chore/#889_OptimiseDependencies branch from a83dbad to 0575762 Compare June 30, 2024 00:01
@thompson-tomo thompson-tomo force-pushed the chore/#889_OptimiseDependencies branch from 0575762 to f57b072 Compare July 10, 2024 11:34
@thompson-tomo thompson-tomo force-pushed the chore/#889_OptimiseDependencies branch 2 times, most recently from bf8264b to 79ed818 Compare July 23, 2024 12:24
@thompson-tomo thompson-tomo force-pushed the chore/#889_OptimiseDependencies branch from 79ed818 to 1ec54e9 Compare August 6, 2024 13:05
@thompson-tomo thompson-tomo force-pushed the chore/#889_OptimiseDependencies branch from 1ec54e9 to 5a60156 Compare August 25, 2024 11:40
@thompson-tomo thompson-tomo force-pushed the chore/#889_OptimiseDependencies branch from 5a60156 to 2f9a5be Compare October 26, 2024 22:40
@thompson-tomo thompson-tomo force-pushed the chore/#889_OptimiseDependencies branch from 2f9a5be to 6aac449 Compare January 2, 2025 10:23
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.

Optimise dependencies to leverage framework
3 participants