Thanks @mkb218 ! This code is clear and stylistically perfect.

My concern with this approach is that any error message coming from remove_all is hidden. To make this method orthogonal with the other File operations, I'd recommend:

• use the non-error_code version of remove_all, so that encountered errors will be caught and reported by the runtime
• set the return value using the bool return value of remove_all.

I think this addition is fine without a test; we currently don't test filesystem operations and I think unless we run into anything significantly annoying, it's more trouble than it's worth. So I've marked this as OK on the test requirement.

Please also add documentation in HelpSource/Classes/File.schelp.

If you have any questions, let me know!

I think I've implemented your recommendations, but now this method's return value is inconsistent with File.delete

@mkb218 That's fine, thanks for the changes! Could you also update the documentation with information about the error throwing conditions?

Overall looks good, but I think you accidentally committed some changes to hidapi as well, can you please revert that and then I'll approve this?

Oops! Hang on while I try to get git to forget about that.

OK, I think this is ready!

No problem! I'll just squash it into a single commit?

yes, i think that would be appropriate here.

@snappizz should be all set now

thank you!