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

use .Sites.Default instead of .Sites.First #622

Closed
wants to merge 1 commit into from

Conversation

grimreaper
Copy link
Contributor

Problem

The warning

INFO  deprecated: .Sites.First was deprecated in Hugo v0.127.0 and will be removed in a future release. Use .Sites.Default instead.

appears

Solution

Follow the warning. Use .Sites.Default

@anthraxx
Copy link

gentle ping @alex-shpak

@alex-shpak
Copy link
Owner

alex-shpak commented Sep 2, 2024

Hi!
Please also bump minimum required version and build version, right now min version is 1.124 and I think .Sites.Default is no available there yet

https://github.com/alex-shpak/hugo-book/blob/master/.github/workflows/main.yml#L12

Otherwise LGTM

# Problem

The warning
```
INFO  deprecated: .Sites.First was deprecated in Hugo v0.127.0 and will be removed in a future release. Use .Sites.Default instead.
```

appears

# Solution

Follow the warning. Use `.Sites.Default`
@grimreaper
Copy link
Contributor Author

I updated the minimum version to 0.124.0. I believe this is the correct.

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.

3 participants