Conversation
There was a problem hiding this comment.
Pull request overview
This PR fixes an inconsistency in MSBuild property naming within the test configuration. The change renames CollectStatement to CollectArgument to align with the existing naming convention used for other dotnet test command-line argument properties.
Changes:
- Renamed the MSBuild property
CollectStatementtoCollectArgumentto match the naming pattern used byFilterArgumentandBlameArgument
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #3975 +/- ##
=========================================
+ Coverage 0 64.59% +64.59%
=========================================
Files 0 282 +282
Lines 0 66415 +66415
=========================================
+ Hits 0 42903 +42903
- Misses 0 23512 +23512
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
Report on CodeCov looks good: https://app.codecov.io/github/dotnet/sqlclient/tree/dev%2Fmdaigle%2Ffix-code-coverage |
No description provided.