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

housekeeping: Correct ReactiveWindow usage example #1899

merged 1 commit into from Jan 10, 2019


None yet
3 participants
Copy link

worldbeater commented Jan 10, 2019

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)

This is documentation improvement. Related to #1888

What is the current behavior? (You can also link to an open issue here)

When bootstrapping a new WPF project with ReactiveUI, I copy-pasted XAML markup snippet from API docs page and it didn't compile due to missing xmlns directives and wrong x:Class syntax.

What is the new behavior (if this is a feature change)?

Now, the sample compiles, and uses correct syntax and correct xmlns directives.

What might this PR break?


@worldbeater worldbeater requested a review from reactiveui/wpf-team as a code owner Jan 10, 2019

@worldbeater worldbeater requested a review from reactiveui/learning-team Jan 10, 2019


This comment has been minimized.

Copy link

codecov bot commented Jan 10, 2019

Codecov Report

Merging #1899 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@          Coverage Diff           @@
##           master   #1899   +/-   ##
  Coverage    66.4%   66.4%           
  Files         110     110           
  Lines        4387    4387           
  Branches      579     579           
  Hits         2913    2913           
  Misses       1305    1305           
  Partials      169     169
Impacted Files Coverage Δ
src/ReactiveUI.Wpf/ReactiveWindow.cs 0% <ø> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 9e20f25...d9ead56. Read the comment docs.

@glennawatson glennawatson merged commit 3246efa into master Jan 10, 2019

4 checks passed

ReactiveUI Tests #9.7.17+ef32dfd103 succeeded
codecov/patch Coverage not affected when comparing 9e20f25...d9ead56
codecov/project 66.4% remains the same compared to 9e20f25
license/cla All CLA requirements met.

@delete-merged-branch delete-merged-branch bot deleted the correct-reactive-window branch Jan 10, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment