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

added more functionality #2

Merged
merged 16 commits into from
Jun 24, 2014
Merged

Conversation

desaivaibhavi
Copy link

pushed up final code for Malaria with all the suggested changes and modifications

@rosariorobinson
Copy link

Everything looks good Rani. Before I merge, would like to review once more tomorrow at our meeting, Wed 18 Jun 1400 UTC.

@desaivaibhavi
Copy link
Author

Sure.

Also, I have made some more changes to the code after this pull request.
Will send you another pull request ( before we meet ), containing the
latest version.

Vaibhavi

On Tue, Jun 17, 2014 at 10:18 PM, Rose Robinson notifications@github.com
wrote:

Everything looks good Rani. Before I merge, would like to review once more
tomorrow at our meeting, Wed 18 Jun 1400 UTC.


Reply to this email directly or view it on GitHub
#2 (comment).

Vaibhavi H Desai | Google Student Ambassador, DA-IICT | rani.vhd@gmail.com |
+91-7698876900

@rosariorobinson
Copy link

Vaibhavi, I did not see another PR. Should i go ahead and merge #2 and #1?

@rosariorobinson
Copy link

Vaibhavi, I'll go ahead and merge this PR. Good that you're using framework/modeling throughout.

rosariorobinson added a commit that referenced this pull request Jun 24, 2014
added more functionality -- merging this PR.
@rosariorobinson rosariorobinson merged commit f68b13c into systers:master Jun 24, 2014
Nerdylicious added a commit that referenced this pull request Jan 11, 2016
Merge request #2 after updating .travis.yml file
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants