Skip to content
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

project: add the Observed property wrapper #9

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

twittemb
Copy link

Hi @guillermomuntaner

Here is an implementation of the Observer pattern with a property wrapper.
I think I've done all what was required in the CONTRIBUTING.md file.

Thanks.

Thibault.

@twittemb twittemb force-pushed the feature/Observed branch 2 times, most recently from 063685f to 94edcde Compare July 15, 2019 02:32
@bobgodwinx
Copy link

@twittemb That's Rx 😂 I love it. Thanks for the contribution.

@twittemb
Copy link
Author

Hi @guillermomuntaner, I know it is never easy to respond to PRs quickly (as I am my self a maintainer of a repo), but I'd like to know when do you think you will be able to review this PR ?

I may reference it in an article I'm writing right now and it could be nice to have a feedback on it.

thanks a lot.

@twittemb
Copy link
Author

twittemb commented Aug 4, 2019

hi @guillermomuntaner,

Here is the article I wrote about Swift 5.1 / property wrappers being some kind of Aspect Oriented Programming language: https://medium.com/genetec-tech/swift-an-aspect-oriented-programming-language-6058b07c1205

Do you have an ETA for the review ?

thanks a lot.

@twittemb
Copy link
Author

twittemb commented Oct 3, 2019

Hi @guillermomuntaner

Do you still accept PR for this project ?

thanks.

@twittemb
Copy link
Author

twittemb commented Feb 4, 2020

Hi @guillermomuntaner

Do you still accept PRs ?

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.

None yet

2 participants