Skip to content
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

Move DriverTestCase to the test module #4922

Merged
merged 3 commits into from
Jan 12, 2023

Conversation

jenshnielsen
Copy link
Collaborator

This has no good reason to live in the test module. This is a breaking change but I don't expect this to be used in a way where this is important enough for a shim to be added

@codecov
Copy link

codecov bot commented Jan 12, 2023

Codecov Report

Merging #4922 (b3c6841) into master (b3c6841) will not change coverage.
The diff coverage is n/a.

❗ Current head b3c6841 differs from pull request most recent head 2793094. Consider uploading reports for the commit 2793094 to get more accurate results

@@           Coverage Diff           @@
##           master    #4922   +/-   ##
=======================================
  Coverage   68.39%   68.39%           
=======================================
  Files         352      352           
  Lines       32349    32349           
=======================================
  Hits        22126    22126           
  Misses      10223    10223           

@jenshnielsen jenshnielsen merged commit 3083ef9 into microsoft:master Jan 12, 2023
@jenshnielsen jenshnielsen deleted the driver/move_test_case branch January 12, 2023 15:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants