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

Use one default completion strategy #889

Conversation

datho7561
Copy link

  • Remove 'suitable binding' completion
  • Fixes to package completionsome
    • Skip suggesting the default package
    • Use the same strategy whenever packages are suggested
    • Improve calculation of the prefix to make the matches more accurate
  • Potentially address dupliate type completions

Fixes #870

- Remove 'suitable binding' completion
- Fixes to package completionsome
  - Skip suggesting the default package
  - Use the same strategy whenever packages are suggested
  - Improve calculation of the prefix to make the matches more accurate
- Potentially address dupliate type completions

Fixes eclipse-jdt#870

Signed-off-by: David Thompson <[email protected]>
@datho7561 datho7561 force-pushed the dom-with-javac-one-default-completion branch from 21fd3e6 to 935c2c4 Compare October 15, 2024 12:49
@datho7561
Copy link
Author

Merging; this fixes many test cases.

@datho7561 datho7561 merged commit 4583e9c into eclipse-jdtls:dom-with-javac Oct 15, 2024
4 of 5 checks passed
@datho7561 datho7561 deleted the dom-with-javac-one-default-completion branch October 15, 2024 14:08
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.

Duplicate completion entries after a statement
1 participant