-
Notifications
You must be signed in to change notification settings - Fork 212
Conversation
Amazing work @sharur7 ! 🎉 Before I approve changes to the PR, I will request you the following changes mentioned below.
Rest LGTM 👍
|
@Neilblaze Can you please help me my image is not showing up in the README.md file. |
Change it to this -
Sharvari, I'll recommend you a small piece of advice. Don't insert any whitespace between any image name. It is considered as a bad practice while working with UTF in markdown. If the change that I just suggested don't works, just rename the jpg image to twitter.png or image.png & replace the name in the |
@Neilblaze Thank You! It Worked... Is there any change that I should do more? |
@@ -0,0 +1,32 @@ | |||
|
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.
Delete the two empty lines at the top
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.
Yep Done!
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.
Should I squash all commits?
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.
Yep, that sounds better! 😃
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.
LGTM!
Awesome work @sharur7 ! 🎉
Code quality looking good 👍 |
Twitter Automation Code
Pre-requisites:
Must have a Twitter account and Twitter developer account.
Need to add your access keys to use twitter APIs
Working:
Just run it and it will like and retweet tweets of the value which you have assigned to the "search".
This was the issue response for #94