-
Notifications
You must be signed in to change notification settings - Fork 787
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
Goose fails in extracting articles from The New York Times #252
Comments
Your url string looks malformed, but anyway I fixed a similar issue in #225 and pull-requested but I don't think it was ever approved. My fork has this issue fixed and I've been parsing NYtimes happily ever since. Good luck! Rob |
@robmcdan the PR has not been accepted because it breaks the test case suites |
I'm curious; this issue was close because there is a work-around involving some external manipulation? |
@richardpetithory No, I closed this issue because @robmcdan has an issue open with this exact same problem #225 so I don't think there is a need for two open requests. |
following code:
The text was updated successfully, but these errors were encountered: