-
-
Notifications
You must be signed in to change notification settings - Fork 6
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
fix: handle const Type Parameters #519
base: main
Are you sure you want to change the base?
Conversation
372beb6
to
0f27bef
Compare
0f27bef
to
4622196
Compare
typings/typescript.d.ts
Outdated
|
||
interface Symbol { | ||
readonly links?: { | ||
readonly checkFlags: ModifierFlags; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah I can't find any other way to do this but don't love relying on TS internals 😬 ... hmm. Do you think you could file an issue on TypeScript @RebeccaStevens asking to make this or an equivalent public?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
4622196
to
15445fb
Compare
@JoshuaKGoldberg I just cleaned up this PR a bunch and made it a bit more robust. |
15445fb
to
0359162
Compare
0359162
to
5b0a2b6
Compare
5b0a2b6
to
74a5952
Compare
PR Checklist
symbolHasReadonlyDeclaration
doesn't supportconst
type parameters #518status: accepting prs
Overview
Initial attempt at resolving the issue.
I've got no idea if this is the correct approach or not.
There might be some cases which this PR doesn't currently handle.