Fix issues with adding/removing SIGINT listeners

[WesCossick]

Closes #709

This PR fixes the problem where Listr is removing all SIGINT listeners, instead of just the listeners it manages. I added a "should not clear other SIGINT listeners" test and verified it failed prior to implementing the fix. Now the test passes.

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 82.25%. Comparing base (cc812f0) to head (0d30480).

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files

@@           Coverage Diff           @@
##           master     #710   +/-   ##
=======================================
  Coverage   82.25%   82.25%           
=======================================
  Files          37       37           
  Lines         896      896           
  Branches      231      231           
=======================================
  Hits          737      737           
  Misses        119      119           
  Partials       40       40           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[cenk1cenk2]

Hmm really interesting if we you wrap whole thing and put in a variable then it finds it.

Thank you for your work on this. Can you just reword the commit to start with fix: instead of the current form so semantic release publishes it.

[WesCossick]

Sure, I've just updated the commit message to start with fix:.

[cenk1cenk2]

🎉 This PR is included in version 8.1.2 🎉

The release is available on npm package (@latest dist-tag)

Your semantic-release bot 📦🚀

[cenk1cenk2]

🎉 This PR is included in version 2.0.5 🎉

The release is available on npm package (@latest dist-tag)

Your semantic-release bot 📦🚀

[cenk1cenk2]

🎉 This PR is included in version 2.0.5 🎉

The release is available on npm package (@latest dist-tag)

Your semantic-release bot 📦🚀

[WesCossick]

@cenk1cenk2 Thanks for releasing this so quickly! While our company was working on integrating the new release into one of our projects, I noticed there's a lingering process.removeAllListeners('SIGINT') call in the signalHandler() function... Unfortunately, I didn't notice it when I was working on this MR. I suspect it's not needed at all, since you don't need to call process.removeListener() when you're using process.once() to add the listener in the first place.

Do you think it should be removed?