Cleanup the preview coverage functionality
See original GitHub issueDescription
Minor cleanup of the preview coverage feature to remove some tech debt (code structure, content security cleanup, etc). This came to my attention via an error that is appearing in the test builds.
ryanluker.vscode-coverage-gutters created a webview without a content security policy: https://aka.ms/vscode-webview-missing-csp
Work
- add a content security policy to preview html (https://code.visualstudio.com/api/extension-guides/webview#content-security-policy)
- move webview creation logic out of the gutters file and into it’s own module (pretty odd having this functionality just living in the
gutters.ts
as that area is meant to be a very thin uncomplicated area of code).
Issue Analytics
- State:
- Created 4 years ago
- Comments:7 (7 by maintainers)
Top Results From Across the Web
Clearing coverage highlighting in Eclipse - java - Stack Overflow
After running coverage reports in Eclipse (using cobertura or an EMMA plugin), my source code files get highlighted in green, red and yellow...
Read more >Overview of automated investigations - Microsoft Learn
Review and approve (or reject) remediation actions following an automated investigation. Investigate Microsoft Defender for Endpoint alerts. Use ...
Read more >Model Preview | DAI Docs - Eggplant | Documentation
Model Preview displays a read-only version of the model for review and testing. ... Type: either Start, Normal, Cleanup or Exception.
Read more >New features available with macOS Ventura. - Apple
Preview the latest features, enhancements, app updates, and more in macOS Ventura for Mac.
Read more >istanbuljs/nyc: the Istanbul command line interface - GitHub
skip-full, Don't show files with 100% statement, branch, and function coverage, Boolean, false. temp-dir, Directory to output raw coverage information to ...
Read more >
Top Related Medium Post
No results found
Top Related StackOverflow Question
No results found
Troubleshoot Live Code
Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start Free
Top Related Reddit Thread
No results found
Top Related Hackernoon Post
No results found
Top Related Tweet
No results found
Top Related Dev.to Post
No results found
Top Related Hashnode Post
No results found
ahh you are correct, this might not have been updated in a long while. The line numbers definitely don’t match up anymore… good catch!
@Kaytbode Sure thing! any tickets that don’t have an assigned owner can be considered up for grabs. We would maybe consider this for the 2.7.0 release if we end up finishing it before I get around to finishing the rest of the items. https://github.com/ryanluker/vscode-coverage-gutters/milestone/32