You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
This issue will contain a list of suggestions that we could take into account for the next major version of this library. Ideas are not complete in the sense that they represent what are going to do. It is merely a place where we can collect idea.
Move towards PHP 7.1+ with string and return types and less docblock.
In order to have your ideas listed, please open an issue for discussion. If there is agreement that it should be taken into account in the major next version, we will list it here.
The text was updated successfully, but these errors were encountered:
I think some of them can be implemented in a BC manner in this version as well, and remove the unwanted behaviour in the next major. Parser and CalculatorRegistry can surely be done this way.
@sagikazarmark True. I rather go for everything in the next major version. I would be more than happy to give the first shot at this and move everything towards PHP 7.1 code.
Yeah, me too, but I would like to avoid maintaining two versions and we've just released Money 3.0, so people using it would probably not want to migrate again.
So as long as we have to maintain 3.0 I would rather pack the new features in it as long as possible.
This issue will contain a list of suggestions that we could take into account for the next major version of this library. Ideas are not complete in the sense that they represent what are going to do. It is merely a place where we can collect idea.
Ideas.
loader
interface injected, possible implementations are StaticLoader, DirectLoader and CachableLoader, see Once currencies.php is outdated it is no longer ISO currencies. #392.CalculatorRegistry
that can be used by Money and other classes that require calculations, see Add IndirectExchange #384 and Better CurrencyPair #363.In order to have your ideas listed, please open an issue for discussion. If there is agreement that it should be taken into account in the major next version, we will list it here.
The text was updated successfully, but these errors were encountered: