-
Notifications
You must be signed in to change notification settings - Fork 0
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
Web crawling 7th PR #20
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
approve
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is what I've done in #18. You should merge the main to your branch first and solve the merge conflict. In this case, you remove what I've done in #18.
Please check the #18 again and adjust your code.
Also read the PR report again.
Moreover, the order of your get function return is:
link, articles
Rather than
article, link
Thanks for the correction, I've just fixed it. Please take a look.
…On Mon, Aug 14, 2023 at 7:24 PM YLee2121 ***@***.***> wrote:
***@***.**** requested changes on this pull request.
This is what I've done in #18 <#18>.
You should merge the main to your branch first and solve the merge
conflict. In this case, you remove what I've done in #18
<#18>.
Please check the #18 <#18> again
and adjust your code.
Moreover, the order of your get function return is:
link, articles
Rather than
article, link
—
Reply to this email directly, view it on GitHub
<#20 (review)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/A4IP7NBXBEKQMYNTXR5OMZTXVKXTXANCNFSM6AAAAAA3PFD2PQ>
.
You are receiving this because you were assigned.Message ID:
***@***.***>
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
approve
👉 Summary
Main in Crawl.py can return links to the articles now.