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

Removed dependency on Prism.Core (Fixes #872) #875

Merged

Conversation

NightOwl888
Copy link
Contributor

Fixes #872.

Imported Prism.Events namespace as Lucene.Net.Util.Events and dropped dependency on Prism.Core, which is now a commercial product. This uses the commit at https://github.com/PrismLibrary/Prism/releases/tag/DNF, since it is the last version released under the MIT license.

@NightOwl888 NightOwl888 force-pushed the feature/remove-prism-dependency branch from 7ec6b86 to ab0df3f Compare October 18, 2023 14:22
@eladmarg
Copy link
Contributor

@NightOwl888 you're simply amazing

@NightOwl888 NightOwl888 merged commit 96b6862 into apache:master Oct 18, 2023
198 checks passed
NightOwl888 added a commit to NightOwl888/lucenenet that referenced this pull request Oct 19, 2023
…sFiring(): Ignored. This test has failed since apache#875 was submitted. Unfortunately, we had no way of knowing that this test was flakey until we inherited it.
NightOwl888 added a commit that referenced this pull request Oct 19, 2023
…sFiring(): Ignored. This test has failed since #875 was submitted. Unfortunately, we had no way of knowing that this test was flakey until we inherited it. (#884)
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.

Prism is now commercial software
2 participants