minor #508 Require iconv for mbstring (jaapio)

This PR was merged into the 1.x branch.

Discussion
----------

Require iconv for mbstring

As the mbstring polyfill using the iconv functions a lot, if not in all calls being done, code without iconv using this polyfill will break.

As this repo also provides a polyfill for iconv I don't think this will be a breaking change.

Commits
-------

bf69c92 Require iconv for mbstring

src/Mbstring/composer.json

@@ -16,7 +16,8 @@
         }
     ],
     "require": {
-        "php": ">=7.2"
+        "php": ">=7.2",
+        "ext-iconv": "*"
     },
     "provide": {
         "ext-mbstring": "*"