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 build fails to build due to shortcode reference #7242

Closed
tomweston opened this issue Mar 18, 2022 · 3 comments · Fixed by #7251
Closed

make build fails to build due to shortcode reference #7242

tomweston opened this issue Mar 18, 2022 · 3 comments · Fixed by #7251
Labels
kind/engineering Work that is not visible to an external user resolution/fixed This issue was fixed

Comments

@tomweston
Copy link
Contributor

Problem description

make build fails to build due to shortcode reference

Error: Error building site: "/tmp/hugo_cache/modules/filecache/modules/pkg/mod/github.com/pulumi/pulumi-hugo/themes/[email protected]/content/blog/aws-eks-managed-nodes-fargate/index.md:36:1": failed to render shortcode "tweet": failed to process shortcode: template: _internal/shortcodes/tweet.html:6:82: executing "_internal/shortcodes/tweet.html" at <index .Params 0>: error calling index: value has type int; should be string
Total in 60070 ms
Makefile:44: recipe for target 'build' failed
make: *** [build] Error 255

Affected product version(s)

master @ 09cef63

@tomweston tomweston added the needs-triage Needs attention from the triage team label Mar 18, 2022
@nimbinatus
Copy link

Hey @tomweston! Quick question: Which version of Hugo are you on? There was an update to the Twitter API, and therefore an update to the shortcode, upstream. We adopted the update, which would leave the error message that you're finding there if you're on a Hugo version before 0.89.0. I bumped up to 0.92.0 and built cleanly.

@tomweston
Copy link
Contributor Author

Thanks @nimbinatus I was using hugo v0.86.0+extended linux/amd64 which was >= 0.81 as referenced in the README.

Updating Hugo to 0.92.0 fixed it.

Ill raise a PR to change the README to the correct version

@susanev susanev added kind/engineering Work that is not visible to an external user and removed needs-triage Needs attention from the triage team labels Mar 19, 2022
@pulumi-bot pulumi-bot added the resolution/fixed This issue was fixed label Mar 19, 2022
@nimbinatus
Copy link

Ah, thanks for catching that in the readme, and thanks for the PR!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/engineering Work that is not visible to an external user resolution/fixed This issue was fixed
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants