marijn/getdocs-ts
1
0
Fork 0
Code Issues Pull requests Releases Activity

fix: void and infer type #1

Closed
Saul-Mirone wants to merge 3 commits from main into main
pull from: main
merge into: marijn:main
Conversation 3 Commits 3 Files changed 1 +2 -2
Saul-Mirone commented 2022-11-11 04:34:47 +01:00 (Migrated from github.com)
Copy link

This PR fix two cases which will cause error now:

// 1. void type will cause error
const fn = (): void => {}

// 2. infer type will cause error
type InferParams<T> = T extends Array<infer U> ? U : never;
This PR fix two cases which will cause error now: ```ts // 1. void type will cause error const fn = (): void => {} // 2. infer type will cause error type InferParams<T> = T extends Array<infer U> ? U : never; ```
marijnh commented 2022-11-11 09:26:19 +01:00 (Migrated from github.com)
Copy link

Merged the void handling in 2af2e4f, did something a bit different for inferred parameters in 6b3ecffc0a

Merged the void handling in 2af2e4f, did something a bit different for inferred parameters in 6b3ecffc0a19ef87588a4
Saul-Mirone commented 2022-11-11 10:02:49 +01:00 (Migrated from github.com)
Copy link

👍 Cool, do you have a release plan for them?

> 👍 Cool, do you have a release plan for them?
marijnh commented 2022-11-11 10:09:30 +01:00 (Migrated from github.com)
Copy link

They have already been tagged as 0.1.4

They have already been tagged as 0.1.4
🚀 1

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
marijn/getdocs-ts!1
No description provided.