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

feat: rest element should contain information about type annotation #108

Merged
merged 2 commits into from
Jan 10, 2019

Conversation

armano2
Copy link
Contributor

@armano2 armano2 commented Jan 9, 2019

This PR changes typeAnnotation location in AST,

after this change typeAnnotation will be correctly assigned to RestElement node.


i think this is a little more logical, type annotation describe ...foo not foo in this case

example:

function foo (...bar: baz[]) {}

JamesHenry
JamesHenry previously approved these changes Jan 10, 2019
@armano2
Copy link
Contributor Author

armano2 commented Jan 10, 2019

can you merge #110 before this one?

@JamesHenry
Copy link
Owner

Done 👍

BREAKING CHANGE: This changes the AST for RestElement node
@JamesHenry JamesHenry merged commit acf10d5 into JamesHenry:master Jan 10, 2019
@armano2 armano2 deleted the typed-rest-element branch January 10, 2019 23:27
JamesHenry added a commit that referenced this pull request Jan 10, 2019
BREAKING CHANGE: This changes the AST for RestElement node
@JamesHenry
Copy link
Owner

🎉 This PR is included in version 14.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