-
-
Notifications
You must be signed in to change notification settings - Fork 471
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
Better error messaging with circular references in Eleventy layouts #513
Comments
Could you share some more information?
I believe, this could help speed up debugging. |
Hey @Ryuno-Ki! Node Version: v10.14.2 I actually ran the second attempt (the second log I pasted in the initial issue) in a separate terminal after I had terminated all the watch tasks. I was literally just trying to run Diving into the code again, I think I found what it was- I created a layout called ---
layout: app
--- This created an infinite loop. Glad I figured it out. Might be nice to try and catch this sort of thing with an exception in the future. |
I'd be happy to submit a PR if you can suggest where in the code I'd need to go to make the change :) |
@weaversam8 Glad that you figured out the root issue :-) @zachleat Your project! You know best where too look at. |
I am getting the same error and did not want to start a new issue so apologies if I posted in the wrong place. Here's what I am seeing via the terminal;
I had been modifying the Eleventy-Netlfy-Boilerplate here but ran into some issues with templating again. So as of right now, I cannot deploy for whatever reason, to Netlify. I was last successful about a week and a half ago, so I don't know what had changed since then, I know I did an OS update on my Mac to the latest Mojave version, but that was pretty much it. Any help would be greatly appreciated. Thanks. |
@colabottles I'm no expert, but the Investigating the limited stack trace, if we look at the Again, I'm no expert in the internals of V8 so I don't really know how accurate this is, but maybe this is a lead to follow... |
@weaversam8 I will take a look through, That had crossed my mind when I initially read your issue. I didn't see anything glaring on my first pass through, but now that I have some time, I'm going to go back through and go through with a fine tooth comb. Thanks for your reply! |
@weaversam8 All files were fine, no loops in any template that I could see, but I'm no expert on this so I could be overlooking something someone with more experience would be able to spot quickly. I may try to back what I have up for code and then start over again from scratch. Not sure. I'm giving another few days before I do that. I'm also trying to find someone that would be willing to guide me through this and some templating questions I had about Eleventy. |
@colabottles probably makes sense to try and make a minimum replication case you can publish... starting from scratch (or walking back to the last working state) should help with that. If you have any questions about Eleventy (I've only worked with it a bit) feel free to reach out. You can get in touch with me via my Keybase! |
Thanks for the issue @weaversam8. Do you care if I switch it over to a Enhancement Request for loop detection in Eleventy layouts? I think this would be a great addition. Sometimes they are hard to spot, I’d love to have better messaging around this |
@zachleat No, I don't mind! That sounds like a great plan! |
This repository is now using lodash style issue management for enhancements. This means enhancement issues will now be closed instead of leaving them open. View the enhancement backlog here. Don’t forget to upvote the top comment with 👍! |
Thanks to @AleksandrHovhannisyan this shipped with 2.0.0-canary.10! |
Hey folks, sorry for the lack of a more descriptive issue name, but it isn't immediately apparent to me what's happening here.
I have my project set up to watch the various source files as I edit them (JS, SCSS, and HTML with Eleventy) and while editing, I noticed oddly that Eleventy seemed to crash with a lot of error messages that appeared low level. I believe it happened when I opened up a split terminal and installed another NPM package, but it could've just been a coincidence.
The relevant scripts from my
package.json
:The initial error log that occurred, in the output:
As you can see, this is quite hefty. I included a successful run of Eleventy from the log right before the error hit... it seemed to come out of the blue. I notice a lot of references to memory issues, but my laptop seems to have enough memory currently free to run JS VM.
The kicker is that now I can't compile anything from Eleventy anymore. I tried to run it again in a new command window with debug flags on and the following output resulted.
Also, worth noting that the above retry attempt hung for several minutes. I ran it with debug flags, saw it hang (right after
TemplatePassthrough copy finished
), started writing this issue, and then looked back and realized it had crashed.I'm probably going to restart my computer next but figured I'd capture this here now. Any ideas as to what could've caused this? I know it might not be related to the project but I wanted to document it in case it is.
The text was updated successfully, but these errors were encountered: