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

Return openstruct from repair so we have more metadata tied to the original email #10

Closed
wants to merge 2 commits into from

Conversation

QuotableWater7
Copy link
Contributor

@QuotableWater7 QuotableWater7 commented Jun 28, 2016

This makes it a little easier to work with results in the outside world.

We can use `reduce` here to easily get the result of applying the transformations
without overwriting the original `email` variable.
@QuotableWater7
Copy link
Contributor Author

on top of #9 (or if it is simple enough, I can close out #9 and we just do this pull?)

@coveralls
Copy link

coveralls commented Jun 28, 2016

Coverage Status

Coverage decreased (-0.08%) to 98.684% when pulling 539aa7b on jt-output into a15c873 on master.

…iginal email

This makes it a little easier to work with results in the outside world.
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.08%) to 98.684% when pulling 781df69 on jt-output into a15c873 on master.

@QuotableWater7 QuotableWater7 deleted the jt-output branch June 28, 2016 18:10
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