Skip to content

Conversation

@ChrisPulman
Copy link
Member

What kind of change does this PR introduce?

fixes #351

What is the current behavior?

#351

What is the new behavior?

Moved several model classes from 'ReactiveUI.SourceGenerators.Input.Models', 'ReactiveUI.SourceGenerators.Reactive.Models', and 'ReactiveUI.SourceGenerators.ObservableAsProperty.Models' to 'ReactiveUI.SourceGenerators.Models'.

Updated all related using statements and namespace declarations to reflect this change.

Also updated references to ReactiveCommand and RxSchedulers in ReactiveCommandGenerator for clarity and global namespace usage.
This improves code organization and consistency across source generators.

What might this PR break?

None expected

Please check if the PR fulfills these requirements

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

Other information:

Moved several model classes from 'ReactiveUI.SourceGenerators.Input.Models', 'ReactiveUI.SourceGenerators.Reactive.Models', and 'ReactiveUI.SourceGenerators.ObservableAsProperty.Models' to 'ReactiveUI.SourceGenerators.Models'. Updated all related using statements and namespace declarations to reflect this change. Also updated references to RxSchedulers in ReactiveCommandGenerator for clarity and global namespace usage. This improves code organization and consistency across source generators.
@codecov
Copy link

codecov bot commented Jan 3, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 43.22%. Comparing base (215611a) to head (bb2a112).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #362   +/-   ##
=======================================
  Coverage   43.22%   43.22%           
=======================================
  Files          64       64           
  Lines        3646     3646           
  Branches      420      420           
=======================================
  Hits         1576     1576           
  Misses       1950     1950           
  Partials      120      120           

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: ReactiveCommand generator doesn't generate global:: for ReactiveUI.ReactiveCommand<,>

2 participants