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

feat: address Hugo v0.92 notice RE changing behaviour of .Path #2642

Merged
merged 3 commits into from
Mar 3, 2022
Merged

feat: address Hugo v0.92 notice RE changing behaviour of .Path #2642

merged 3 commits into from
Mar 3, 2022

Conversation

mcanouil
Copy link
Contributor

@mcanouil mcanouil commented Feb 28, 2022

Based on Hugo documentation (https://gohugo.io/variables/files/) and deprecation notice from HUGO > v0.92.0.
It seems the solution is to replace the use of .Pathwith .File.Path.
Fix #2592

@netlify
Copy link

netlify bot commented Feb 28, 2022

✔️ Deploy Preview for hugo-portfolio-theme canceled.

🔨 Explore the source changes: 5b93076

🔍 Inspect the deploy log: https://app.netlify.com/sites/hugo-portfolio-theme/deploys/622028eec2ef360008ad4f0f

@netlify
Copy link

netlify bot commented Feb 28, 2022

✔️ Deploy Preview for markdown-slides canceled.

🔨 Explore the source changes: 5b93076

🔍 Inspect the deploy log: https://app.netlify.com/sites/markdown-slides/deploys/622028ee54b31a0008f7ab6c

@gcushen
Copy link
Collaborator

gcushen commented Mar 2, 2022

@mcanouil thanks for contributing!

There appears to be a typo in the PR. .File is an object, not a string as you intended. Perhaps you were thinking of .File.Path?

@mcanouil
Copy link
Contributor Author

mcanouil commented Mar 3, 2022

@gcushen Oh yes (failed search/replace ...) and in fact, I do not think path.Split is still needed, if .File.Dir is used directly.

@gcushen gcushen changed the title Hugo-deprecation-path feat: address Hugo v0.92 notice RE changing behaviour of .Path Mar 3, 2022
@gcushen gcushen merged commit 97a4b87 into HugoBlox:main Mar 3, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Deprecation notice of .Path for Hugo > 0.92.0
2 participants