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

add NewDocumentWithTimeout #240

Closed
wants to merge 1 commit into from
Closed

add NewDocumentWithTimeout #240

wants to merge 1 commit into from

Conversation

cokeboL
Copy link

@cokeboL cokeboL commented Mar 7, 2018

I see this issues: #238
then create this pull request

@mna
Copy link
Member

mna commented Mar 10, 2018

Hello,

Thans a lot for caring about this project, but as mentioned in the #238 issue, this is code that belongs outside goquery. The #173 issue indicates the NewDocument* functions that should be deprecated. Those are not recommended ways to use goquery in any "production" code (e.g. outside quick prototypes and experiments).

For this reason, I will have to decline this PR, sorry :(
Martin

@mna mna closed this Mar 10, 2018
@cokeboL
Copy link
Author

cokeboL commented Mar 12, 2018

ok, got it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants