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

Add support for variable paging #68

Closed
wants to merge 3 commits into from
Closed

Conversation

felixfbecker
Copy link
Contributor

namedVariables is correctly returned as being larger than the actual number of returned variables, but VS Code refuses to do any follow-up requests for paging or show any UI. I guess it is not implemented yet?

@codecov-io
Copy link

codecov-io commented Oct 12, 2016

Current coverage is 69.43% (diff: 100%)

Merging #68 into master will increase coverage by 1.17%

@@             master        #68   diff @@
==========================================
  Files             5          5          
  Lines           816        818     +2   
  Methods           5          6     +1   
  Messages          0          0          
  Branches        134        135     +1   
==========================================
+ Hits            557        568    +11   
+ Misses          258        249     -9   
  Partials          1          1          

Powered by Codecov. Last update 02eae03...5c07981

@felixfbecker
Copy link
Contributor Author

Depends on microsoft/vscode#13878

@zobo
Copy link
Contributor

zobo commented Mar 22, 2023

Will close this as it was implemented in #805 with indexedVariables.

@zobo zobo closed this Mar 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants