-
Notifications
You must be signed in to change notification settings - Fork 34
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
Modularize code for #36. #50
Conversation
Updating master
Merging master
Listener.py
Outdated
import json | ||
import sys | ||
from os.path import expanduser | ||
from config.ConfigReader import ConfigurationReader |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is not needed I think
Listener.py
Outdated
|
||
# Twitter API credentials | ||
cfg = {} | ||
home = expanduser("~") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These lines are not needed in this file
main.py
Outdated
import json | ||
import sys | ||
from os.path import expanduser | ||
from config.ConfigReader import ConfigurationReader |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nt needed
main.py
Outdated
|
||
# Twitter API credentials | ||
home = expanduser("~") | ||
Configuration = ConfigurationReader() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nt needed.
main.py
Outdated
|
||
# Twitter API credentials | ||
home = expanduser("~") | ||
Configuration = ConfigurationReader() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not needed
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please check comments. I think the code will be alright. I didn't test it. But it should work perfectly though.
Implemented changes as suggested in comments. |
@thakkardharmik What changes should be made in the setup.py? |
@nithinswarrier I think some changes would be required @ https://github.com/CruiseDevice/twweet-cli/blob/master/setup.py#L97 and |
@thakkardharmik merged 👍 ✨ 🎉 Sorry for merging this so late. I was busy with other commitments. |
Please review the code. I need some help to modify setup.py in case the code looks fine. Also if someone can help me test the code.