-
-
Notifications
You must be signed in to change notification settings - Fork 503
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
Eleventy 3 no longer finds helper modules when using Node 23 #3625
Comments
I fail to reproduce the behaviour. Steps I took:
I can observe some warnings from the UpgradePlugin in the console output but aside of that it builds. |
I used npm run production as the command, because I didn't want the server started at that time. However, my working directory was upgraded from eleventy 2 to 3 in a prior step, which worked fine, but after the upgrade to eleventy 3 it failed. I also removed the dist folder to make sure it was clean, but same result. I am at a loss... |
Cleaned the Perhaps you can run your upgrade in two phases?
According to https://github.com/11ty/eleventy-upgrade-help/tree/v2.x it would be installed using Then once you are on version 2 have another upgrade to version 3. |
That is exactly what I did. And the upgrade from v1 to v2 worked fine. The plugin for v2 didn't complain. I uninstalled it and removed it from the Eleventy config file. I then installed Eleventy v3, and the plugin, added the lines for the plugin v3, and then the problems started. First there was something about quotes in a file's front matter that hadn't been an issue before, and after I corrected that, the current problem appeared. So either there is something cached that shouldn't, or while upgrading Eleventy from v2 to v3, some dependency didn't work out as it would if v3 was freshly installed. Anyway, my status is that it doesn't work. Frustrating…
|
More info: I clobbered both the dist and node_modules folders, ran npm install again, and got the same result. I also ran an npm audit fix to update some modules that could be updated without issue, no change. Node was installed today via Home Brew which had just before been updated to the current state. No building from source, just the install and update from the HomeBrew website. I don't know what else to try. |
A helpful user on social media just found that my project doesn't work for them in Node 23.6.0, which is what I am using. That was installed by HomeBrew today. They got it working in Node 22 in a Docker container. So it may be fixed by #3519. However, if it is not, #3577 seems to also cover various Node compatibility issues. Advice would be appreciated if that bug fix currently in 3.0.1 alpha should help with this. |
I can see the original branch merged about three hours ago. |
Well, I resolved my problem by reverting to Node 22. However, I can test with node 23 via NVM if need be if Eleventy is updated. So in a way, my specific problem is solved by going back to an LTS version of Node, but if the actual Node 23 incompatibility affecting my project is solved already in Eleventy, I don’t know.
|
Glad you got it working 🥳 Given #3577 (comment) I suspect that odd versions are not officially supported. I would suggest to update the title to indicate it to be related to Node v23. I could imagine it being added to Eleventy 3.0.1 Milestone once Zach found time to look into it. |
Operating system
MacOS 15.2
Eleventy
3
Describe the bug
A project that built in v2.0.1 no longer builds in 3.0. The error is that a helper function is undefined or false. However, the line and column pointed to is actually a comment, not the JS module referenced. That is in the line below. Eleventy-Upgrade-Helper does not flag anything amiss with the project right above the error.
Reproduction steps
the lines in question are:
The error message is:
The line that is being complained about is the one with the comment, column 51 is the position of the first letter after the apostrophe.
Expected behavior
I expected the project to build as it was in 2.0.
Reproduction URL
Clone this to reproduce
Screenshots
No response
The text was updated successfully, but these errors were encountered: