Skip to content

Conversation

@LucianoPAlmeida
Copy link
Contributor

Improve diagnostics involving key path root and generic arguments.

cc @xedin @hamishknight
Let me know what you think :)

Resolves SR-12827.

@LucianoPAlmeida LucianoPAlmeida force-pushed the SR-12827-keypath-hole branch from cdeadc8 to f3f34d9 Compare May 19, 2020 22:11
@LucianoPAlmeida LucianoPAlmeida requested a review from xedin May 19, 2020 22:18
@LucianoPAlmeida LucianoPAlmeida force-pushed the SR-12827-keypath-hole branch 6 times, most recently from e399a25 to c61a8e1 Compare May 21, 2020 01:19
@LucianoPAlmeida LucianoPAlmeida requested a review from xedin May 21, 2020 01:23
@LucianoPAlmeida LucianoPAlmeida force-pushed the SR-12827-keypath-hole branch from c61a8e1 to 47509d3 Compare May 21, 2020 09:51
@LucianoPAlmeida LucianoPAlmeida requested a review from xedin May 21, 2020 10:34
@LucianoPAlmeida LucianoPAlmeida force-pushed the SR-12827-keypath-hole branch from 47509d3 to 1097dae Compare May 21, 2020 14:36
…r being bound to hole has a key path root representatee
@LucianoPAlmeida LucianoPAlmeida force-pushed the SR-12827-keypath-hole branch from 1097dae to 3f51637 Compare May 21, 2020 20:15
@LucianoPAlmeida LucianoPAlmeida force-pushed the SR-12827-keypath-hole branch from 3f51637 to f7234a3 Compare May 21, 2020 20:17
@LucianoPAlmeida LucianoPAlmeida requested a review from xedin May 21, 2020 20:20
@LucianoPAlmeida LucianoPAlmeida requested a review from xedin May 21, 2020 21:17
Copy link
Contributor

@xedin xedin left a comment

Choose a reason for hiding this comment

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

Thank you!

@xedin
Copy link
Contributor

xedin commented May 21, 2020

@swift-ci please smoke test

@xedin xedin merged commit 28fa3ab into swiftlang:master May 22, 2020
@LucianoPAlmeida LucianoPAlmeida deleted the SR-12827-keypath-hole branch May 22, 2020 00:45
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