matrix-org / matrix-spec

The Matrix protocol specification
Apache License 2.0
171 stars 91 forks source link

Upgrade version of Hugo used to build the spec in CI #1794

Closed zecakeh closed 2 months ago

zecakeh commented 2 months ago

Bumped to Hugo 0.124.1, because there is an incompatibility with docsy in the latest version (0.125.0): https://github.com/google/docsy/issues/1930.

The second commit is a fix for the rendered spec in the render-object-table partial.

The behavior of delimit changed, Hugo doesn't recognize "safe" HTML passed to it anymore, so it escapes nested HTML links even if they were marked as "safe", like this:

image

To fix that we escape the schema data manually and consider the output of the property-type partial as "safe".

I usually do a diff to check if there are visible changes in the rendered spec. Sadly, it seems that the way that URLs are built has changed (most of them are absolute now when they where relative before), so the diff is filled with that. I didn't notice any other change in the rendered spec though.

Preview: https://pr1794--matrix-spec-previews.netlify.app