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.

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

Manually merge 3.0.x into master #3904

Merged
merged 51 commits into from
Mar 18, 2020
Merged

Manually merge 3.0.x into master #3904

merged 51 commits into from
Mar 18, 2020

Conversation

morozov
Copy link
Member

@morozov morozov commented Mar 18, 2020

Q A
Type improvement
BC Break no

morozov and others added 30 commits January 4, 2020 05:19
Fix DebugStack#queries docblock type
…hema name (eg "812pp")

also simplifies the code and now exactly matches postgresql search rules
Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
…ntation

Fix docblock typos in DriverManager docs
Fix unquoted stmt fragments backslash escaping
Fixed the QueryBuilder::setMaxResults() signature to accept NULL
…olution

[pg] fix getting table information if search_path contains escaped schema name
When a Docker version of the build is available, use it.
PHP 7.4 has stable versions now.
Hopefully it will work more reliably.
It seems to be enabled by default.
They are no longer supported by their vendors, but until we do support
them, we should continue testing them.
@morozov
Copy link
Member Author

morozov commented Mar 18, 2020

@greg0ire please do not auto-merge. There's already a merge commit that we can use.

@morozov morozov added the WIP label Mar 18, 2020
@morozov
Copy link
Member Author

morozov commented Mar 18, 2020

In order to address the build failure, we need to deprecate the usage of the platforms like Postrgres 9.2, etc in 2.11.x (#3855) and drop their support in 3.0.x.

Actually, let's do this after the merge in #3906. I don't want to resolve the same conflicts again. For now, I just removed the PostrgreSQL 9.2 and 9.3 build jobs in the merge commit.

@greg0ire
Copy link
Member

Oh right I merged #3902, shouldn't have. I should know this, I do it all the time on the Sonata repos 🤦‍♂️

@greg0ire
Copy link
Member

IIRC we can use draft PRs to shield ourselves from my stupidity 🙂

@morozov
Copy link
Member Author

morozov commented Mar 18, 2020

@greg0ire IIRC the draft PRs had problems with Travis integration. In any event, this is no WIP and can be reviewed.

@greg0ire
Copy link
Member

@greg0ire IIRC the draft PRs had problems with Travis integration. In any event, this is no WIP and can be reviewed.

I remember issues too at the beginning. Will try on Sonata several times and let you know.

@morozov morozov merged commit 01143c9 into doctrine:master Mar 18, 2020
@morozov morozov self-assigned this Mar 18, 2020
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 4, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

10 participants