-
Notifications
You must be signed in to change notification settings - Fork 370
Add missing RSS feed, update zola #2632
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
@thibaultamartin what version does cloudflare currently use? :) We obviously cant merge this if our cloudflare deployment doesnt support it yet. |
You are uploading the built html page to cloudflare. The zola build process is running inside the GitHub Actions |
Its a little more difficult than that in reality since we also use the preview feature of cloudflare which is built on the side of cloudflare instead :) |
In the meanwhile 0.20.0 has been released. Could you update this PR to it? :) https://github.com/getzola/zola/releases/tag/v0.20.0 |
(SRE was pinged we are looking into the cloudflare side of things still) |
Signed-off-by: Daniel Kilimnik <[email protected]>
Signed-off-by: Daniel Kilimnik <[email protected]>
Signed-off-by: Daniel Kilimnik <[email protected]>
This is https://github.com/element-hq/sre-internal/issues/3390 in SRE's internal issue tracker |
Zola has been updated via #2713 Sorry for doing a separate PR, there had to be some coordination between the Cloudflare and GitHub side! Thanks for getting the effort going :) |
While zola was updated, the rss.xml is not there yet. |
I noticed that the templates include an
rss.xml
definition, though you are not using it. The default zola behaviour is to use theatom.xml
, so I updated theconfig.toml
to use both.I also changed
generate_feed
togenerate_feeds
as this is the new option which should be used with the new version.As this is a new zola feature, I also decided to update zola to the newest version.