Skip to content
This repository has been archived by the owner on Mar 3, 2020. It is now read-only.

Modernizing the change dictionary to use the new string enum NSKeyValueChangeKey #124

Merged
merged 1 commit into from
Mar 11, 2017
Merged

Conversation

mohamede1945
Copy link
Contributor

This way it would help Swift bridging to use an enum for the change dictionary.

@nlutsenko
Copy link
Contributor

This is a great modernization. Thank you!
Will merge as soon as the CI passes.

@nlutsenko nlutsenko self-requested a review March 2, 2017 18:54
@mohamede1945
Copy link
Contributor Author

Great! Thank you so much! Is it possible to publish a new Cocoapods version so that I can use it in my project?

@nlutsenko
Copy link
Contributor

Sure, I don't see why not...
Also, be aware, this might actually fail CI, since it's still configured to use Xcode 7.3, but I am going to fix it in a little bit.

What this means is that you might need to rebase this pul request on top of the master.
Will update here as soon as I have something up.

@mohamede1945
Copy link
Contributor Author

Sounds good to me! Thank you so much for the quick responses!

drkibitz added a commit to drkibitz/KVOController that referenced this pull request Mar 7, 2017
@nlutsenko nlutsenko added this to the 1.2.1 milestone Mar 11, 2017
@nlutsenko nlutsenko merged commit 09be0e5 into facebookarchive:master Mar 11, 2017
rjstelling pushed a commit to MonikerSonic/KVOController that referenced this pull request Sep 14, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants