Skip to content
This repository has been archived by the owner on May 19, 2018. It is now read-only.

Add yield result #736

Closed
wants to merge 18 commits into from
Closed

Add yield result #736

wants to merge 18 commits into from

Conversation

zoecarver
Copy link
Contributor

Q A
Bug fix? no
Breaking change? no
New feature? no
Deprecations? no
Spec compliancy? N/A
Tests added/pass? yes
Fixed tickets #100
License MIT

I added test cases for yield result and yield result with parenthesis.

@nicolo-ribaudo
Copy link
Member

This PR shouldn't include the commits merged in #731

@jridgewell
Copy link
Member

Yup, looks like this needs a rebase.

@zoecarver
Copy link
Contributor Author

Sorry about that, I can fix it later (I am at school the rest of today sorry).

@zoecarver
Copy link
Contributor Author

I rebased the repo, I am going to close this PR and create a new one. Thanks!

@zoecarver zoecarver closed this Sep 27, 2017
@existentialism
Copy link
Member

@pudility sounds good, but next time you can always force push to this branch too!

@zoecarver
Copy link
Contributor Author

@existentialism Thanks for the tip! I will try to use that next time. I am pretty new to git and still learning about all of it's features.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants