Skip to content

More map by keyPath#1121

Merged
mxcl merged 10 commits intomxcl:masterfrom
RomanPodymov:master
Jan 15, 2020
Merged

More map by keyPath#1121
mxcl merged 10 commits intomxcl:masterfrom
RomanPodymov:master

Conversation

@RomanPodymov
Copy link
Copy Markdown
Collaborator

Hello.
Thank you for PromiseKit.
In this pull request I added more mapping functions that accepts KeyPath instead of a closure (started in #1118).

@mxcl
Copy link
Copy Markdown
Owner

mxcl commented Jan 14, 2020

I have confirmed that Swift 5.2 adds the SE proposal that KeyPaths are compiler converted to functions with literal syntax, thus this addition is welcome, but should be guarded with #if swift(<5.2) to prevent collision.

@RomanPodymov
Copy link
Copy Markdown
Collaborator Author

Hello @mxcl
I see that the Travis CI build could not complete due to an error. How can I restart it?

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