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

Feat: FlexJobs Scrapper added #940

Merged
merged 6 commits into from
May 16, 2024
Merged

Conversation

AKASH722
Copy link
Contributor

@AKASH722 AKASH722 commented May 13, 2024

Description

The changes made in this pull request include the creation of a FlexJobs scrapper, implementation of unit tests for the scrapper, and updates to the documentation.

Resolves: [ #881 ]

Checklist

Before submitting this pull request, kindly verify that the ensuing checkpoints have been reached.

  • Have you adhered to the repository's defined coding convention rules?
  • Have you updated the 'documentation.md' file with the method/function documentation?
  • Have you sent a message along with the result or response?
  • Have you used the try-catch technique?
  • Has the method/class been added to the documentation (md file)?

Screenshots

image

image

image

image

Additional Notes/Comments

I have created the flexjobs scrapper please check that out.
I want Indeed website scrapper next, so should i open another issue, or work under the same as both are job scrappers


I certify that I have carried out the relevant checks and provided the requisite screenshot for validation by submitting this pull request.
I appreciate your contribution.


class FlexJobs:
"""
A class to scrape job listings from FlexJobs website based on search query and location.
Copy link
Member

Choose a reason for hiding this comment

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

The docstring of the class and the methods are not properly formatted. You can check modules like GitHub and IMDB for a reference to how these modules have their docstring formatted.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Okay i will fix that out.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@nikhil25803 changed the docstrings could you check now

- Changed documentation for flexjobs scrapper
- Removed documentation from documentation.md and added it to dev-documentation.md
- Changed documentation for flexjobs scrapper
- Removed documentation from documentation.md and added it to dev-documentation.md
@AKASH722 AKASH722 requested a review from nikhil25803 May 14, 2024 15:12
@nikhil25803 nikhil25803 merged commit f6a9ea0 into Clueless-Community:main May 16, 2024
@nikhil25803
Copy link
Member

Great work @AKASH722 !!
Do not forget to

  • Share your contribution on socials and tag Clueless 🫂
  • Give this repository a star ✨
  • Follow Clueless Community on GitHub ✅

@nikhil25803 nikhil25803 added level3 gssoc GSSoC 2024 and removed under review labels May 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
gssoc GSSoC 2024 level3
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants