Skip to content

[d16-0 P3] Merge pull request #5402 from migueldeicaza/fix-uigesturerecognizer-r… - #5414

Merged
chamons merged 1 commit into
dotnet:d16-0from
chamons:d16-0-p3-UIGestureRecognizerFix
Jan 30, 2019
Merged

[d16-0 P3] Merge pull request #5402 from migueldeicaza/fix-uigesturerecognizer-r…#5414
chamons merged 1 commit into
dotnet:d16-0from
chamons:d16-0-p3-UIGestureRecognizerFix

Conversation

@chamons

@chamons chamons commented Jan 15, 2019

Copy link
Copy Markdown
Contributor

…egression

[UIKit] UIGestureRecognizer's custon OnDispose method needs to queue the actual operation on the main thread

#5388

…izer-regression

[UIKit] UIGestureRecognizer's custon OnDispose method needs to queue the actual operation on the main thread
@chamons chamons added do-not-merge Do not merge this pull request requires-approval-before-merge The pull request requires special approval before it can be merged labels Jan 15, 2019

@VincentDondain VincentDondain left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let us get the test fix in this PR before merging. See: #5402 (comment)

@monojenkins

Copy link
Copy Markdown
Contributor

Build success
Build succeeded
API Diff (from stable)
ℹ️ API Diff (from PR only) (please review changes)
Generator Diff (no change)
Test run succeeded

@rolfbjarne rolfbjarne left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 once the failing test is fixed.

@chamons chamons removed do-not-merge Do not merge this pull request requires-approval-before-merge The pull request requires special approval before it can be merged labels Jan 18, 2019
@chamons

chamons commented Jan 30, 2019

Copy link
Copy Markdown
Contributor Author

#5488 went in to "fix" test failure.

@chamons
chamons merged commit 274c5c8 into dotnet:d16-0 Jan 30, 2019
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.

6 participants