Skip to content

Add support for signature help for new self, new static and new parent expressions#51

Merged
AJenbo merged 9 commits into
PHPantom-dev:mainfrom
RemcoSmitsDev:improve-signature-help
Mar 29, 2026
Merged

Add support for signature help for new self, new static and new parent expressions#51
AJenbo merged 9 commits into
PHPantom-dev:mainfrom
RemcoSmitsDev:improve-signature-help

Conversation

@RemcoSmitsDev
Copy link
Copy Markdown
Contributor

Hey, I'm new to the codebase so might choose the wrong paths to fix things, but this PR fixes the following 2 issues that are related to each other:

  1. Signature help for new self, new static and new parent expressions was not working.
  2. Auto completion for new self, new static and new parent expression arguments was not working.

Let me know if you want me to make any changes, so the code is maybe a bit more inline with the other part of the codebase.

Copy link
Copy Markdown
Collaborator

@AJenbo AJenbo left a comment

Choose a reason for hiding this comment

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

Thank, I think rebasing on main should fix the CI issues.

Comment thread src/completion/handler.rs Outdated
Comment thread tests/completion_callable_snippets.rs Outdated
Comment thread src/completion/handler.rs Outdated
@RemcoSmitsDev
Copy link
Copy Markdown
Contributor Author

RemcoSmitsDev commented Mar 29, 2026

Thanks for the review, I think I have addressed all the requests changes, let me know if you want me to make more changes!

@AJenbo AJenbo enabled auto-merge (squash) March 29, 2026 19:42
AJenbo
AJenbo previously approved these changes Mar 29, 2026
@AJenbo AJenbo merged commit 3557e6b into PHPantom-dev:main Mar 29, 2026
@AJenbo
Copy link
Copy Markdown
Collaborator

AJenbo commented Mar 29, 2026

Thanks you for you helping in make the project better

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