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

Fix: visitor-keys for TSAbstractClassProperty #560

Merged
merged 2 commits into from
Nov 28, 2018

Conversation

armano2
Copy link
Contributor

@armano2 armano2 commented Nov 21, 2018

This PR fixes order of visiting in TSAbstractClassProperty and adds decoratos

decoratos are already in scope analyze handled by ClassProperty

@armano2 armano2 mentioned this pull request Nov 21, 2018
14 tasks
Copy link
Member

@platinumazure platinumazure left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks!

its already supported in ClassProperty
@armano2 armano2 changed the title Fix: order of visiting in TSAbstractClassProperty Fix: order of visiting in TSAbstractClassProperty and add missing decorators Nov 21, 2018
@armano2 armano2 changed the title Fix: order of visiting in TSAbstractClassProperty and add missing decorators Fix: order of visiting in TSAbstractClassProperty Nov 21, 2018
Copy link
Member

@platinumazure platinumazure left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks!

@armano2 armano2 changed the title Fix: order of visiting in TSAbstractClassProperty Fix: visitor-keys for TSAbstractClassProperty Nov 21, 2018
Copy link
Member

@mysticatea mysticatea left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thank you!

@armano2
Copy link
Contributor Author

armano2 commented Nov 28, 2018

@platinumazure can i have some info when those fixes can be merged?

@platinumazure
Copy link
Member

@armano2 Apologies, I guess we haven't coalesced on if any of the maintainers currently working on this project will act as "lead maintainer" or "PR merger". As you may know, we're still somewhat in transition since the previous lead maintainer left the project.

I can merge a few of these since we've had more than just my review (I don't consider myself a TypeScript expert or anywhere near, so I don't want to merge anything on my own).

Thanks very much for your patience as I ramp up on this project a little more and as the ESLint team figures out the best way to move forward!

@platinumazure platinumazure merged commit c19e479 into eslint:master Nov 28, 2018
@platinumazure
Copy link
Member

@armano2 I've merged this one, thanks for contributing!

@armano2 armano2 deleted the patch-3 branch November 28, 2018 21:30
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.

3 participants