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

Extend AbstractMySQLPlatform::getColumnTypeSQLSnippets() deprecation layer #6215

Merged

Conversation

derrabus
Copy link
Member

@derrabus derrabus commented Nov 5, 2023

Q A
Type improvement
Fixed issues Follows #6213

Summary

We want to stop guessing the current database.

cc @ausi

@derrabus derrabus added this to the 3.8.0 milestone Nov 5, 2023
@derrabus derrabus merged commit 4dd0796 into doctrine:3.8.x Nov 6, 2023
84 checks passed
@derrabus derrabus deleted the improvement/extend-gcts-deprecations branch November 6, 2023 18:50
derrabus added a commit to derrabus/dbal that referenced this pull request Nov 11, 2023
* 3.8.x:
  Enable skipping locked rows in QueryBuilder
  Fix coding standard violation
  Remove unused unit test fixtures
  Add Oracle v23 to CI testing (doctrine#6217)
  Extend `AbstractMySQLPlatform::getColumnTypeSQLSnippets()` deprecation layer (doctrine#6215)
  Deprecate AbstractMySQLPlatform::getColumnTypeSQLSnippets() (doctrine#6213)
  Fix MariaDB list columns performance (doctrine#6202)
derrabus added a commit to derrabus/dbal that referenced this pull request Nov 11, 2023
* 3.8.x:
  Enable skipping locked rows in QueryBuilder
  Fix coding standard violation
  Remove unused unit test fixtures
  Add Oracle v23 to CI testing (doctrine#6217)
  Extend `AbstractMySQLPlatform::getColumnTypeSQLSnippets()` deprecation layer (doctrine#6215)
  Deprecate AbstractMySQLPlatform::getColumnTypeSQLSnippets() (doctrine#6213)
  Fix MariaDB list columns performance (doctrine#6202)
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Nov 6, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants