Update README.md - #8
Conversation
galetahub
left a comment
galetahub
left a comment
There was a problem hiding this comment.
Thanks for the fixes, @contributor860!
Could you also apply the same changes to the README.md.erb?
@galetahub I have made the corrections in the 'README.md.erb' also |
|
@galetahub, could you please review my changes in both README.md and README.md.erb |
|
I know you have a busy schedule, but I just wanted to give a reminder |
|
@contributor860, thanks for the reminder! |
|
Yes, I have pushed the changes, and it is also showing in the pull requests section. |
|
Thanks, @contributor860 - now I see what happened. |
|
@galetahub, I have made the changes in the README. md. erb and committed them in this PR as well |
|
Also, should I delete the other PR |
|
@galetahub Will there be changes made to this README file, like the overall structure? |
|
@galetahub Could you please review the changes and merge them? Actually, I am currently creating PRs for README fixes to gain momentum in contributing to this project, and I am using SerpApi to build a project, and during the building of that project if I found any errors or get an idea for a feature then I will update with that |
|
@contributor860 Thank you for pointing it out. |
|
I have regenerated README.md |
galetahub
left a comment
galetahub
left a comment
There was a problem hiding this comment.
Thank you, @contributor860! 👍
Only one comment from my side.
Co-authored-by: Igor Galeta <galeta.igor@gmail.com>
Co-authored-by: Igor Galeta <galeta.igor@gmail.com>
|
I have made the required changes |
galetahub
left a comment
galetahub
left a comment
There was a problem hiding this comment.
Thank you @contributor860, LGTM 👍
|
Thanks for merging my PR, and sorry if I bothered you during the conversations. As I am building a project using SerpApi, if I encounter any issues or discover a new feature idea, I will create an issue for that |
2 participants
Corrected grammatical and spelling errors