Havret commented on PR #92: URL: https://github.com/apache/activemq-nms-amqp/pull/92#issuecomment-3029183862
Thank you for your contribution and for taking the time to submit this PR! After reviewing the changes, I don’t see a clear benefit to specifically targeting .NET 6 (or .NET 8) in this project. Since the library already targets .NET Standard, it remains fully compatible with .NET 6, .NET 8, and future runtimes. The runtime will utilize the most up-to-date implementations of dependencies when running on newer platforms, so adding a .NET 6 target doesn’t provide additional optimization or compatibility. If there’s a specific issue or scenario you’re aiming to address with this change, could you please elaborate? Otherwise, I think we can keep the current approach. Thanks again for your effort and looking forward to your thoughts! -- 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: gitbox-unsubscr...@activemq.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: gitbox-unsubscr...@activemq.apache.org For additional commands, e-mail: gitbox-h...@activemq.apache.org For further information, visit: https://activemq.apache.org/contact
