Skip to content

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

Merged
merged 1 commit into from
Jun 30, 2017
Merged

fix Operation.isExecuting #1080

merged 1 commit into from
Jun 30, 2017

Conversation

weissi
Copy link
Contributor

@weissi weissi commented Jun 27, 2017

No description provided.

@weissi weissi requested a review from phausler June 27, 2017 16:10
@weissi
Copy link
Contributor Author

weissi commented Jun 27, 2017

@swift-ci please test

Copy link
Contributor

@phausler phausler left a 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

@weissi
Copy link
Contributor Author

weissi commented Jun 27, 2017

@swift-ci please test

@phausler
Copy link
Contributor

@swift-ci please test and merge

@weissi
Copy link
Contributor Author

weissi commented Jun 28, 2017

@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?

@weissi
Copy link
Contributor Author

weissi commented Jun 28, 2017

@swift-ci please test and merge

1 similar comment
@weissi
Copy link
Contributor Author

weissi commented Jun 29, 2017

@swift-ci please test and merge

@phausler
Copy link
Contributor

Sometimes swift-ci ignores certain channels for some reason. This change looks solid to me.

@weissi
Copy link
Contributor Author

weissi commented Jun 29, 2017

@phausler dang, forgot to record a little change in the tests so tests will fail :. Pushing a new commit any second now

@weissi
Copy link
Contributor Author

weissi commented Jun 29, 2017

@swift-ci please test

1 similar comment
@weissi
Copy link
Contributor Author

weissi commented Jun 29, 2017

@swift-ci please test

@weissi
Copy link
Contributor Author

weissi commented Jun 30, 2017

@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.

@phausler phausler merged commit d8c2af3 into swiftlang:master Jun 30, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants