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

test: FluentAssertions removed #346

Merged
merged 5 commits into from
Jan 25, 2025
Merged

Conversation

marklechtermann
Copy link
Member

Removed FluentAssertions

fluentassertions/fluentassertions#2943

Fluent assertions has changed its license model.

See video from Nick: https://www.youtube.com/watch?v=ZFc6jcaM6Ms&t=2s

Fluent assertions is just a nice to have library.
All functions can also be replaced by the classic Xunit.Assert. This is not fluent but who cares.
I would not recommend any company to use FluentAssertions and then pay 130$ per developer for it.
If you had never had a library like fluentassertions before, nobody would have thought of paying 130$ for it.

I understand it when it comes to a library where there is still a lot of development work to be done.
But the FluentAssertions Library is feature complete.

If I change the license of a library that is actually in maintanance mode, then it is a bit immoral to ask for money now if you have pushed another commercial library out of the market for years by making your library open source.

https://www.reddit.com/r/dotnet/comments/1i17jm0/fluentassertions_becomes_paid_software_for/?rdt=58726

There are many reasons why a person would not want to be paid for their open source work.

  • They may already have a full-time job that they love, which enables them to contribute to open source in their spare time.

https://opensource.guide/getting-paid/

Modern companies encourage their employees to participate in open source even during their working hours.
None of us developers can give away our working time without financial compensation.
Btw. Thank you dSPACE for allowing us to contribute to open source projects .
Even if DSCOM is only a small library, we remain OpenSource and free for use ;-)

This is just my opinion ❤️

Copy link
Member

@matthiasnissen matthiasnissen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for your effort in changing so many occurrences!

@matthiasnissen matthiasnissen merged commit 74f069c into main Jan 25, 2025
5 checks passed
@marklechtermann marklechtermann deleted the test/fluentassertions_removed branch February 8, 2025 14:18
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