marl / pysox

Python wrapper around sox.
BSD 3-Clause "New" or "Revised" License
513 stars 79 forks source link

change sox() returned type and success value #166

Open mikeziminio opened 5 months ago

mikeziminio commented 5 months ago

changed docstring for sox() returned type and success value

lostanlen commented 5 months ago

Thanks! please add a unit test

mikeziminio commented 4 months ago

Thanks! please add a unit test

Vincent, thanks for the quick response! I only changed the docstring, so I have no idea what tests can be added. Please advise me.

lostanlen commented 4 months ago

test that indeed sox returns an integer and that this integer is zero

mikeziminio commented 4 months ago

test that indeed sox returns an integer and that this integer is zero

ok, i will do it next week