-
Notifications
You must be signed in to change notification settings - Fork 1.2k
fix Operation.isExecuting #1080
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
fix Operation.isExecuting #1080
Conversation
@swift-ci please test |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I guess that is about as correct as we can get w/o KVO
@swift-ci please test |
@swift-ci please test and merge |
@phausler I can't see the results of the CI run here even though we both asked to test it. Any idea what's going on? |
@swift-ci please test and merge |
1 similar comment
@swift-ci please test and merge |
Sometimes swift-ci ignores certain channels for some reason. This change looks solid to me. |
@phausler dang, forgot to record a little change in the tests so tests will fail :. Pushing a new commit any second now |
@swift-ci please test |
1 similar comment
@swift-ci please test |
@phausler this now passed the checks. Didn't merge it as I made some (cosmtic) changes in the tests and wanted to give you the chance to have a look at that. |
No description provided.