Skip to content

Conversation

@panosru
Copy link
Contributor

@panosru panosru commented Aug 7, 2022

The code has been modified to follow the CA1068 rule, which indicates that the CancellationToken parameter must come last, while in some of MediatR methods the rule was followed, in many others it wasn't.

https://docs.microsoft.com/en-us/dotnet/fundamentals/code-analysis/quality-rules/ca1068

The tests are also modified accordingly:

image

Handle and Publish now taking the next as a second param and the cancelationToken as the last param.

Although this pull request isn't backwards compatible, the modification isn't hard to implement, all it needs is to switch places between next and cancelationToken arguments.

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.

2 participants