search

jbogard / MediatR

Simple, unambitious mediator implementation in .NET
Apache License 2.0
10.84k stars 1.15k forks source link

Moved interfaces to Contracts #1015

Closed harrypooni closed 2 months ago

harrypooni commented 3 months ago

Extracted Interfaces to Contracts to provide abstraction to multi project solutions.

Mayron commented 3 months ago

@harrypooni I have created my own pull request based on the discussion in the issue,

I think your PR has a problem by including MediatR.Contracts as a project reference when it should only depend on it via a NuGet package.

The PR I created tries to create the other abstractions/interfaces as additional NuGet packages and MediatR could eventually be updated to include them as dependencies instead of directly referencing the projects. Also, the total number of files changed is much smaller because of this, which is always a plus!

jbogard commented 2 months ago

No, I really really really don't want to have to maintain all these packages and projects when I've never needed to have them broken out as such. I do not have any scenarios where breaking them out further would enable some scenario or solve some dependency problem.