Skip to content
This repository has been archived by the owner on Aug 18, 2020. It is now read-only.

cleaning some code #21

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

infinite-Joy
Copy link
Contributor

One of the tests are failing, I think that the current implementation should be fine. Please let me know your views

@infinite-Joy
Copy link
Contributor Author

If everything is good will change the tests

@tasdikrahman
Copy link
Owner

tasdikrahman commented Jul 30, 2016

@infinite-Joy The tests are failing.

@tasdikrahman
Copy link
Owner

@infinite-Joy test_synonym_valid_phrase (__main__.TestModule) ... FAIL

Could you check on this

@infinite-Joy
Copy link
Contributor Author

Yes, Will look more into this and come back to you.

@infinite-Joy
Copy link
Contributor Author

Its passing on my machine, Can you tell me how you are executing. I am
using nose.

Regards
Joydeep Bhattacharjee
phone 8981849869
mailto: [email protected]
linkedinID: in.linkedin.com/pub/joydeep-bhattacharjee/57/a11/934/
blog: joydeepbhatt.com
Github: https://github.com/infinite-Joy
http://in.linkedin.com/pub/joydeep-bhattacharjee/57/a11/934/

On Sat, Jul 30, 2016 at 10:29 PM, Tasdik Rahman [email protected]
wrote:

@infinite-Joy https://github.com/infinite-Joy test_synonym_valid_phrase
(main.TestModule) ... FAIL

Could you check on this


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#21 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AJiR3ao1GqvafCYSlfSOUB90F1Z8eAFQks5qa4MOgaJpZM4JY1w2
.

@tasdikrahman tasdikrahman force-pushed the master branch 2 times, most recently from eed5e59 to d795fef Compare April 6, 2017 06:11
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.

2 participants