-
-
Notifications
You must be signed in to change notification settings - Fork 7
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add loc
getters to Value Definition
#114
Add loc
getters to Value Definition
#114
Conversation
5386352
to
b698054
Compare
2b2814b
to
b4ef719
Compare
Thanks for opening this pull request, @nachiket87! For the |
432377e
to
414d12f
Compare
@marcoroth hey! I've used the kind value in the last commit. |
@nachiket87 awesome thank you! Is this ready to go from your perspective? |
loc
getters to Value Definition
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.
Thanks for working on this @nachiket87! 🙏🏼
Hey @marcoroth,
Based on our discussion, I'm trying to add the various Loc's as getters on ValueDefinition. Here is a draft for you to see what I'm doing. The
TypeLoc
getter seems to be challenging as thethis.node
value of the valuedefinition can have various different syntaxes and we would have to filter throughObjectExpression
type orLiteral
types.