-
Notifications
You must be signed in to change notification settings - Fork 21
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
[scraper-documentation] beautify readme #49
base: master
Are you sure you want to change the base?
Conversation
@manasakalaga These changes are great, but not in respect to the correct Issue. I think I wasn't able to explain the issue well. I referred to in Scrapper Service in Deployment here in this README. |
@sagar-sehgal given that the changes are required in that file too, I'd make the changes to README in the root folder and push it as-well. Would that be okay? |
It would be great if you make a separate PR for that. It would be easy to separate and understand. |
Here it is. #61 |
@manasakalaga I will merge this one, can you attend to the other files also. See where it can use some improvements. Like all the sections of the project. Maybe setup headers to be filled with content later etc. |
Fixes issues with
README
in scrapper service as mentioned in #40