-
Notifications
You must be signed in to change notification settings - Fork 344
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
Some of the TODOs got into the final rendered HTML #11401
Comments
You're not using the correct EJS comment syntax: https://stackoverflow.com/questions/29180611/how-can-i-comment-the-ejs-code-js-node-without-getting-an-error |
These are not my comments :) linked code snippets belong here - |
You should configure your TODO detector to ignore the |
Ah, sorry. It turns out that we don't mind those TODOs there, but I'll make the change anyway. |
My auto TODO detector is screaming when looking into the

_site
subdir:This are repeating across the subpage and it looks like it mostly coming from those two:
quarto-cli/src/resources/formats/html/templates/quarto-html.ejs
Line 287 in adc7186
quarto-cli/src/resources/formats/html/templates/quarto-html.ejs
Line 510 in adc7186
It looks like it got injected into every single page I have. Since it's a comment it's probably not a big deal though but I thought I would share this with you.
The text was updated successfully, but these errors were encountered: