Ensure IEquatable takes nullable type

[Evangelink]

Ensure that IEquatable<T> targets nullable type and update related implementation.

Related Issue

Follow-up of #2765

[asbjornu]

Can you please provide a bit more context and information regarding this change? I hoped we would get fewer ? in the codebase going forward, not more?

[Evangelink]

I just assumed that as we were doing null checks in the equals, it was better to explicitly allow nulls but I understand your point. We can probably close the PR.

[asbjornu]

Ok, I see. I suppose it's difficult to ensure these types aren't null earlier in the codebase?

[Evangelink]

At this point yes. I assume/hope it will be easier after a couple of null-check iterations :)

[asbjornu]

Indeed, let's merge this then. Can you please rebase and resolve conflicts?

[Evangelink]

@asbjornu Done.

[mergify]

Thank you @Evangelink for your contribution!

[arturcic]

🎉 This issue has been resolved in version 5.7.0 🎉
The release is available on:

• GitHub Release
• Chocolatey Package
• .Net Global Tool

Your GitReleaseManager bot 📦🚀