Skip to content

Update estravere version#313

Closed
roiDaniela wants to merge 17 commits intoestools:masterfrom
roiDaniela:master
Closed

Update estravere version#313
roiDaniela wants to merge 17 commits intoestools:masterfrom
roiDaniela:master

Conversation

@roiDaniela
Copy link
Copy Markdown

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
Copy Markdown
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