Happen Issue when try to watch:start with examples
I meet this issue when trying to clone example and watch:start
43 Replies
Using nove v18 , ts 5.0.4
GitHub
TS 5.1.3 Regression: TSC Compiler throws FATAL ERROR: Reached heap ...
Bug Report 🔎 Search Terms tsc, compiler, heap limit, crash, memory 🕗 Version & Regression Information This is a crash This changed between versions 5.0.4 and 5.1.3 ⏯ Repository Link https://git...
Some people said that relate to shapeshift but example not install shapeshift
framework->@discordjs/builders->shapeshift
So it crash
@Favna hope when you online can check this issue since people can't start new fresh bot development
@HellCatVN I noticed it as well but I have no idea what the solution is
Building shapeshift on 5.1.3 causing an out of memory exception
Oh I just read the issue
Time to apply those fixes
cc @kyra @vladdy y'all didn't reply to me noting the issue before but above is the link with the reason and fix
Hah its the thing i wanna cleanup
Which is removing the types that ensures tuple length
I'm also facing this, 5.0.3 works for me, latest version doesn't
Its a mix of us doing something overkill and TS breaking
https://github.com/MRDGH2821/Perpetual-Mechanical-Array-Bot/blob/rewrite-sapphire/package.json
Fortunately my stuff is working with this package.json
The moment I upgrade Typescript, all GC hell breaks loose
Yeah that's clear
Will you make a PR then? As in, today preferebly?
Well lets see
I'm in Bulgaria rn
I could probably try to
how's it looking @vladdy ?
I'm still in bulgaria
And will still be in bulgaria until tomorrow
Including tomorrow
"no" is also a valid answer
No is also a useless answer
In this context
no i mean to this question ofc
Oh
Does that fix it?
O.o
If yes wtf typescript
If Anders says it does then it probably does
it's Anders after all
Well idc what they say, try it and see
linting is still f*cked urgh
but yes the type instantion is fixed
Building works in other projects?
I assume so
Can you try it please
already on it
Bless
The code looks fine (dumb, but fine), so..if it builds then it builds
W
Stupid w
just fixing the linting now
for some stupid reason the
disable-next-line
still has the rule being checked in some way
bah
the only way around it is disabling the rule entirely for the project
stupid but whatever
we dont really need the rule anyway
we can check it ourselves well enough
for the record it's @typescript-eslint/dot-notation
Or disable it via /* */ comments
eslint-disable
eslint-enable
tried it
didnt work
Or even outright for the file
O.o?
yeah idk man
tried overrides in eslintrc too
didnt work
Whats the lint rule?
the one that fails?
Ye
the expect here:
That's not a lint rule o.o
Or?
That doesn't answer my question mind you, idk what error you saw 😂😂
ANYWAY overrides does work
I just had a wrong glob before
pushed, reapprove pls ❤️
Oh LOL that's...funky
Cri
Hmm, you disabled it for tests only but edited validators too
Idk
yeah my bad
I went a bit too hard on find and replace
I revert the ones in src
Released shapeshift v3.5.2 @HellCatVN
Testing tonight and feedback soon
it works for @Dragonite and @Gemboard so no doubt it will for youy
Yeah working correct that is v3.9.2