Chore: Ignore process.exit eslint warnings

[gu-stav]

What does it do?

Every PR has now a list of attached eslint warnings, which clutters them quite a bit. This PR adds eslint-ignore comments to each call to reduce that cluttering.

An alternative would be to disable the warnings globally, but I think it is a good idea to leave it on.

Why is it needed?

Less warnings. They were introduced by #14017

[codecov]

Codecov Report

Merging #14119 (1194616) into main (e4e0981) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main   #14119   +/-   ##
=======================================
  Coverage   55.58%   55.58%           
=======================================
  Files        1275     1275           
  Lines       31829    31829           
  Branches     5733     5733           
=======================================
  Hits        17692    17692           
  Misses      12323    12323           
  Partials     1814     1814           

Flag	Coverage Δ
front	58.13% <ø> (ø)
unit	49.38% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

[alexandrebodin]

I would just disable the rule instead as it will remove the warnings from eslint too. Can you just set it to off in the eslintrc file please?

[gu-stav]

@alexandrebodin Sure, I can do that too 👍🏼

I didn't do it, because I thought it is still valuable so that devs do add new calls only if it is really needed.

[alexandrebodin]

@alexandrebodin Sure, I can do that too 👍🏼

I didn't do it, because I thought it is still valuable so that devs do add new calls only if it is really needed.

Yes that makes total sense.

I think we will enable it again later and go over all the calls again I'm not confident saying the current ones are all good :)