Making the `ToString` paramters optional (IFormattable)
- removed the
ToString(IFormatProvider?)overload from theIQuantityinterface - removed the
ToStringoverloads from the generated quantities (andHowMuch) - replaced the usages of
ToString(IFormatProvider?)in the tests
duplicate of #1552
Hm, I was expecting there to be a very small size difference compared to #1552, but
before: 2.19 MB (2 301 440 bytes) after: 2.18 MB (2 287 616 bytes) #1552 : 2.17 MB (2 286 080 bytes) 😮
I don't know what to make of these numbers..
Anyway, I was also a little surprised by the number of breaking changes that this caused in the tests (non-generated).. I initially found the ToString extensions a little awkward, but I now I think they might be the better option..
This PR is stale because it has been open 60 days with no activity. Remove stale label or comment or this will be closed in 7 days.
This PR was automatically closed due to inactivity.