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

Syntax error in tsconfig.json causes out-of-memory error #614

Open
NullVoxPopuli opened this issue Aug 29, 2023 · 3 comments
Open

Syntax error in tsconfig.json causes out-of-memory error #614

NullVoxPopuli opened this issue Aug 29, 2023 · 3 comments

Comments

@NullVoxPopuli
Copy link
Contributor

This was a goofy discovery 😅

I kept getting OOM exceptions from node when trying to run glint via pnpm lint:types (package.json "lint:types": "glint"), so I kept increasing the amount of memory used -- I finally ended up with

NODE_OPTIONS="--max-old-space-size=12000" pnpm lint:types # or just pnpm glint directly

And I forgot the process was running cause prior attempts at increase the node ram were getting slower and slower -- this ultimately took 36 minutes before the OOM error occurred.

I was ultimately trying to debug performance, and eventually ran:

pnpm tsc --generateTrace trace.txt --traceResolution

(or something to that effect)
And tsc provided me the error right away:
image

So, from my perspective, it looks like glint is trying to do something and continuing to do something even thought tsc should error? idk.

@chriskrycho
Copy link
Member

Thanks for the report; this seems like a case where Glint should be bailing and is not! Do you happen to have a minimal reproduction of this?


Aside: In the future, please copy and paste the output rather than sharing a screenshot. Screenshots are fundamentally not accessible to lots of folks who may need to interact with the repo. I happen to be well-sighted, but not everyone is!

@NullVoxPopuli
Copy link
Contributor Author

Screenshots are fundamentally not accessible to lots of folks who may need to interact with the repo

yes, I did not have the text available at the time. <3

@NullVoxPopuli
Copy link
Contributor Author

Do you happen to have a minimal reproduction of this?

I do not :(

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

No branches or pull requests

2 participants