Align with the commerceguys/intl changes

Committed!

Automatically closed - issue fixed for 2 weeks with no activity.

This is a BC break, no, shouldn't this go into a new major release?

Also, without explicit dependency/conflict declarations between this, commerce and commerceguys/intl, it is possible to update to incompatible combinations of these packages. A project I updated somehow only updated physical and not commerce/intl, so I had the latest version of physical and commerceguys/intl 1.x, resulting in this very confusing error:

Fatal error: Declaration of CommerceGuys\Intl\Formatter\NumberFormatter::format($number, array $options = []) must be compatible with Drupal\physical\NumberFormatterInterface::format(string $number, array $options = []): string in vendor/commerceguys/intl/src/Formatter/NumberFormatter.php on line 68

(confusing because that physical interface is only implemented in the commerce subclass that isn't mentioned here)

I have the same issue as described in #6. I would be nice to have a fix for this soon...

There won't be a fix. Either downgrade physical back to 1.2, which is OK as this is the only change in that release or upgrade commerce. The conflict should have been added to the 1.3 release, but nothing can be done about that now.

Apologize for this, I thought about adding the conflict in Commerce, but forgot to do the same here.

For anyone still looking temp solution is

composer require drupal/physical:1.2