At the outset, our test suite only flagged the use of YAML.load in our Cucumber helper, but what if allowing psych 4 breaks some plugins downstream?
The Jekyll Core uses SafeYAML extensively. So what if we were to switch the use of YAML.load with its SafeYAML counterpart?

I like the idea of using SafeYAML everywhere, I'll try to have a patch out later today 👍

Hi, could this be related to #8772 ?

I have now updated the PR to use SafeYAML rather than depend on psych 👍

@fauno I would say that this is not directly related to #8772 as it is smaller in scope. It does however affect it as it introduces a new place where SafeYAML has to be swapped for psych's YAML.

CI is failing on ruby 2.6 Windows with:

Failure:
TestSite#test_: configuring sites should configure cache_dir.  [C:/projects/jekyll/test/test_site.rb:81]
Minitest::Assertion: Expected false to be truthy.
Skipped:

I can't tell if this is a flake or if it's related to this though? 🤔

Thanks @yboulkaid
@jekyllbot: merge +fix