Skip to content
This repository has been archived by the owner on Jul 15, 2023. It is now read-only.

fixes #1383 debug sessions #2185

Closed
wants to merge 1 commit into from
Closed

fixes #1383 debug sessions #2185

wants to merge 1 commit into from

Conversation

oneslash
Copy link
Contributor

@oneslash oneslash commented Dec 13, 2018

Trying to fix #1383

@oneslash
Copy link
Contributor Author

@ramya-rao-a, not sure why tests are failing.

@oneslash
Copy link
Contributor Author

oneslash commented Jan 8, 2019

@ramya-rao-a, okay after rebase all tests are passing :)

@ramya-rao-a
Copy link
Contributor

@oneslash The codelens get rendered once for each function and dont get re-rendered until that line goes out of sight from the user.

So, this change will ensure that we dont render the code lens if there is an active session.

But once the code lens does appear, and the user clicks on it, this change will not make it go away, will it?

@oneslash
Copy link
Contributor Author

@ramya-rao-a, yes you are right, I will close this for now, I will be busy for one month.

@oneslash oneslash closed this Jan 15, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Clicking 'debug test' twice runs two instances of the debugger simultaneously
2 participants