Skip to content
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

Quotes Extractor #796

Merged
merged 45 commits into from Jul 19, 2021
Merged

Quotes Extractor #796

merged 45 commits into from Jul 19, 2021

Conversation

Gaurav1401
Copy link
Contributor

@Gaurav1401 Gaurav1401 commented Jul 18, 2021

Related Issue

  • Extracting Quotes from Website

  • LGM-SOC'21 Participant

  • DevIncept Participant

  • Contributor

Closes: #798

Describe the changes you've made

Wrote complete code to extract codes from the website along with proper description in the markdowns and proper comments

Type of change

What sort of change have you made:

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Code style update (formatting, local variables)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

Checklist:

  • My code follows the guidelines of this project.
  • I have performed a self-review of my own code.
  • I have commented my code, particularly whereever it was hard to understand.
  • I have made corresponding changes to the documentation.
  • My changes generate no new warnings.
  • I have added tests that prove my fix is effective or that my feature works.
  • Any dependent changes have been merged and published in downstream modules.

@welcome
Copy link

welcome bot commented Jul 18, 2021

Hello there! 👋 Welcome to the project! 💖
Thank you and congrats 🎉 for opening your first pull request. Please adhere to our Code of Conduct. 🙌🏻 We will get back to you as soon as we can. 😄

Feel free to get in touch with me through social media handles. Hope to see you there!😄

@Debashish-hub
Copy link
Collaborator

Debashish-hub commented Jul 18, 2021

Please follow proper PR template and before that create an issue and get it assigned Gaurav

@Gaurav1401
Copy link
Contributor Author

Gaurav1401 commented Jul 18, 2021

@Debashish-hub I have created the issue

@Gaurav1401
Copy link
Contributor Author

@Debashish-hub Now, please check my pull request.

@prathimacode-hub prathimacode-hub changed the title Web Scraping Tutorial added Quotes Extractor Jul 18, 2021
Copy link
Owner

@prathimacode-hub prathimacode-hub left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this not a project first of all? It accepts only projects. And project iles should be only in .py format. There is also a project structure given. Follow it. Your project titles, your file names everything should be changed. You said quotes extractor using web scraping. Add that project. @Gaurav1401

@Gaurav1401
Copy link
Contributor Author

@prathimacode-hub mam, thank you very much.
I actually this is first time I am contributing to open source, that's why don't have idea about the guidelines and syntax.
I seriously learnt a lot with you.

I have added some more information in the README.md file.

I hope now it looks good, please let me know anything else is required to be added or not.

Copy link
Owner

@prathimacode-hub prathimacode-hub left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Changes to be done:
Add workflow of the project and compilation steps in README.md file
And also the image links you added up in README.md are of "user-images" content instead it should be the file pathn of the images present in this repo.
The project file name should be as "quotes_extractor.py"

Kindly do these changes. @Gaurav1401

@prathimacode-hub
Copy link
Owner

@prathimacode-hub mam, thank you very much.
I actually this is first time I am contributing to open source, that's why don't have idea about the guidelines and syntax.
I seriously learnt a lot with you.

I have added some more information in the README.md file.

I hope now it looks good, please let me know anything else is required to be added or not.

Sure, glad to see you select this project. I had advised the necessary changes and don't worry about being a beginner, that's how you learn it. So you are going good. @Gaurav1401

@Gaurav1401
Copy link
Contributor Author

@prathimacode-hub So mam, now will this project be added on the main repository?
Also please accept my LinkedIn request

@prathimacode-hub
Copy link
Owner

@prathimacode-hub So mam, now will this project be added on the main repository?
Also please accept my LinkedIn request

Have you done the changes? I don't find any changes here. And Yeah, I will accept the request. It will be merged into main repo once things are in right place. You can do this. @Gaurav1401

@Gaurav1401
Copy link
Contributor Author

@prathimacode-hub yes mam, I have changed the location of those images and also added more info. in my readme file

@prathimacode-hub
Copy link
Owner

@prathimacode-hub yes mam, I have changed the location of those images and also added more info. in my readme file

You hadn't done these changes yet. @Gaurav1401
Add workflow of the project and compilation steps in README.md file
The project file name should be as "quotes_extractor.py"

@Gaurav1401
Copy link
Contributor Author

@prathimacode-hub Sorry for the delay mam,
Actually I forgot to commit and push all the changes, but now I have made it, I hope now it's clear :).

Copy link
Owner

@prathimacode-hub prathimacode-hub left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

PR approved. Yayy, it's perfect. Congrats on your first PR. Keep it up. This was a great start. And welcome to the repo officially. 👏✨🎉@Gaurav1401

@prathimacode-hub prathimacode-hub added the 🟨 Level2 This issue will be considered as level 2 issue for LGM-SOC 21 & DCP21. label Jul 19, 2021
@prathimacode-hub prathimacode-hub merged commit 41b771d into prathimacode-hub:main Jul 19, 2021
@welcome
Copy link

welcome bot commented Jul 19, 2021

Congrats on merging your first Pull Request! 🎉 All the best for your amazing open source journey ahead. 🚀⚡️

@prathimacode-hub prathimacode-hub removed the To Do Update Pending label Jul 28, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🟨 Level2 This issue will be considered as level 2 issue for LGM-SOC 21 & DCP21. LGMSOC21 This issue will be considered for LGM-SOC 21
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Extracting Quotes from Website
3 participants