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

Pin FluentAssertions dependency to prevent future updates #232

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Asafima
Copy link
Contributor

@Asafima Asafima commented Jan 17, 2025

FluentAssertions v8 is now paid for commercial use, package manager should prevent any updates so that FluentResults could be used commercially.

Next phase should probably be passing all dependencies to Shouldly ,
which is now sponsored by FluentAssertions old sponsors.

Closes #231

@savornicesei
Copy link

Either move to Shouldly or replace it with it's already released fork AwesomeAssertions

@Asafima
Copy link
Contributor Author

Asafima commented Jan 17, 2025

@savornicesei
The problem with all forks is maintenance, often most forks are not maintained, that’s why I guess Shouldly is the better option.

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.

Pin FluentAssertions nuget version
2 participants