Skip to content

Conversation

ssreerama
Copy link
Contributor

Pull Request Template – vscode-mssql

Description

This PR fixes the cred scan issue identified in latest build with the Publish dialog related PR merge.

Code Changes Checklist

  • New or updated unit tests added
  • All existing tests pass (npm run test)
  • Code follows contributing guidelines
  • Telemetry/logging updated if relevant
  • No regressions or UX breakage

Reviewers: Please read our reviewer guidelines

Copy link

PR Changes

Category Target Branch PR Branch Difference
Code Coverage 56.56% 56.56% ⚪ 0.00%
VSIX Size 4867 KB 4867 KB ⚪ 0 KB ( 0% )
Webview Bundle Size 4012 KB 4016 KB ⚪ 4 KB ( 0% )

@codecov-commenter
Copy link

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 54.54%. Comparing base (607b38b) to head (e4e8ba8).

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main   #20329   +/-   ##
=======================================
  Coverage   54.54%   54.54%           
=======================================
  Files         200      200           
  Lines       17338    17338           
  Branches     1151     1151           
=======================================
  Hits         9457     9457           
  Misses       7881     7881           
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@ssreerama ssreerama merged commit b8d588b into main Oct 21, 2025
3 checks passed
@ssreerama ssreerama deleted the sai/credScanIssueFix branch October 21, 2025 15:26
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.

3 participants