Skip to content
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

Make the site compatible with latest Hugo #64

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

DavidSpickett
Copy link
Contributor

@DavidSpickett DavidSpickett commented Nov 19, 2024

I tested this with Hugo v0.119.0 and latest at time of writing, v0.139.0. It works with both.

(though I don't know which version is being used for the live site, it needs testing with that too)

This fixes a number of issues in our theme, https://github.com/avianto/hugo-kiera.

I'm doing this here instead of upstream because upstream has not been updated in 5 years. I am also not updating the theme to the "latest" because it's likely to break more than it fixes for us.

I have fixed the following problems we had with the latest Hugo.

WARN  deprecated: site config key paginate was deprecated in Hugo v0.128.0 and will be removed in a future release. Use pagination.pagerSize instead.

Just moved the setting in config.toml.

.Site.DisqusShortName was deprecated and is now removed (https://discourse.gohugo.io/t/error-deprecated-site-disqusshortname-was-deprecated-in-hugo-v0-120-0-and-will-be-removed-in-hugo-0-133-0-use-site-config-services-disqus-shortname-instead/51111)

Use the new name (though we don't enable Disqus anyway).

.Site.GoogleAnalytics was deprecated and is now removed (https://discourse.gohugo.io/t/site-googleanalytics-was-deprecated-in-hugo-v0-120-0-and-will-be-removed-in-hugo-0-134-0/51395)

Use the new name again (another thing we didn't set ayway).

ERROR deprecated: .Site.Author was deprecated in Hugo v0.124.0 and will be removed in Hugo 0.140.0. Implement taxonomy 'author' or use .Site.Params.Author instead.

Moved the author settings into params.author in config.toml.

I'm not sure every option still works but all the things I could get to on the site do work still.

I tested this with Hugo v0.119.0 and latest at time of writing,
v0.139.0. It works with both.

(though I don't know which version is being used for the live site,
it needs testing with that too)

This fixes a number of issues in our theme, https://github.com/avianto/hugo-kiera.

I'm doing this here instead of upstream because upstream has not been updated
in 5 years. I am also not updating the theme to the "latest" because it's likely
to break more than it fixes for us.

I have fixed the following problems we had with the latest Hugo.

WARN  deprecated: site config key paginate was deprecated in Hugo v0.128.0 and will be removed in a future release. Use pagination.pagerSize instead.

Just moved the setting in config.toml.

.Site.DisqusShortName was deprecated and is now removed (https://discourse.gohugo.io/t/error-deprecated-site-disqusshortname-was-deprecated-in-hugo-v0-120-0-and-will-be-removed-in-hugo-0-133-0-use-site-config-services-disqus-shortname-instead/51111)

Use the new name (though we don't enable Disqus anyway).

.Site.GoogleAnalytics was deprecated and is now removed (https://discourse.gohugo.io/t/site-googleanalytics-was-deprecated-in-hugo-v0-120-0-and-will-be-removed-in-hugo-0-134-0/51395)

Use the new name again (another thing we didn't set ayway).

ERROR deprecated: .Site.Author was deprecated in Hugo v0.124.0 and will be removed in Hugo 0.140.0. Implement taxonomy 'author' or use .Site.Params.Author instead.

Moved the author settings into params.author in config.toml.

I'm not sure every option still works but all the things I could get to on
the site do work still.
@DavidSpickett DavidSpickett marked this pull request as ready for review November 19, 2024 15:40
@DavidSpickett
Copy link
Contributor Author

Actually there is a fork that is updated https://github.com/funkydan2/hugo-kiera. I will try upgrading to that so you can compare the options.

@DavidSpickett
Copy link
Contributor Author

#65 is the alternative.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant