Skip to content

Conversation

@h3xds1nz
Copy link
Member

@h3xds1nz h3xds1nz commented Jan 11, 2025

Description

Adds test coverage for MouseGestureConverter. Sending these ahead of time for PR related to MouseGestureConverter.

Customer Impact

Improved test coverage on public API surface.

Regression

No.

Testing

Local build.

Risk

None.

Microsoft Reviewers: Open in CodeFlow

@h3xds1nz h3xds1nz requested review from a team as code owners January 11, 2025 17:49
@dotnet-policy-service dotnet-policy-service bot added PR metadata: Label to tag PRs, to facilitate with triage Community Contribution A label for all community Contributions labels Jan 11, 2025
@dipeshmsft dipeshmsft self-assigned this Jan 21, 2025
Copy link
Member

@dipeshmsft dipeshmsft left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@dipeshmsft dipeshmsft merged commit 928d828 into dotnet:main Jan 21, 2025
8 checks passed
@dipeshmsft
Copy link
Member

Thanks @h3xds1nz for the following contributions on addition of tests.

@h3xds1nz
Copy link
Member Author

@dipeshmsft Thank you :)

P.S. Would be nice if the other 3 test-only PRs weren't sad sitting there and got merged too 😀

@dipeshmsft
Copy link
Member

Already on those, btw did I miss any ? I reviewed 3 of them I guess.

@h3xds1nz
Copy link
Member Author

Already on those, btw did I miss any ? I reviewed 3 of them I guess.

They've been rotting here for a bit.

#9895
#9891

@dipeshmsft
Copy link
Member

@h3xds1nz I will take a look at them next.

@h3xds1nz
Copy link
Member Author

@dipeshmsft Great, thanks :)

@github-actions github-actions bot locked and limited conversation to collaborators Feb 21, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Community Contribution A label for all community Contributions PR metadata: Label to tag PRs, to facilitate with triage

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants