Microsoft.Toolkit.Mvvm package (part 2)

[Sergio0694]

Follow up to #3229

NOTE

Marking this not as a draft so that the CI will run, but added the "DO NOT MERGE ⚠" tag as we might come up with further improvements to add to this PR before actually deciding to merge this and finalize the package.

PR Type

What kind of change does this PR introduce?

• Bugfix
• Feature
• Refactoring (no functional changes, no api changes)

Notable changes

This PR includes some refinements and tweaks to the Microsoft.Toolkit.Mvvm package, like:

• Added new ObservableRecipient.SetProperty overloads with Expression<Func<T>> and bool broadcast params
• Fixed a possible bug in Messenger.Type2.Equals (just to be extra sure)
• Improved codegen for ObservableRecipient.Set<T>(ref T, T, string) (missing EqualityComparer<T>.Default.Equals inlining)

PR Checklist

Please check if your PR fulfills the following requirements:

• Tested code with current supported SDKs
• Pull Request has been submitted to the documentation repository instructions. Link:
• Sample in sample app has been added / updated (for bug fixes / features)
  • Icon has been created (if new sample) following the Thumbnail Style Guide and templates
• Tests for the changes have been added (for bug fixes / features) (if applicable)
• Header has been added to all new source files (run build/UpdateHeaders.bat)
• Contains NO breaking changes

[ghost]

Thanks Sergio0694 for opening a Pull Request! The reviewers will test the PR and highlight if there is any conflict or changes required. If the PR is approved we will proceed to merge the pull request 🙌

[ghost]

Hello @azchohfi!

Because this pull request has the auto merge label, I will be glad to assist with helping to merge this pull request once all check-in policies pass.

p.s. you can customize the way I help with merging this pull request, such as holding this pull request until a specific person approves. Simply @mention me (@msftbot) and give me an instruction to get started! Learn more here.