[GitHub] [geode-native] moleske commented on pull request #428: GEODE-6241: Makes .NET integration tests more consistent with C++

Stray comment [ Full content available at: https://github.com/apache/geode-native/pull/428 ] This message was relayed via gitbox.apache.org for notifications@geode.apache.org

[GitHub] [geode-native] moleske commented on pull request #428: GEODE-6241: Makes .NET integration tests more consistent with C++

I don't think you need `IDisposable` anymore since you removed all the code of `Dispose()` below, though I don't think it really hurts anything. [ Full content available at: https://github.com/apache/geode-native/pull/428 ] This message was relayed via gitbox.apache.org for notifications@geode.a

[GitHub] [geode-native] moleske commented on pull request #428: GEODE-6241: Makes .NET integration tests more consistent with C++

Just want to say I like this section, much easier to read [ Full content available at: https://github.com/apache/geode-native/pull/428 ] This message was relayed via gitbox.apache.org for notifications@geode.apache.org