Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Modernize pgsql driver for PHP 8.1 #5904

Merged
merged 1 commit into from
Feb 4, 2023
Merged

Modernize pgsql driver for PHP 8.1 #5904

merged 1 commit into from
Feb 4, 2023

Conversation

derrabus
Copy link
Member

@derrabus derrabus commented Feb 3, 2023

Q A
Type improvement
Fixed issues N/A

Summary

Let's throw all the PHP 7 quirks out of our shiny new pgsql driver.

@derrabus derrabus added this to the 4.0.0-beta2 milestone Feb 3, 2023
@derrabus derrabus force-pushed the improvement/modern-pgsql branch from 4d74a70 to f5bf5ae Compare February 3, 2023 23:54
greg0ire
greg0ire approved these changes Feb 4, 2023
View reviewed changes
Copy link
Member

@greg0ire greg0ire left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice to get rid of all that cruft 😌

@derrabus derrabus merged commit e543ade into doctrine:4.0.x Feb 4, 2023
@derrabus derrabus deleted the improvement/modern-pgsql branch February 4, 2023 08:56
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Feb 5, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@greg0ire greg0ire greg0ire approved these changes

@SenseException SenseException Awaiting requested review from SenseException

Assignees
No one assigned
Labels
Drivers Improvement pgsql PHP
Projects
None yet
Milestone
4.0.0-beta2
Development

Successfully merging this pull request may close these issues.
3 participants
@derrabus @greg0ire @SenseException