[Yaml] Add the`Yaml::DUMP_FORCE_DOUBLE_QUOTES_ON_VALUES` flag to enforce double quotes around string values#20758

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Jump to bottom

Merged

javiereguiluz merged 1 commit intosymfony:7.3fromalamirault:feature/20734-dump-force-double-quotes

Mar 12, 2025

Merged

[Yaml] Add the`Yaml::DUMP_FORCE_DOUBLE_QUOTES_ON_VALUES` flag to enforce double quotes around string values#20758

javiereguiluz merged 1 commit intosymfony:7.3fromalamirault:feature/20734-dump-force-double-quotes

Mar 12, 2025

Conversation

Copy link

Contributor

alamirault commentedMar 12, 2025

Fix#20734

[Yaml] Add the Yaml::DUMP_FORCE_DOUBLE_QUOTES_ON_VALUES flag to enfor…

152d436

…ce double quotes around string values

alamirault requested a review fromxabbuh as acode owner

March 12, 2025 10:02

carsonbot added Status: Needs Review Yaml labels

Mar 12, 2025

carsonbot added this to the7.3 milestone

Mar 12, 2025

alexandre-daubois approved these changes

Mar 12, 2025

View reviewed changes

carsonbot added Status: Reviewed and removed Status: Needs Review labels

Mar 12, 2025

javiereguiluz merged commit068abee intosymfony:7.3

Mar 12, 2025

3 checks passed

Copy link

Member

javiereguiluz commentedMar 12, 2025

Antoine, thanks for your contribution!

Your proposed contents were correct, but I was missing some precision when we mentioned "values" (which values? all? only string? etc.) So, I tested this feature a bit and found that only applies to string values, so I updated the code examples to show all types of values so it's more clear the impact of this feature. See409ec02