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.