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

Rename ExperimentalSpreadProperty #428

Closed
duailibe opened this issue Jan 9, 2018 · 1 comment
Closed

Rename ExperimentalSpreadProperty #428

duailibe opened this issue Jan 9, 2018 · 1 comment

Comments

@duailibe
Copy link
Contributor

duailibe commented Jan 9, 2018

What version of TypeScript are you using?

2.6.2

What version of typescript-eslint-parser are you using?

master

What code were you trying to parse?

const foo = { ...bar };

What did you expect to happen?

An SpreadElement or SpreadProperty should be in the AST, not ExperimentalSpreadProperty, since they are not considered experimental anymore (microsoft/TypeScript@39b9163)?

I'm not exactly sure how this should go but would make things a lot easier to maintain in prettier (for example prettier/prettier#3681) :-)

I can help with this, just need a direction (cc @JamesHenry)

@JamesHenry
Copy link
Member

Thanks @duailibe! As now discussed on the PR, currently ESLint's policy dictates that because this is only stage 3, it is still experimental.

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

No branches or pull requests

2 participants