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

Retire debug codelens #950

Merged
merged 3 commits into from
Nov 21, 2022

Conversation

connectdotz
Copy link
Collaborator

debug has long been available in the TestExplorer context menu. In an effort to reduce redundant features and simplify the code base, we will remove the DebugCodeLens and related settings in v5.

@coveralls
Copy link

Pull Request Test Coverage Report for Build 3510153515

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.09%) to 96.953%

Totals Coverage Status
Change from base Build 3510068899: 0.09%
Covered Lines: 3200
Relevant Lines: 3245

💛 - Coveralls

@connectdotz connectdotz merged commit 9942dee into jest-community:master Nov 21, 2022
@connectdotz connectdotz deleted the retire-debug-codelens branch November 21, 2022 03:30
@markcarroll
Copy link

I really do not like this change at all. Having to go to the other side of the screen (on a 32" 4k monitor) to find the debug button in the explorer is a real pain compared to just clicking above the test you are debugging. @connectdotz can you either revert this change or add it as a config option please?

@nausaf
Copy link

nausaf commented Dec 30, 2022

@markcarroll VS Code built-in Testing provides a replacement: in your USer Settings file (setttings.json), put this line in:

"testing.defaultGutterClickAction": "contextMenu"

You should now be able to access Debug test etc. from the Play button in the gutter:

image

legend1202 pushed a commit to legend1202/vscode-jest that referenced this pull request Jun 18, 2023
* removed debugCodeLens and settings

* clean up more snapshot and debug codeLens settings
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.

4 participants