feat(): Add recursive option for rmdir

[27pchrisl]

Filesystem.rmdir has different behaviour on iOS to other platforms (deletes recursively on iOS, does not on other platforms). This addresses the inconsistency by making all platforms delete recursively.

[jcesarmobile]

Thanks for the PR, it looks good and I've tested and also works fine.

I agree that it should work the same way in all platforms, but we are deciding what the default behaviour should be, delete recursively, fail if not empty or add a new option on the rmdir command to set it to recursive or not. Will let you know once we have a final decision.

[27pchrisl]

Hi, I think that it should be possible to command the native code to do the rmdir on your behalf (either with or without an option), as the performance better than the JS code doing it over the bridge. That was my motivation for making that behaviour common across platforms for our app.

[jcesarmobile]

We've decided that it should have a "recursive" option. (with that name)

So can you update this platforms to only have the new behaviour if that option is passed?
and also modify iOS to fail if the folder is not empty and no recursive option was passed?

[27pchrisl]

Behaviour updated, added 'recursive' option that defaults to false

[27pchrisl]

Tests seem to be failing due to an internal CircleCI error

[jcesarmobile]

I've merged your other PR and not this one have conflicts, can you fix them?

Hopefully tests will pass this time, they sometimes hang for no reason.

[27pchrisl]

Conflicts fixed

[jcesarmobile]

Pushed 2 minor changes and 2 fixes to your branch, will merge once tests finish.

Thanks again for your PR!