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

Complete existing tests #1

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

MarcAtrapalo
Copy link

Hello Thomas,

Thank you for this great koans. I'm using it to train my fellow colleagues in generators and redux-saga, and this is just what I needed, and a great way to learn it.

I have taken the liberty to complete the two tests that were unfinished: The last in saga-basics and the one in yield-star. Please tell me if they suit your code, and feel free to incorporate or improve them.

Thanks,
Marc.

@ttmarek
Copy link
Owner

ttmarek commented Jan 29, 2018

@MarcAtrapalo I had no idea anyone would find use for this. I'm really really really happy you are getting value from it though. I will dedicate some time to improving it and sharing it with others. I'll have a read through the pull request tonight.

🙂

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