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

Merge branch for issue #110 #122

Merged
merged 1 commit into from
May 6, 2015
Merged

Merge branch for issue #110 #122

merged 1 commit into from
May 6, 2015

Conversation

stleary
Copy link
Owner

@stleary stleary commented May 6, 2015

Replaced tab chars, updated versions

@stleary stleary changed the title Replaced tab chars, updated versions merge-branch-for-issue-110 May 6, 2015
@stleary stleary changed the title merge-branch-for-issue-110 Merge branch for issue #110 May 6, 2015
@stleary
Copy link
Owner Author

stleary commented May 6, 2015

What problem does this code solve?
These are the changes from pull request #110

Has any code changed?
Yes

Verified via unit test?
Yes. I had to add 2 unit tests to make sure each changed line was executed at least once. Will commit the updated unit tests after this commit.

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

1 participant