Skip to content
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

Update estravere version #313

Closed
wants to merge 17 commits into from
Closed

Update estravere version #313

wants to merge 17 commits into from

Conversation

roiDaniela
Copy link

No description provided.

gadisn and others added 17 commits June 5, 2016 17:49
In regard to issue #206 ("Attach comment to an empty object"):

I have an initial modification of attachComments() and addComments() which supports comments in empty object/block.

I've called them inline comments, and they are attached to the node which represents the empty object/block (i.e. ObjectExpression/BlockStatement).
Indentation got broken with the introduction of inline comments
Add tests to Gadi's inline comments implemetation
Add tests to inline comments
Add support for inline comments
Add tests to Gadi's inline comments implemintation
Updated according to new traverese
Update try-block-line-comment.expected.js
@roiDaniela
Copy link
Author

roiDaniela commented Oct 9, 2016

The new 2 tests failed beacause this change - estools/estraverse#72
is still not added by the owners of estraverse.

@roiDaniela roiDaniela closed this Oct 9, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants