-
-
Notifications
You must be signed in to change notification settings - Fork 3.3k
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
fix(crawler): using newspaper and fixed recursive by merging content #955
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Risk Level 2 - /home/runner/work/quivr/quivr/backend/core/crawl/crawler.py
import logging
try:
# some code
except Exception as e:
logging.error(e)
raise
from urllib.parse import urlparse
def checkGithub(self):
parsed_url = urlparse(self.url)
return parsed_url.netloc == 'github.com'
🖨️🔀🔗 Powered by Code Review GPT |
Suspect IssuesThis pull request was deployed and Sentry observed the following issues:
Did you find this useful? React with a 👍 or 👎 |
…955) * fix(crawler): using newspaper and fixed recursive by merging content * feat(code-review): added feedback from code review
Description
Please include a summary of the changes and the related issue. Please also include relevant motivation and context.
Checklist before requesting a review
Please delete options that are not relevant.
Screenshots (if appropriate):