Skip to content
This repository was archived by the owner on Dec 15, 2022. It is now read-only.

Added support for properties #146

Closed
wants to merge 2 commits into from
Closed

Added support for properties #146

wants to merge 2 commits into from

Conversation

TechnoBerry
Copy link
Contributor

Issues #37 , #97 , #137

@@ -431,6 +431,10 @@
'name': 'support.constant.dom.js'
}
{
'match': '(?<=(?:[a-zA-Z_$0-9]\\.))([a-zA-Z_$][a-zA-Z_$0-9]*)'
Copy link
Contributor

Choose a reason for hiding this comment

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

This seems a bit out-of-place since the surrounding capture rules all start with support. Is there by any chance a better place to put it?

@winstliu
Copy link
Contributor

winstliu commented Jun 9, 2015

This no longer merges cleanly - could you please fix the merge conflicts? Thanks.

@pchaigno
Copy link
Contributor

pchaigno commented Sep 6, 2015

@AMalininHere Did you get a chance to look into this?

@pchaigno
Copy link
Contributor

@AMalininHere Could you add a test to make sure that stuff.method() isn't recognized as a property?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants