NightOwl888 commented on issue #872: URL: https://github.com/apache/lucenenet/issues/872#issuecomment-1752590728
> I'd not use `WeakEventManager`. It's tied to WPF/WindowsBase, which is something a lot of projects do not want to pull in. Agreed. WeakEventManager seems over-engineered, anyway. IMO, the best option would be to bring in the types from Prism into our codebase under a commit with the former MIT license. We haven't had any complaints with this solution from .NET Framework users. Removing the dependency on Prism will avoid any "gotcha" issues with people upgrading and being out of compliance with the license. @rauhs - It is [Hacktoberfest](https://hacktoberfest.com/) month, so if you wish to submit a PR it will apply toward your free swag for participation. -- 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: dev-unsubscr...@lucenenet.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
