-
Notifications
You must be signed in to change notification settings - Fork 12.9k
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
Delete long-ignored and useless test #57910
Conversation
The job Click to expand the log.
I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
697f138
to
6d69389
Compare
Let's keep it. Perhaps comments in the test could be adjusted though. |
(Totally off-topic, but speaking of unused tests, the whole |
I believe that directory is gone after my quote macro removal PR. |
Ah, reopening to track adjusting the comment. |
6d69389
to
e0bc0ba
Compare
r? @petrochenkov -- updated the comment, probably best to wait for Travis as I've not tested locally |
@bors r+ rollup |
📌 Commit e0bc0ba has been approved by |
Delete long-ignored and useless test cc @pnkfelix (#13745 (comment)) r? @petrochenkov as you re-enabled this test in 21d67c4, #55236
☀️ Test successful - checks-travis, status-appveyor |
cc @pnkfelix (#13745 (comment))
r? @petrochenkov as you re-enabled this test in 21d67c4, #55236