-
-
Notifications
You must be signed in to change notification settings - Fork 398
Allow symfony 6.4 #1551
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
Allow symfony 6.4 #1551
Conversation
|
@sreichel we need to drop PHP 7.4 support. Min PHP Version is 8.1. |
|
for reference: netz98/n98-magerun2#1608 |
|
Thats magerun2 😀 |
I know. Same topic. :-) |
|
Thanks! Time for a new release? :) |
|
@sprankhub yes, its time for. I have some write permissions, but creating a new release belongs to @cmuench hands. I dont have further plans ... #1458 is for next release. PS: sponsoring welcome. Or, if you are looking for a dev .... |
|
@sprankhub sadly I have less time for n98-magerun1 (at work I have not any M1 project anymore). So it is not realistic to find time for the QA and resolving of the merge conflicts in the next weeks. |
|
I am just annoyed by the security notifications for https://github.com/customgento/magerun-tax-editor, which I cannot resolve due to this :D @cmuench, can't you just create a new release? I am not sure which separate QA you usually do before a release, but at least resolving merge conflicts should not be needed, I guess? |
Merge conflicts are resolved. lgtm btw ... ialso dont have any M1 project. I help to maintain just for fun. |
|
@cmuench pls make a new major release. Possible fixes can be added later. |
|
Yep. Please ignore #1458. It can be added in 3,1 or 4.x. |
|
@sreichel @sprankhub There was a small issue in the deployment process but it's now released as 3.0.1 |
|
Thanks! |
OpenMage requires php81 with next release.
We also use some symfony compents, that are now locked at v6.4.
@cmuench i know you wanted to keep php74-support. That was last year. Do we still need it?
@sprankhub any opinions one that?