Hide ToDictionary() in net8+

[viceroypenguin]

This PR updates ToDictionary() to be hidden on net8, since the BCL has added a duplicate method.

Fixes #1012
Relates to #945

[atifaziz]

Now that #1041 is merged with 9a4ea61, can we resolve the conflicts so there's a smaller diff to review? Thanks!

[viceroypenguin]

Now that #1041 is merged with 9a4ea61, can we resolve the conflicts so there's a smaller diff to review? Thanks!

Done!

[atifaziz]

Done!

Thanks, though the build is broken. Could you have a look?

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (9a4ea61) 92.62% compared to head (c9144e4) 92.52%.
Report is 5 commits behind head on master.

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #1040      +/-   ##
==========================================
- Coverage   92.62%   92.52%   -0.11%     
==========================================
  Files         113      113              
  Lines        3419     3413       -6     
  Branches     1055     1058       +3     
==========================================
- Hits         3167     3158       -9     
  Misses        189      189              
- Partials       63       66       +3     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[atifaziz]

Thanks for your help with this!

---

PS I reverted some unrelated formatting changes with c9144e4, but I would appreciate it if you can avoid them in the first place in your PRs; it really helps with reducing the review time. Thanks.