-
Notifications
You must be signed in to change notification settings - Fork 13
Conversation
i feel like i should split this PR cus alignment changes are getting big |
7507043
to
89f2fea
Compare
@armano2 Got some conflicts from merging your other PR. Please could you also be sure to add:
As the body of your commit That will trigger a major version to be released and for the release notes to contain info about the breaking change. @ikatyang @j-f1 Please could you take a look at this, this will undoubtedly affect prettier |
@JamesHenry thats why i wanted to split this to few commits, and looks like i'm going to do so. |
BREAKING CHANGE: This changes the AST
@JamesHenry updated |
BREAKING CHANGE: This changes the AST
@JamesHenry is there something i have to change in this PR? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice one, thanks! Been looking forward to this change for like a year 😂
🎉 This PR is included in version 7.0.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |
This PR adds
declare
to nodes:changes:
DeclareFunction
toTSDeclareFunction
changes:
VariableDeclarator[kind='type']
toTSTypeAliasDeclaration
makes
abstract
optional in interfacesfixes: #17 eslint/typescript-eslint-parser#414