Fix compatibility with Mojolicious 9 and YAML 1.30

preaction / Statocles

Static website CMS

http://preaction.me/statocles

Other

84 stars 33 forks source link

Fix compatibility with Mojolicious 9 and YAML 1.30 #602

Closed voegelas closed 3 years ago

voegelas commented 3 years ago

There are three commits in this pull request.

lib/Statocles/Command/daemon.pm: Taken from Yancy for compatibility with Mojolicious 9.
t/theme/check.t: Several parens are in the wrong places and warnings are output.
lib/Statocles/Test.pm: If $YAML::Stringify is not set to 1, the serialized Path::Tiny objects break the site.yml so that some tests fail.

Tested on Linux with Perl 5.32.1 and the most recent versions of all required modules.

coveralls commented 3 years ago

Coverage increased (+0.007%) to 89.929% when pulling e8b25e2901864784233f85153aaa61781565e74a on voegelas:mojolicious-9 into 5d631ccfefe6bb1ee192e70566c3b87fb8902311 on preaction:master.

preaction commented 3 years ago

Thanks! I fixed the YAML issue slightly differently, but I had no idea what the Mojo::DOM issue was, so thanks for fixing that one! I've cherry-picked that commit onto v0 and will release it shortly.