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

Pull in the latest code from Go's template packages #11771

Merged
merged 1 commit into from Dec 4, 2023

Conversation

bep
Copy link
Member

@bep bep commented Dec 1, 2023

Fixes #11757
Fixes #11507

@bep bep force-pushed the fix/go-jsliterals-11757-11507 branch from b638f90 to 5ce8bc1 Compare December 1, 2023 11:31
@bep bep force-pushed the fix/go-jsliterals-11757-11507 branch from 5ce8bc1 to f6e0663 Compare December 1, 2023 13:54
@bep
Copy link
Member Author

bep commented Dec 1, 2023

@jmooring I have a nagging feeling that this i going to bite us, but I suspect this to be a big enough problem to not wait for Go 1.22?

@jmooring
Copy link
Member

jmooring commented Dec 1, 2023

This PR has no effect on #11757, as expected. #11757 is invalid (user error).

This PR fixes:

I suspect this to be a big enough problem to not wait for Go 1.22

Of the two issues above, only 11507 was a regression, so I don't think this is a big problem. However, getting it out there early might uncover something new, which could potentially be fixed prior to Go 1.22 (Feb 2024).

@bep bep merged commit 9f978d3 into gohugoio:master Dec 4, 2023
8 checks passed
@jmooring jmooring mentioned this pull request Feb 7, 2024
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.

Can't build site after upgrading Hugo Hugo 0.119 no longer builds - issue with js html template parsing
2 participants