search

getkirby / starterkit

Kirby's sample site – the easiest way to get started with Kirby
https://getkirby.com
176 stars 52 forks source link

Use Symfony YAML handler #52

Open distantnative opened 2 months ago

distantnative commented 2 months ago

🚨 Only merge after 4.5.0 release

Description

Summary of changes

Reasoning

Before we switch the default in the Core, it would be good to run our kits first with this handler. Gives us more chances to catch any issues/differences that the new handler has, which would then allow us to make the default switch easier for developers.

distantnative commented 2 months ago

@bastianallgeier thinking about it. Maybe we wait for 4.5 with https://github.com/getkirby/kirby/pull/6638 included?

Yes to the comment.

distantnative commented 2 months ago

What could be a good comment?

already makes use of the more modern Symfony YAML parser: https://getkirby.com/docs/reference/system/options/yaml (will become the default in a future Kirby version)

bastianallgeier commented 2 months ago

That comment suggestion is great!

distantnative commented 2 months ago

Added the comment