[MRG] Improve file handling, add overwrite parameter

[sappelhoff]

closes #63

• better file handling: Delete intermediate files when writing fails
• better input checks: raise error when ch_names is not list of str (or list of int which we can easily turn into str)
• add overwrite parameter and set new default (False) --> previously pybv always overwrote files, which was a bit dangerous IMHO

Also adds tests for all of those

[codecov]

Codecov Report

Merging #78 (efbb1fc) into main (e68158d) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##              main       #78   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            3         3           
  Lines          346       386   +40     
=========================================
+ Hits           346       386   +40     

Impacted Files	Coverage Δ
pybv/io.py	100.00% <100.00%> (ø)
pybv/tests/test_bv_writer.py	100.00% <100.00%> (ø)

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e68158d...efbb1fc. Read the comment docs.

[cbrnr]

In addition to my specific comments I was wondering if os.path could also be replace by pathlib not just in the tests but also in the rest of the code?

[sappelhoff]

In addition to my specific comments I was wondering if os.path could also be replace by pathlib not just in the tests but also in the rest of the code?

Yes, I thought about that myself. Would probably be prettier/clearer. Feel free to open an issue that then somebody can do at some point.

[cbrnr]

Thanks @sappelhoff!