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

Dispose keyEnumerator #690

Merged
merged 1 commit into from
Oct 19, 2024
Merged

Conversation

Henr1k80
Copy link
Contributor

Part of #687

@stakx
Copy link
Member

stakx commented Oct 16, 2024

I think I'm going to leave this PR for someone else to review, even though it looks trivial and basically correct. My focus with this project is on DynamicProxy, and I suspect that most other facilities in this library are essentially legacy anyway (perhaps with the exception of the logging facilities). I'm not going to invest any of my personal time here... sorry. :-/

@stakx stakx merged commit 28b8715 into castleproject:master Oct 19, 2024
4 checks passed
@stakx
Copy link
Member

stakx commented Oct 19, 2024

I have a tiny bit of time to kill today, so why not. :-) Thanks @Henr1k80!

@stakx stakx added this to the vNext milestone Oct 19, 2024
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.

2 participants