Skip to content
This repository was archived by the owner on Jan 14, 2019. It is now read-only.

feat: update TSFunctionType and TSConstructorType nodes #91

Merged
merged 6 commits into from
Jan 6, 2019

Conversation

armano2
Copy link
Contributor

@armano2 armano2 commented Jan 5, 2019

This PR fixes range of TypeAnnotation in function type:
before: > void
after: => void

@armano2
Copy link
Contributor Author

armano2 commented Jan 5, 2019

I'm not shure if we should change name of paramerters here to as we did for other function like nodes, typeAnnotation and parameters to returnType and params

@JamesHenry
Copy link
Owner

Yeah, let's do that for now please

@armano2
Copy link
Contributor Author

armano2 commented Jan 5, 2019

should we also do this for other function like types?

like: TSConstructorType

type Constructor<T> = new (...args: any[]) => T;

@armano2
Copy link
Contributor Author

armano2 commented Jan 5, 2019

ohh, right this issue is going to be present in TSConstructorType to xd

@armano2
Copy link
Contributor Author

armano2 commented Jan 5, 2019

@JamesHenry can you review it once more? i fixed also issue with TSConstructorType

@armano2 armano2 changed the title fix: correct start location of TypeAnnotation in FunctionType fix: correct location of TypeAnnotation in FunctionType and TSConstructorType Jan 6, 2019
@armano2 armano2 changed the title fix: correct location of TypeAnnotation in FunctionType and TSConstructorType feat: update TSFunctionType and TSConstructorType nodes Jan 6, 2019
@JamesHenry JamesHenry merged commit 7032f96 into JamesHenry:master Jan 6, 2019
@JamesHenry
Copy link
Owner

🎉 This PR is included in version 13.0.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants