MediatR icon indicating copy to clipboard operation
MediatR copied to clipboard

Allows for generic request handlers to be resolved.

Open WillMarcouiller opened this issue 3 years ago • 5 comments

For more information please refer to this so question: https://stackoverflow.com/questions/69546722/ninject-just-wont-register-mediatr-irequesthandler-using-convention-based-bi

WillMarcouiller avatar Oct 25 '21 14:10 WillMarcouiller

Can you add a test to show a scenario that this addresses?

jbogard avatar Nov 05 '21 13:11 jbogard

Can you add a test to show a scenario that this addresses?

Just saw your answer, sorry for delay. In fact, I just can't see what kind of a test I could write to demonstrate what it addresses apart the given SO link provided (https://stackoverflow.com/questions/69546722/ninject-just-wont-register-mediatr-irequesthandler-using-convention-based-bi).

I'll try and perhaps come up with something. In the meantime, I would not have created this request if I had not experienced any trouble.

WillMarcouiller avatar Feb 02 '22 21:02 WillMarcouiller

No I understand you had an issue, but I don't like taking PRs that change behavior without a unit test that demonstrates what behavior this addresses. Otherwise I might break this behavior in the future.

jbogard avatar Feb 03 '22 14:02 jbogard

No I understand you had an issue, but I don't like taking PRs that change behavior without a unit test that demonstrates what behavior this addresses. Otherwise I might break this behavior in the future.

I totally understand it. I'll write a test to cover this issue then. :)

WillMarcouiller avatar Feb 05 '22 17:02 WillMarcouiller

No I understand you had an issue, but I don't like taking PRs that change behavior without a unit test that demonstrates what behavior this addresses. Otherwise I might break this behavior in the future.

I'm gonna need a hand to make the solution work on my computer as I just can't compile the cloned project.

First, I cloned using:

gh repo clone jbogard/MediatR gh/MediatR cd /gh/MediatR gh pr checkout 665

Then I can see my changes within the ContravariantBindingResolver class in the Ninject sample.

What I don't understand is how can't I even make it restore and compile? Here's a screenshot of the error list. This happens when I rebuild the entire solution. It seems to bug on Ninject, and I suspect the problem is deeper than that. Help would be appreciated. I'm not used to contribute on gh.

image

WillMarcouiller avatar Feb 12 '22 02:02 WillMarcouiller

This PR is stale because it has been open 28 days with no activity. Remove stale label or comment or this will be closed in 14 days.

github-actions[bot] avatar Nov 02 '22 12:11 github-actions[bot]

This PR was closed because it has been stalled for 14 days with no activity.

github-actions[bot] avatar Nov 17 '22 12:11 github-actions[bot]