-
Notifications
You must be signed in to change notification settings - Fork 1k
Add test for PR 13320 #13342
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
Add test for PR 13320 #13342
Conversation
@LeafShi1 - can we avoid SendKeys as well? |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #13342 +/- ##
===================================================
- Coverage 76.61355% 76.57947% -0.03409%
===================================================
Files 3237 3243 +6
Lines 639477 640152 +675
Branches 47319 47389 +70
===================================================
+ Hits 489926 490225 +299
- Misses 146030 146289 +259
- Partials 3521 3638 +117
Flags with carried forward coverage won't be shown. Click here to find out more. 🚀 New features to boost your workflow:
|
23185b8
to
f5934ca
Compare
Hey all, before we push changes - can we make sure for efficiency reasons, that a command-line build (not a build inside of VS) will pass before we check in? That will at least prevent to have certain warnings or other compile errors, which would not be shown immediately in the UI. Thanks! |
…ode_WithBindingSource"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
All LGTM, but conflicts need to be solved.
8a9d7a9
to
892c59a
Compare
Proposed changes
Microsoft Reviewers: Open in CodeFlow