Support AbortController

[jopemachine]

Fixes #449.

I'd appreciate it if you could let me know if I need to write more tests or documents.

Thanks for all your efforts for this awesome library :)

[ehmicky]

Thanks a lot for this contribution @jopemachine!

We might want to name this option signal instead of abortSignal to match the name currently used by child_process.spawn().

More fundamentally, I am wondering whether the core signal option should just be used instead? I have written some potential ideas in the issue, please let me know what you think.

[jopemachine]

We might want to name this option signal instead of abortSignal to match the name currently used by child_process.spawn().

Thanks for pointing it out, I just updated the option.

[jopemachine]

Should I delete this document too? or should I mark the cancel method as deprecated?

[ehmicky]

I believe we can directly delete that section of the documentation, good catch! 👍

Thanks again for this PR @jopemachine! There are a couple of things I have noted.
The tests look great! (But the CI tests are currently failing)

[jopemachine]

I believe we can directly delete that section of the documentation, good catch! 👍

Thanks again for this PR @jopemachine! There are a couple of things I have noted. The tests look great! (But the CI tests are currently failing)

I appreciate your detailed feedback :)
I tried to reflect on all the feedback as much as I can.
If I'm missing something or got something wrong, I appreciate your letting me know.

[ehmicky]

Thanks a lot for those edits, they look great! And the tests are now passing on Node 16 🎉
I have a few more comments, but we're getting there. Thanks for your help @jopemachine.

[ehmicky]

This looks good to me, thanks a lot @jopemachine!

Now waiting for another review from @sindresorhus.