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

Fix issue in ReactiveUI.Wpf when windows10.0.19041 is selected at runtime by end user #3494

Merged
merged 1 commit into from Mar 4, 2023

Conversation

ChrisPulman
Copy link
Member

What kind of change does this PR introduce?

Fix for #3493

What is the current behaviour?

Works on all targets except windows10.0.19041

What is the new behaviour?

Works on all targets
Added windows10.0.19041.0 target as a workaround for System.Reactive targeting net5.0-windows10.0.19041.0
When windows10.0.19041.0 is selected by the end user it causes the DispatcherScheduler in System.Reactive to be exposed and used.

What might this PR break?

none expected

Please check if the PR fulfils these requirements

  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been added / updated (for bug fixes / features)

Other information:

Fix for #3493
Added 19041 target as a workaround for System.Reactive targeting net5.0-windows10.0.19041.0
When 19041 is selected by the end user it causes the DispatcherScheduler in System.Reactive to be exposed and used.
@codecov
Copy link

codecov bot commented Mar 4, 2023

Codecov Report

Patch coverage has no change and project coverage change: -0.23 ⚠️

Comparison is base (e666feb) 63.83% compared to head (e0c4f2c) 63.60%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3494      +/-   ##
==========================================
- Coverage   63.83%   63.60%   -0.23%     
==========================================
  Files         157      157              
  Lines        5768     5768              
==========================================
- Hits         3682     3669      -13     
- Misses       2086     2099      +13     
Impacted Files Coverage Δ
.../ReactiveUI/Platforms/net/PlatformRegistrations.cs 57.14% <0.00%> (-28.58%) ⬇️
src/ReactiveUI.Wpf/Registrations.cs 82.35% <0.00%> (-11.77%) ⬇️
src/ReactiveUI.Winforms/Registrations.cs 82.35% <0.00%> (-11.77%) ⬇️
.../Converter/NullableIntegerToStringTypeConverter.cs 79.16% <0.00%> (-8.34%) ⬇️
src/ReactiveUI/RxApp.cs 75.00% <0.00%> (-6.95%) ⬇️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@ChrisPulman ChrisPulman enabled auto-merge (squash) March 4, 2023 18:07
@glennawatson glennawatson merged commit 2ebf67d into main Mar 4, 2023
@glennawatson glennawatson deleted the CP_FIxDispatcherScheduler branch March 4, 2023 21:04
@github-actions
Copy link

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 19, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug]: CS0433 for DispatcherScheduler with 18.4.22 on net7.0-windows10.0.19041
2 participants