Typehints

[tvdijen]

This is a good start into PHP 7.1 and PSR-12 compliancy.. Some Psalm-issues remain

[codecov-io]

Codecov Report

❗ No coverage uploaded for pull request base (master@78ca6f0). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #208   +/-   ##
=========================================
  Coverage          ?   69.28%           
  Complexity        ?      402           
=========================================
  Files             ?        5           
  Lines             ?     1133           
  Branches          ?        0           
=========================================
  Hits              ?      785           
  Misses            ?      348           
  Partials          ?        0           

[tvdijen]

This is only half the work.. I took out the low hanging fruit, but the remaining Psalm-issues should be addressed

[robrichards]

@tvdijen I am just going to hang on to this until I am considering a 4.x branch. There are too many BC breaking changes. I will go through the type hint only change to see if that might be able to be applied. May also grab some of your changes that fix codecov as well.

[tvdijen]

Yes, the downside of changes like this it that it touches literally every method/file, but the benefits for fully typehinting a library like this are huge..
I can split it in multiple smaller PR's if that helps you to review things?