-
-
Notifications
You must be signed in to change notification settings - Fork 431
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
Revert #621 changes #627
Revert #621 changes #627
Conversation
Great. I should be at a computer within the hour. Assuming all is good I'll look to ship this as 2.3.6 |
Hm. Seems that the CI builds failed at the tests this PR removed. That's weird. May I have overlooked something? |
Weirdly it seems to be failing on the test that your PR removes? Odd |
I'm going take a gamble on this |
I guess/hope it's fine. The builds seemed to fail only on the removed tests. Even more strange that they only failed for two TS versions (1.7 and 2.5). Seems like some kind of weird caching issue to me but I doubt that those "simple" bugs are still around in those established CI tools. ¯\_(ツ)_/¯ |
Problem was to do with some of the test files still hanging around (no idea why). Should now be gone. Just running tests locally now... |
2.3.6 has shipped! |
Great. Thanks for your patience! |
No worries - all is good now; thanks for the PR! 👍 🥇 |
Reverts changes of #621 since they were flawed by design.