Navigation Menu

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

Test cleanup #53

Closed
wants to merge 0 commits into from
Closed

Test cleanup #53

wants to merge 0 commits into from

Conversation

zmughal
Copy link
Member

@zmughal zmughal commented Mar 3, 2015

DO NOT MERGE!

Still a work in progress. I will make smaller PRs off this branch for code review soon.

Associated with issues #8 and #9.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 50.23% when pulling 437682f on test-cleanup into aedd1ea on master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 50.23% when pulling 437682f on test-cleanup into aedd1ea on master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 50.23% when pulling 43a44b3 on test-cleanup into aedd1ea on master.

@mohawk2
Copy link
Member

mohawk2 commented Mar 13, 2015

Per @zmughal request, I am adding to here my request that before this can be merged, the num_ok changes be edited out of the history.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 50.29% when pulling f458fa1 on test-cleanup into d751ac7 on master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 49.01% when pulling 2e25053 on test-cleanup into fc32b6b on master.

@coveralls
Copy link

Coverage Status

Coverage increased (+13.48%) to 62.49% when pulling 0a91f23 on test-cleanup into fc32b6b on master.

@zmughal
Copy link
Member Author

zmughal commented Mar 22, 2015

This has been merged in 2d3fdf2...c08d8c4.

@zmughal zmughal removed the pr:wip label Mar 22, 2015
@zmughal zmughal mentioned this pull request Sep 9, 2016
22 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants