Skip to content

Resolves #939 replaced single quotes with nowdoc #1473

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

Merged
merged 1 commit into from
Nov 30, 2024

Conversation

kamil-karkus
Copy link
Contributor

Q A
Type improvement
BC Break no
Fixed issues #939

@greg0ire
Copy link
Member

Thanks for working on this. It seems there are CI jobs failing. Please take a look at this guide for more on how to handle those.

@kamil-karkus kamil-karkus force-pushed the 3.8.x branch 3 times, most recently from 4e4f8c4 to 7b67927 Compare November 29, 2024 14:33
@kamil-karkus
Copy link
Contributor Author

@greg0ire thanks, I had to fix units.

@greg0ire greg0ire changed the base branch from 3.8.x to 3.9.x November 29, 2024 14:55
@greg0ire

This comment was marked as resolved.

It makes it easier to write queries that contain quotes and single
quotes, and to get syntactic coloration.

Fixes doctrine#939
@greg0ire greg0ire added this to the 3.9.0 milestone Nov 30, 2024
@greg0ire greg0ire merged commit 72f58ae into doctrine:3.9.x Nov 30, 2024
12 checks passed
@greg0ire
Copy link
Member

Thanks @kamil-karkus !!!

@kamil-karkus
Copy link
Contributor Author

@greg0ire Thank you for taking care of this issue. I was planning to get back to it on Monday, following the guide you sent.

@vinise
Copy link
Contributor

vinise commented May 28, 2025

Really sad not to see any option on this feature.
could have been great to be able to disable it, or an option to activate this either on formatted sql only or based on --line-length=LINE-LENGTH (or other option)
(I might create a PR for this if I find some time)

;)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants