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

Feature/delete UI updates#4

Merged
jbarden merged 3 commits into
mainfrom
feature/delete-ui-updates
Apr 2, 2026
Merged

Feature/delete UI updates#4
jbarden merged 3 commits into
mainfrom
feature/delete-ui-updates

Conversation

@jaybarden1

@jaybarden1 jaybarden1 commented Apr 2, 2026

Copy link
Copy Markdown
Contributor

Summary

got a little carried away... cleanup, logging and delete ui updates...


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 00:13
@jbarden jbarden enabled auto-merge (squash) April 2, 2026 00:13
@jbarden jbarden merged commit 031fba6 into main Apr 2, 2026
3 of 9 checks passed
@jbarden jbarden deleted the feature/delete-ui-updates branch April 2, 2026 00:13
@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