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

Fix of signature help feature when typing '(' symbol #11696

Merged
merged 2 commits into from Oct 25, 2018
Merged

Conversation

dkuleshov
Copy link

Related issue: #10699

The cursor position that client was sending to a language server was incorrect and the signature help symbol filtering algorithm was not accurate enough.

Dmytro Kulieshov added 2 commits October 22, 2018 16:17
Signed-off-by: Dmytro Kulieshov <dkuliesh@redhat.com>
Signed-off-by: Dmytro Kulieshov <dkuliesh@redhat.com>
@dkuleshov
Copy link
Author

ci-test

@riuvshin
Copy link
Contributor

Results of automated E2E tests of Eclipse Che Multiuser on OCP:
Build details
Test report
docker image: eclipseche/che-server:11696
https://github.com/orgs/eclipse/teams/eclipse-che-qa please check this report.

@benoitf benoitf added kind/bug Outline of a bug - must adhere to the bug report template. status/code-review This issue has a pull request posted for it and is awaiting code review completion by the community. labels Oct 22, 2018
@dkuleshov
Copy link
Author

ci-test

@riuvshin
Copy link
Contributor

Results of automated E2E tests of Eclipse Che Multiuser on OCP:
Build details
Test report
docker image: eclipseche/che-server:11696
https://github.com/orgs/eclipse/teams/eclipse-che-qa please check this report.

@musienko-maxim
Copy link
Contributor

ci-test

@riuvshin
Copy link
Contributor

Results of automated E2E tests of Eclipse Che Multiuser on OCP:
Build details
Test report
docker image: eclipseche/che-server:11696
https://github.com/orgs/eclipse/teams/eclipse-che-qa please check this report.

@dkuleshov dkuleshov merged commit d7a678c into master Oct 25, 2018
@dkuleshov dkuleshov deleted the che#10699 branch October 25, 2018 09:01
@benoitf benoitf removed the status/code-review This issue has a pull request posted for it and is awaiting code review completion by the community. label Oct 25, 2018
@benoitf benoitf added this to the 6.14.0 milestone Oct 25, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Outline of a bug - must adhere to the bug report template.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants