Skip to content
This repository was archived by the owner on Apr 2, 2026. It is now read-only.

refactor: converted to ReactiveUI, builds, let's see if it runs...#5

Merged
jaybarden1 merged 1 commit into
mainfrom
feature/migrate-to-reactiveui
Apr 2, 2026
Merged

refactor: converted to ReactiveUI, builds, let's see if it runs...#5
jaybarden1 merged 1 commit into
mainfrom
feature/migrate-to-reactiveui

Conversation

@jaybarden1

Copy link
Copy Markdown
Contributor

Summary

Please include a brief description of the change and the motivation.


TDD Checklist (required)

  • I added one or more failing tests that demonstrate the desired behavior before implementing production code.
  • I confirmed the new test(s) fail locally before writing production code.
  • I implemented the minimal production code to make the tests pass.
  • I ran the full test suite locally and all tests pass.
  • The failing-test commit is included in this branch history (the failing test must be present in the PR history before or alongside production code).

If the TDD checklist is not applicable for this PR (e.g., documentation-only or chore), explain why in the description.


How to run tests locally

# Restore and run tests
dotnet restore AStar.Dev.OneDrive.Sync.Client.slnx
dotnet test --verbosity normal

Notes for reviewers

  • Verify that the PR history includes the failing-test commit (or an explanation why not).
  • Ensure CI passed for all OS runners.

@jaybarden1 jaybarden1 requested a review from a team April 2, 2026 10:28
@jaybarden1 jaybarden1 merged commit 1badd59 into main Apr 2, 2026
3 of 9 checks passed
@jaybarden1 jaybarden1 deleted the feature/migrate-to-reactiveui branch April 2, 2026 10:28
@sonarqubecloud

sonarqubecloud Bot commented Apr 2, 2026

Copy link
Copy Markdown

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.

2 participants