brudo commented on PR #23: URL: https://github.com/apache/activemq-nms-openwire/pull/23#issuecomment-1165878579
Oh, shoot - I hasdn't seen that existing #21 where it was already done, to make it "None". But there are other fixes included in the current PR, to clear up some compile-time warnings. Should I remove the SslContext commits (and maybe also the SharpZip version increment, which the dependabot picked up in #22)? Or should I close this and submit a new PR with just the changes that were not in #21 or #22? Or do you want to merge this PR with all squashed together, with a commit message that conforms to https://cwiki.apache.org/confluence/display/GEODE/Commit+Message+Format? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
