-
-
Notifications
You must be signed in to change notification settings - Fork 1.4k
Merge 3.10.x up into 4.3.x #6809
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
Conversation
* 3.9.x: PHPStan 2.1.1 (doctrine#6690) PHPUnit 9.6.22 (doctrine#6691) Bump doctrine/.github from 6.0.0 to 7.1.0 (doctrine#6649) Bump doctrine/.github from 5.3.0 to 6.0.0 (doctrine#6642)
Merge 3.9.x up into 3.10.x
Merge 3.9.x up into 3.10.x
* 3.9.x: bugfix: deallocate mysqli prepared statement (doctrine#6681)
| Q | A |------------- | ----------- | Type | improvement | Fixed issues | N/A #### Summary We'd like to flag the the doctrine/cache package as abandoned which means it needs to disappear from the list of mandatory dependencies of DBAL. In DBAL 3.9, the doctrine/cache classes are used for deprecated code paths only. Note: This change could break downstream projects if they rely on DBAL installing the Cache package for them.
Merge 3.9.x up into 3.10.x
Merge 3.9.x up into 3.10.x
- "pdo_sqlite" | ||
include: | ||
- os: "ubuntu-20.04" | ||
php-version: "8.1" | ||
dependencies: "lowest" | ||
extension: "pdo_sqlite" | ||
- os: "ubuntu-22.04" | ||
php-version: "8.1" | ||
php-version: "8.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.
Both conflicts in this file emanate from #6740
I'm not sure if this is the right conflict resolution or if I should stick with 8.1 here.
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 think 8.4 is the right choice as #6740 made this change. IIRC, I resoved it the same when I merged it to 4.2.x.
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 assume this was supposed to be ready for review based on the PR's comments
I think Grégoire will push it directely, since there's already a merge commit. Hence the draft status. |
No description provided.