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: v4.4.21
Choose a base ref
...
head repository: symfony/yaml
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: v4.4.22
Choose a head ref
  • 7 commits
  • 5 files changed
  • 4 contributors

Commits on Apr 1, 2021

  1. fixed parser

    a1812 authored and nicolas-grekas committed Apr 1, 2021
    Copy the full SHA
    416d6c0 View commit details

Commits on Apr 5, 2021

  1. [Yaml] Fixed infinite loop when parser goes through an additional and…

    … invalid closing tag
    
    Instead of letting the parser goes in an infinite loop, throw an exception when the additional and invalid is found
    alexandre-daubois committed Apr 5, 2021
    Copy the full SHA
    bcef62d View commit details

Commits on Apr 23, 2021

  1. Copy the full SHA
    ea53824 View commit details
  2. bug #40514 [Yaml] Allow tabs as separators between tokens (bertramakers)

    This PR was merged into the 4.4 branch.
    
    Discussion
    ----------
    
    [Yaml] Allow tabs as separators between tokens
    
    | Q             | A
    | ------------- | ---
    | Branch?       | 4.4
    | Bug fix?      | yes
    | New feature?  | no
    | Deprecations? | no
    | Tickets       | Fix #40507
    | License       | MIT
    | Doc PR        | None
    
    As described in #40507, the Yaml spec allows tabs as whitespace characters between tokens. However, the Yaml parser crashes on this as it only expects spaces after the colon. https://yaml.org/spec/1.2/spec.html#id2778241
    
    While I'm not a huge fan of it personally, it's an issue when a different linter tells us that a given YAML file with content that we have little control over has valid syntax in an unrelated check, and then our app crashes because it cannot be parsed after all.
    
    Commits
    -------
    
    9a130ae93e Fix issue 40507: Tabs as separators between tokens
    xabbuh committed Apr 23, 2021
    Copy the full SHA
    ef879ba View commit details
  3. make fabbot happy

    xabbuh committed Apr 23, 2021
    Copy the full SHA
    8c5137b View commit details
  4. minor #40924 [Yaml] make fabbot happy (xabbuh)

    This PR was merged into the 4.4 branch.
    
    Discussion
    ----------
    
    [Yaml] make fabbot happy
    
    | Q             | A
    | ------------- | ---
    | Branch?       | 4.4
    | Bug fix?      | no
    | New feature?  | no
    | Deprecations? | no
    | Tickets       |
    | License       | MIT
    | Doc PR        |
    
    Commits
    -------
    
    9826c05928 make fabbot happy
    xabbuh committed Apr 23, 2021
    Copy the full SHA
    e1515d3 View commit details
  5. Copy the full SHA
    1c2fd24 View commit details
Loading