Closed eric closed 4 years ago
Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).
:memo: Please visit https://cla.developers.google.com/ to sign.
Once you've signed (or fixed any issues), please reply here with @googlebot I signed it!
and we'll verify it.
ℹ️ Googlers: Go here for more info.
@googlebot I signed it!
When I try to move it under FBLPromise+Testing.h
it gives me a warning that it wants me to implement it in FBLPromise+Testing.m
. Should I do that?
I've updated based on comments.
Huh... the tests were passing for me. I wonder what’s going on.
You can find the details on Travis CI
I realized I was only running the Swift tests and not Objective-C ones. This should be better now.
Haha. Would you rather I get rid of the checks or just keep them?
Feel free to remove them as the property is already gone.
I've removed it. It should be ready to merge.
Swift uses the NSMutableSet to hold on to references of Swift promises but the modifications must be synchronized.
This is intended to fix #141.