Skip to content

Conversation

kermorgant
Copy link
Contributor

@kermorgant kermorgant commented Jul 20, 2018

$ symbol has to be present in the prefix in order to match phpactor's
suggestions. Thus, we need a specific function to extract this prefix.

see issue 28

$ symbol has to be present in the prefix in order to match phpactor's
suggestions. Thus, we need a specific function to extract this prefix.
@zonuexe zonuexe merged commit b5ec5f1 into emacs-php:master Jul 20, 2018
@zonuexe
Copy link
Member

zonuexe commented Jul 20, 2018

@kermorgant Thanks!

@kermorgant
Copy link
Contributor Author

I just came back to my computer to add a comment about lack of testing about my PRs.

Well, it's still in alpha so I guess it's not that important :-)

kermorgant added a commit to kermorgant/phpactor.el that referenced this pull request Jul 20, 2018
pr#29 added avriable completion but completion of object methods
stopped working. This should now be fixed.
kermorgant added a commit to kermorgant/phpactor.el that referenced this pull request Jul 26, 2018
pr#29 added avriable completion but completion of object methods
stopped working. This should now be fixed.
zonuexe added a commit that referenced this pull request Jul 26, 2018
Fix regression from pr #29 causing completion on object to fail
@zonuexe zonuexe mentioned this pull request Aug 8, 2018
@kermorgant kermorgant deleted the variable-completion branch September 16, 2018 11:56
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