-
Notifications
You must be signed in to change notification settings - Fork 13
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: drop support for php < 8.1 #15
Conversation
drop support for symfony < 5.4 add "ext-simplexml" as required add strong types
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great work!
composer.json
Outdated
@@ -13,8 +13,9 @@ | |||
} | |||
], | |||
"require": { | |||
"php": ">=5.5", | |||
"symfony/yaml": "^3.2 | ^4.0 | ^5.0 | ^6.0" | |||
"php": ">=8.1", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
~8.1.0 || ~8.2.0
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You mean ~8.1.0 || ~8.2.0 || ~8.3.0 ?
src/Country.php
Outdated
* @param int $minorUnit | ||
*/ | ||
public function __construct($entity, $currency, $alphabeticCode, $numericCode, $minorUnit) | ||
public function __construct(private readonly string $entity, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Add new line
src/Country.php
Outdated
private readonly string $currency, | ||
private readonly string $alphabeticCode, | ||
private readonly int $numericCode, | ||
private readonly int $minorUnit) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Add new line, identation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i changed identation betwen type and variable name. I am not sure if you that is what you mean.
I am not sure where exact you want me to add the new line
Let's get this is in, with the minor changes I suggested. |
drop support for symfony < 5.4 add "ext-simplexml" as required add strong types
The two new lines and indentation are still left. If you correct them I will press the button. |
drop support for symfony < 5.4 add "ext-simplexml" as required add strong types
I am not sure how exactly you want it. |
drop support for symfony < 5.4
add "ext-simplexml" as required
add strong types
As i was working in #14
i made some more changes for a new major version. There are BC breaks, and i am not sure about the changes in the interface.