You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
This repository was archived by the owner on Feb 2, 2022. It is now read-only.
The parse_page function currently gets called from within an async function, fetch_and_parse. This makes it hard for people who don't understand async programming (me) to step through and debug the parsing code. Specifically I'm having trouble debugging pull request #184.
Expected Behavior
Although it makes sense for the "fetch" part of fetch_and_parse to be async because that happens over the network, parsing doesn't need to be async.
Possible Solution
Take the parsing functionality out of the async functions.
Issue Type
Current Behavior
The
parse_pagefunction currently gets called from within an async function,fetch_and_parse. This makes it hard for people who don't understand async programming (me) to step through and debug the parsing code. Specifically I'm having trouble debugging pull request #184.Expected Behavior
Although it makes sense for the "fetch" part of
fetch_and_parseto be async because that happens over the network, parsing doesn't need to be async.Possible Solution
Take the parsing functionality out of the async functions.
Steps to Reproduce
Context