-
Notifications
You must be signed in to change notification settings - Fork 5.8k
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
migrate test-infra to testify for ddl/stat_test.go
#29114
Comments
/assign |
@tisonkun i notice |
@tisonkun The migration of a single test file is not very easy to do. Why don't I replace all of them with testing and remove the previous tests using Check |
@jayl-zxl you can always take multiple issues at the same time. |
@tisonkun i see .. |
@jayl-zxl I'm ok if you'd like to just rewrite the tests. But maybe you should reach a consensus with ones of @tiancaiamao @zimulala @djshow832 . Just create a new issue and close these migration tests if that one accepted and resolved. |
/assign |
No description provided.
The text was updated successfully, but these errors were encountered: