Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

SqlPacakge command generation feature implementation for Publish Dialog#20807

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Draft
ssreerama wants to merge7 commits intomain
base:main
Choose a base branch
Loading
fromsai/sqlPkgCmdGen

Conversation

@ssreerama
Copy link
Contributor

Pull Request Template – vscode-mssql

Description

TODO...

Code Changes Checklist

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

Reviewers:Please read our reviewer guidelines

SS:
image

@github-actions
Copy link

PR Changes

CategoryTarget BranchPR BranchDifference
vscode-mssql VSIX5283 KB5285 KB⚪ 2 KB ( 0% )
sql-database-projects VSIX5668 KB5668 KB⚪ 0 KB ( 0% )

@codecov-commenter
Copy link

codecov-commenter commentedDec 14, 2025•
edited
Loading

Codecov Report

❌ Patch coverage is55.00000% with18 lines in your changes missing coverage. Please review.
✅ Project coverage is 65.88%. Comparing base (eb6daf1) to head (75fbc8f).

Files with missing linesPatch %Lines
.../publishProject/publishProjectWebViewController.ts52.00%12 Missing⚠️
...ons/mssql/src/languageservice/sqlPackageService.ts40.00%3 Missing⚠️
extensions/mssql/src/constants/locConstants.ts0.00%2 Missing⚠️
extensions/mssql/src/controllers/mainController.ts50.00%1 Missing⚠️

❌ Your patch status has failed because the patch coverage (55.00%) is below the target coverage (70.00%). You can increase the patch coverage or adjust thetarget coverage.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@##             main   #20807      +/-   ##==========================================- Coverage   65.90%   65.88%   -0.03%==========================================  Files         210      212       +2       Lines       19989    20028      +39       Branches     2596     2604       +8     ==========================================+ Hits        13174    13195      +21- Misses       6718     6736      +18  Partials       97       97
Files with missing linesCoverage Δ
...models/contracts/sqlPackage/sqlPackageContracts.ts100.00% <100.00%> (ø)
...nsions/mssql/src/sharedInterfaces/publishDialog.ts100.00% <100.00%> (ø)
extensions/mssql/src/controllers/mainController.ts17.25% <50.00%> (+0.05%)⬆️
extensions/mssql/src/constants/locConstants.ts86.87% <0.00%> (-0.18%)⬇️
...ons/mssql/src/languageservice/sqlPackageService.ts40.00% <40.00%> (ø)
.../publishProject/publishProjectWebViewController.ts64.82% <52.00%> (-0.96%)⬇️
🚀 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.

Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

@kburtramkburtramAwaiting requested review from kburtramkburtram will be requested when the pull request is marked ready for reviewkburtram is a code owner

@aasimkhan30aasimkhan30Awaiting requested review from aasimkhan30aasimkhan30 will be requested when the pull request is marked ready for reviewaasimkhan30 is a code owner

@caohaicaohaiAwaiting requested review from caohaicaohai will be requested when the pull request is marked ready for reviewcaohai is a code owner

@laurenastrid1laurenastrid1Awaiting requested review from laurenastrid1laurenastrid1 will be requested when the pull request is marked ready for reviewlaurenastrid1 is a code owner

@lewis-sanchezlewis-sanchezAwaiting requested review from lewis-sanchezlewis-sanchez will be requested when the pull request is marked ready for reviewlewis-sanchez is a code owner

@BenjinBenjinAwaiting requested review from BenjinBenjin will be requested when the pull request is marked ready for reviewBenjin is a code owner

@allancascanteallancascanteAwaiting requested review from allancascanteallancascante will be requested when the pull request is marked ready for reviewallancascante is a code owner

@manujoseph85manujoseph85Awaiting requested review from manujoseph85manujoseph85 will be requested when the pull request is marked ready for reviewmanujoseph85 is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@ssreerama@codecov-commenter
[8]ページ先頭
©2009-2025 Movatter.jp