8000 Upgrade MediatR to v12.4.1 to support updated AddMediatRClasses signature by ssuermann · Pull Request #6403 · elsa-workflows/elsa-core · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Upgrade MediatR to v12.4.1 to support updated AddMediatRClasses signature #6403

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

Merged

Conversation

ssuermann
Copy link
@ssuermann ssuermann commented Feb 12, 2025

Summary

This PR upgrades the MediatR package from v12.0.1 to v12.4.1. The update was required because the
AddMediatRClasses method in MediatR.Registration.ServiceRegistrar now includes an optional
CancellationToken parameter. Without this upgrade, solutions using the latest MediatR version would
encounter a MissingMethodException due to the absence of the old method signature.

Changes

  • Updated the MediatR package reference to v12.4.1.
  • Ensured compatibility across the project.

Rationale

The introduction of the CancellationToken parameter in the AddMediatRClasses method by MediatR means
that maintaining compatibility with the latest version is crucial. This change prevents runtime errors
and ensures that the project remains up-to-date with its dependencies.

Testing

All unit tests have been executed and passed successfully.


This change is Reviewable

…ture

Updated MediatR from v12.0.1 to v12.4.1 to accommodate the new optional
CancellationToken parameter in MediatR.Registration.ServiceRegistrar.AddMediatRClasses.
This resolves the MissingMethodException issue when using the latest MediatR version.
All unit tests passed.
@ssuermann
Copy link
Author

@dotnet-policy-service agree company="adesso se"

@sfmskywalker sfmskywalker merged commit 4191291 into elsa-workflows:2.x Feb 23, 2025
6 of 8 checks passed
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
0