You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
This might be a no-fix (but I'm not closing this because this is a good issue) for the following reasons:
Since solid-refresh doesn't do cross-file analysis, there is no way for it to know that ResultKind is a const enum (or even just enum)
TypeScript seems to ignore this fact too, but it does raise a compiler warning.
Take note that the transformation happens before TypeScript gets to compile anything.
Currently, there is no fix for this issue (until we do cross-file analysis, but I don't see it happening any time), so I would recommend using some alternatives (either use enum, which is bad, as I am a hater of it myself or you use string unions)
Activity
lxsmnsyc commentedon Jan 13, 2025
This might be a no-fix (but I'm not closing this because this is a good issue) for the following reasons:
ResultKindis aconst enum(or even justenum)Take note that the transformation happens before TypeScript gets to compile anything.
Currently, there is no fix for this issue (until we do cross-file analysis, but I don't see it happening any time), so I would recommend using some alternatives (either use
enum, which is bad, as I am a hater of it myself or you use string unions)tmm1 commentedon Jan 13, 2025
Makes sense, I can work-around for now. Thanks!