Errors for "NoReturn" type

[Sara]

The NoReturn type is used in these places:

• mab2rec/mab2rec/rec.py

  Line 155 in 6980888

   def _init(self, arms: List[Union[Arm]]) -> NoReturn: 

• mab2rec/mab2rec/utils.py

  Line 399 in 6980888

   item_id_col: str = Constants.item_id, response_col: str = Constants.response) -> NoReturn: 

• mab2rec/mab2rec/pipeline.py

  Line 36 in 6980888

   save_file: Union[str, bool] = None) -> NoReturn: 

"NoReturn" is meant to be used to end further processing, for example when calling exit(). The functions above returns nothing (explicitly) but are not meant to end the program, which results in editor warnings about code after this being unreachable. What you might want to use instead is the None type, i.e.: func() -> None instead of func() -> NoReturn

[skadio]

Hi @SaraEkmanSVT , thank you for pointing it out, you are absolutely correct! We have noticed this too, but never had the chance to get to it..