-
Notifications
You must be signed in to change notification settings - Fork 7
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
[2.x] feat(migr): bump codebase to make compatible with sylius 2 and symfony 7 #43
base: main
Are you sure you want to change the base?
Conversation
maxperei
commented
Mar 14, 2025
Q | A |
---|---|
Bug fix? | no |
New feature? | yes |
BC breaks? | yes |
Fixed issue | n/a |
License | MIT |
…match sylius2 requirements and sf7 standards
df2beae
to
d5d899b
Compare
c7d31cb
to
791f53d
Compare
6bb1b36
to
520cf13
Compare
520cf13
to
ed96219
Compare
ed96219
to
ebc3028
Compare
@@ -19,7 +19,8 @@ jobs: | |||
- 8.2 | |||
- 8.3 | |||
symfony: | |||
- '6.4.*' |
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.
Sylius v2 is still compatible with Symfony 6.4, can't we keep it?
symfony: | ||
- 6.4 |
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.
Sylius v2 is still compatible with Symfony 6.4, can't we keep it?
"sylius/sylius": "^1.12", | ||
"symfony/framework-bundle": "^6.4", | ||
"sylius/sylius": "~2.0.0", | ||
"symfony/framework-bundle": "^6.4 || ^7.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.
This requirement is already in sylius/sylius, so we can remove it.