fix: add timeout options when exec arp

[howel52]

#12

[natterstefan]

Thank you for your PR and help @howel52. I have just one thing I'd suggest to change.

@DylanPiercey Do you agree with the value for the timeout? Edit: and we might need to think about a way developers can define the timeout and maxBuffer themselves. But that's most likely out of scope of this PR.

[howel52]

Thank you for your PR and help @howel52. I have just one thing I'd suggest to change.

@DylanPiercey Do you agree with the value for the timeout? Edit: and we might need to think about a way developers can define the timeout and maxBuffer themselves. But that's most likely out of scope of this PR.

yeah, I think developers define the options by themselves is right.

And now, In this PR, I have modified the code for suggestions on code review. @natterstefan

[DylanPiercey]

@howel52 thanks for this PR, it looks great 🙂.

I agree with @natterstefan that this should be configurable, but it isn’t absolutely required that we do that in this PR.

This change will of course require a major release since the current default timeout is effectively Infinity. We can then do a minor release to expose both the max buffer and timeout.

[howel52]

I want to know when this PR can be merged. And when will the new npm package be released ?

[natterstefan]

Hi @howel52.

I have just merged it, but it's up to @DylanPiercey to release a new version of the package.

@DylanPiercey we could then probably also include (merge+release) this PR: #9.

[howel52]

Hi @howel52.

I have just merged it, but it's up to @DylanPiercey to release a new version of the package.

@DylanPiercey we could then probably also include (merge+release) this PR: #9.

thx :)