-
-
Notifications
You must be signed in to change notification settings - Fork 907
Updates Nimble submdule commit. #507
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
Conversation
Thanks!! Hopefully peeps aren't using the version of Nimble Quick's tests depend on as their Nimble dependency, but I have a feeling some do. Feel free to merge once CI passes. |
Hmm, both builds stalled out. Should I just try to restart? |
Ha, looks like this happened the last time we upped the Nimble submodule commit: #505 I'll try restarting. |
The same test is failing on both CI providers, probably not a coincidence... Since the Nimble change we're trying to bring in has to do with when test observation starts, I'm willing to bet it's a valid failure. I'll look into it and report back. |
Tests didn't fail when running within Xcode. Invoking |
Hmm, might be a trickier issue than expected. Kif ran into similar problems that Quick/Nimble#271 was supposed to fix, but decided not to use the solution we did Suggestions on how to proceed would be awesome 🙇 I feel a little out of my depth. |
Looks like KIF's solution was to move their test observation functionality into the The Nimble test observation logic appears to only be used when |
Hmm. You reckon its worth opening an issue on Nimble to discuss options? Maybe there's another hook we could use, and |
Yes, let's open a Nimble issue to discuss. I've been doing some debugging and have a theory. |
@ashfurrow can you update your branch to Quick/Nimble@1a9e53e? |
3023674
to
c44d7b6
Compare
Done 👍 Let's see how CI goes 😄 |
Aw yeah! 🚀 |
Awesome! Thanks guys 🙌 |
Time for a release? |
Sure! @phatblat, I've added you as a |
There's also the nice section in the Contributing document about making releases: https://github.com/Quick/Quick/blob/master/CONTRIBUTING.md#creating-a-release |
As discussed, we need to update this to get the latest fix for #494.