Skip to content
This repository has been archived by the owner on Apr 18, 2022. It is now read-only.

Manually add _called attribute on Timeout object #16

Merged
merged 2 commits into from
Aug 1, 2019
Merged

Manually add _called attribute on Timeout object #16

merged 2 commits into from
Aug 1, 2019

Conversation

MarcAbonce
Copy link
Contributor

Newer versions of NodeJS removed the _called attribute from the Timeout object (nodejs/node#20555), so it's always undefined.

In the current implementation of Timeout I didn't find anything equivalent to _called, so I just made this workaround instead.

Newer versions of nodejs removed the _called
attribute from the Timeout object.

See: nodejs/node#20555
@unixfox
Copy link
Owner

unixfox commented Jul 16, 2019

Hello, thank you for your pull request. I knew that _called isn't available on newer versions of nodejs that's why my Dockerfile is still targeting the 10th version.
I'll review your PR as soon as possible (I'm currently on vacation) to check if it's still working as intended (and on older versions of NodeJS) .

@MarcAbonce
Copy link
Contributor Author

Don't worry, I'm also very busy now so I have no hurry.
It was my mistake for not of using the Dockerfile anyway.

@unixfox unixfox merged commit 1cd8146 into unixfox:master Aug 1, 2019
@MarcAbonce MarcAbonce deleted the hotfix/called_attr branch August 2, 2019 04:18
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants