Skip to content
Permalink

Comparing changes

Choose two branches to see what’s changed or to start a new pull request. If you need to, you can also or learn more about diff comparisons.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also . Learn more about diff comparisons here.
base repository: symfony/yaml
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: 7.2
Choose a base ref
...
head repository: symfony/yaml
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: 7.3
Choose a head ref
Checking mergeability… Don’t worry, you can still create the pull request.
  • 16 commits
  • 8 files changed
  • 8 contributors

Commits on Oct 29, 2024

  1. [Yaml] Add support for dumping null as an empty value by using the …

    …`Yaml::DUMP_NULL_AS_EMPTY` flag
    alexandre-daubois committed Oct 29, 2024
    Configuration menu
    Copy the full SHA
    606d725 View commit details
    Browse the repository at this point in the history

Commits on Dec 12, 2024

  1. chore: PHP CS Fixer fixes

    keradus authored and fabpot committed Dec 12, 2024
    Configuration menu
    Copy the full SHA
    254b973 View commit details
    Browse the repository at this point in the history

Commits on Dec 14, 2024

  1. chore: PHP CS Fixer fixes

    keradus authored and fabpot committed Dec 14, 2024
    Configuration menu
    Copy the full SHA
    7a9c072 View commit details
    Browse the repository at this point in the history

Commits on Dec 29, 2024

  1. feature #58243 [Yaml] Add support for dumping null as an empty valu…

    …e by using the `Yaml::DUMP_NULL_AS_EMPTY` flag (alexandre-daubois)
    
    This PR was merged into the 7.3 branch.
    
    Discussion
    ----------
    
    [Yaml] Add support for dumping `null` as an empty value by using the `Yaml::DUMP_NULL_AS_EMPTY` flag
    
    | Q             | A
    | ------------- | ---
    | Branch?       | 7.2
    | Bug fix?      | no
    | New feature?  | yes
    | Deprecations? | no
    | Issues        | Fix #58155
    | License       | MIT
    
    Another try after #58232. I went for the second solution from `@stof` (symfony/symfony#58232 (comment)), namely dumping empty value as null just in expanded sequences and mappings. When the inline limit of the dumper is reached, it fallbacks on using `null` (examples in tests).
    
    I'm not 100% happy about the name of the constant, `DUMP_NULL_AS_EMPTY_IN_EXPANDED_NOTATION`. It's verbose. I'm open to suggestions!
    
    Commits
    -------
    
    1fc1379028 [Yaml] Add support for dumping `null` as an empty value by using the `Yaml::DUMP_NULL_AS_EMPTY` flag
    xabbuh committed Dec 29, 2024
    Configuration menu
    Copy the full SHA
    f5fa82b View commit details
    Browse the repository at this point in the history

Commits on Jan 2, 2025

  1. [Yaml] Add compact nested mapping support to Dumper

    gr8b authored and fabpot committed Jan 2, 2025
    Configuration menu
    Copy the full SHA
    b5176af View commit details
    Browse the repository at this point in the history

Commits on Jan 7, 2025

  1. Merge branch '7.2' into 7.3

    * 7.2: (47 commits)
      Remove comment about AppVeyor in `phpunit`
      [Webhook][RemoteEvent] fix SendgridPayloadConverter category support
      Update old Appveyor skip conditions
      sync the Dutch translation file with changes from the 7.2 branch
      [Yaml] fix inline notation with inline comment
      clean up code for doctrine/persistence 2.x
      Generate missing translations using Gemini
      fix(property-info): make sure that SerializerExtractor returns null for invalid class metadata
      add translations for the Slug constraint
      [RemoteEvent][Webhook] fix SendgridRequestParser & SendgridPayloadConverter in case of missing sg_message_id
      [Messenger] Fix `TransportMessageIdStamp` not always added
      [DoctrineBridge] Fix compatibility to Doctrine persistence 2.5 in Doctrine Bridge 6.4 to avoid Projects stuck on 6.3
      [PropertyInfo] Fix add missing composer conflict
      [ErrorHandler] Don't trigger "internal" deprecations for anonymous LazyClosure instances
      [VarDumper] Fix displaying closure's "this" from anonymous classes
      [Doctrine][Messenger] Prevents multiple TransportMessageIdStamp being stored in envelope
      [HttpKernel] Don't override existing LoggerInterface autowiring alias in LoggerPass
      reject inline notations followed by invalid content
      Fix predis command error checking
      [Security] Fix triggering session tracking from ContextListener
      ...
    xabbuh committed Jan 7, 2025
    Configuration menu
    Copy the full SHA
    17f70c9 View commit details
    Browse the repository at this point in the history

Commits on Jan 10, 2025

  1. chore: PHP CS Fixer fixes

    keradus authored and nicolas-grekas committed Jan 10, 2025
    Configuration menu
    Copy the full SHA
    8e71385 View commit details
    Browse the repository at this point in the history

Commits on Jan 19, 2025

  1. chore: PHP CS Fixer fixes

    keradus authored and GromNaN committed Jan 19, 2025
    Configuration menu
    Copy the full SHA
    5fbfc82 View commit details
    Browse the repository at this point in the history

Commits on Feb 4, 2025

  1. Configuration menu
    Copy the full SHA
    669059b View commit details
    Browse the repository at this point in the history
  2. bug #59671 [Yaml] Yaml::DUMP_COMPACT_NESTED_MAPPING does not apply wh…

    …en dumping sequences (xabbuh)
    
    This PR was squashed before being merged into the 7.3 branch.
    
    Discussion
    ----------
    
    [Yaml] Yaml::DUMP_COMPACT_NESTED_MAPPING does not apply when dumping sequences
    
    | Q             | A
    | ------------- | ---
    | Branch?       | 7.3
    | Bug fix?      | yes
    | New feature?  | no
    | Deprecations? | no
    | Issues        |
    | License       | MIT
    
    following #59315, the flag should only affect how maps are dumped but shouldn't influence dumping sequences (see also symfony/symfony-docs#20510 (comment))
    
    Commits
    -------
    
    0c124a99e91 [Yaml] Yaml::DUMP_COMPACT_NESTED_MAPPING does not apply when dumping sequences
    nicolas-grekas committed Feb 4, 2025
    Configuration menu
    Copy the full SHA
    491c6ac View commit details
    Browse the repository at this point in the history

Commits on Mar 3, 2025

  1. Merge branch '7.2' into 7.3

    * 7.2:
      Review validator-related japanese translations with ids 114-120
      don't trigger "internal" deprecations for PHPUnit Stub objects
      drop comments while lexing unquoted strings
    xabbuh committed Mar 3, 2025
    Configuration menu
    Copy the full SHA
    3d545d9 View commit details
    Browse the repository at this point in the history

Commits on Mar 6, 2025

  1. [Yaml] Add the Yaml::DUMP_FORCE_DOUBLE_QUOTES_ON_VALUES flag to enf…

    …orce double quotes around string values
    dkarlovi committed Mar 6, 2025
    Configuration menu
    Copy the full SHA
    a2f18c2 View commit details
    Browse the repository at this point in the history

Commits on Mar 8, 2025

  1. chore: PHP CS Fixer fixes

    keradus authored and fabpot committed Mar 8, 2025
    Configuration menu
    Copy the full SHA
    e359788 View commit details
    Browse the repository at this point in the history
  2. feature #59880 [Yaml] Add the `Yaml::DUMP_FORCE_DOUBLE_QUOTES_ON_VALU…

    …ES` flag to enforce double quotes around string values (dkarlovi)
    
    This PR was merged into the 7.3 branch.
    
    Discussion
    ----------
    
    [Yaml] Add the `Yaml::DUMP_FORCE_DOUBLE_QUOTES_ON_VALUES` flag to enforce double quotes around string values
    
    | Q             | A
    | ------------- | ---
    | Branch?       | 7.3
    | Bug fix?      | no
    | New feature?  | yes
    | Deprecations? | no
    | Issues        | Fix #59680
    | License       | MIT
    
    In some scenarios, quoting even safe values is encouraged by various inspectors, for example PHPStorm inspectors will mark it with a warning when used in Kubernetes `ConfigMap` dump.
    
    This option allows forcing the quotes even when technically not required, see #59680 for an example.
    
    Commits
    -------
    
    1c06770e099 [Yaml] Add the `Yaml::DUMP_FORCE_DOUBLE_QUOTES_ON_VALUES` flag to enforce double quotes around string values
    fabpot committed Mar 8, 2025
    Configuration menu
    Copy the full SHA
    54005e3 View commit details
    Browse the repository at this point in the history

Commits on Apr 4, 2025

  1. make data provider static

    xabbuh committed Apr 4, 2025
    Configuration menu
    Copy the full SHA
    feaf385 View commit details
    Browse the repository at this point in the history
  2. Merge branch '7.2' into 7.3

    * 7.2:
      make data provider static
    fabpot committed Apr 4, 2025
    Configuration menu
    Copy the full SHA
    cea40a4 View commit details
    Browse the repository at this point in the history
Loading