Skip to content

fix: handle const Type Parameters#519

Merged
JoshuaKGoldberg merged 3 commits into
mainfrom
issue-518-initial
Oct 22, 2024
Merged

fix: handle const Type Parameters#519
JoshuaKGoldberg merged 3 commits into
mainfrom
issue-518-initial

Conversation

@RebeccaStevens

@RebeccaStevens RebeccaStevens commented May 22, 2024

Copy link
Copy Markdown
Collaborator

PR Checklist

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.

@RebeccaStevens RebeccaStevens force-pushed the issue-518-initial branch 3 times, most recently from 372beb6 to 0f27bef Compare May 23, 2024 02:33
@RebeccaStevens RebeccaStevens marked this pull request as draft May 23, 2024 02:34
Comment thread typings/typescript.d.ts Outdated
@RebeccaStevens

RebeccaStevens commented May 25, 2024

Copy link
Copy Markdown
Collaborator Author

@JoshuaKGoldberg I just cleaned up this PR a bunch and made it a bit more robust.

Comment thread typings/typescript.d.ts

@JoshuaKGoldberg JoshuaKGoldberg left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry for the delay, this looks good to me. Thanks!

@JoshuaKGoldberg

Copy link
Copy Markdown
Owner

Builds are very broken on main right now. I filed #530 to use the latest pnpm. Anyway, this has been ready for a while, and I just lost track of it 🤦

@JoshuaKGoldberg JoshuaKGoldberg merged commit 41cd21d into main Oct 22, 2024
@JoshuaKGoldberg JoshuaKGoldberg deleted the issue-518-initial branch October 22, 2024 16:56
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

Successfully merging this pull request may close these issues.

🐛 Bug: symbolHasReadonlyDeclaration doesn't support const type parameters

2 participants