Skip to content

TypeScript <3.7 support by self-reference type rewrite#248

Merged
dcousens merged 3 commits intoJedWatson:masterfrom
coadan:236-extended-typescript-support
May 5, 2021
Merged

TypeScript <3.7 support by self-reference type rewrite#248
dcousens merged 3 commits intoJedWatson:masterfrom
coadan:236-extended-typescript-support

Conversation

@coadan
Copy link
Copy Markdown

@coadan coadan commented May 5, 2021

No description provided.

Copy link
Copy Markdown
Author

@coadan coadan left a comment

Choose a reason for hiding this comment

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

Just squash when merging to master, I realized that the type that I suggested in the issue was not entierly correct when reviewing the code after my morning coffee. The array type should be part of its own generic and the ArgumentArray in the Argument type should not have [] symbol after it.

Copy link
Copy Markdown
Collaborator

@dcousens dcousens left a comment

Choose a reason for hiding this comment

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

LGTM, waiting on CI and then I'll try this out locally!

Thanks @coadan 👍

@dcousens dcousens merged commit 2e87b50 into JedWatson:master May 5, 2021
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.

2 participants