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

Add support for intellisense in experimental debugger #1076

Closed
DonJayamanne opened this Issue Mar 14, 2018 · 3 comments

Comments

Projects
None yet
3 participants
@DonJayamanne

DonJayamanne commented Mar 14, 2018

  • Enable tests in capabilities.test.ts
@brettcannon

This comment has been minimized.

Show comment
Hide comment
@brettcannon

brettcannon Apr 18, 2018

Member

Support how?

Member

brettcannon commented Apr 18, 2018

Support how?

@DonJayamanne

This comment has been minimized.

Show comment
Hide comment
@DonJayamanne

DonJayamanne Apr 18, 2018

Debugger protocol supports this today. Its up to the debugger adapter to return the completion items.
We could either perform something as dirty as dir on the objects (I believe PTVSD does this) or use Jedi (other IDEs seem to use jedi in the debugger to provide intellisense).

DonJayamanne commented Apr 18, 2018

Debugger protocol supports this today. Its up to the debugger adapter to return the completion items.
We could either perform something as dirty as dir on the objects (I believe PTVSD does this) or use Jedi (other IDEs seem to use jedi in the debugger to provide intellisense).

@DonJayamanne

This comment has been minimized.

Show comment
Hide comment
@DonJayamanne

DonJayamanne Sep 27, 2018

@brettcannon Shouldn't this be back in the current sprint.
It never got done in the last sprint, i'd have thought such issues would automatically migrate to the following sprint.

Please feel free to remove from sprint if you think otherwise.

DonJayamanne commented Sep 27, 2018

@brettcannon Shouldn't this be back in the current sprint.
It never got done in the last sprint, i'd have thought such issues would automatically migrate to the following sprint.

Please feel free to remove from sprint if you think otherwise.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment