Skip to content

Fix package metadata to meet compliance requirements.#3999

Merged
paulmedynski merged 1 commit into
mainfrom
dev/mdaigle/fix-metadata
Mar 5, 2026
Merged

Fix package metadata to meet compliance requirements.#3999
paulmedynski merged 1 commit into
mainfrom
dev/mdaigle/fix-metadata

Conversation

@mdaigle
Copy link
Copy Markdown
Contributor

@mdaigle mdaigle commented Mar 4, 2026

This pull request focuses on improving package metadata for several projects and updating development tooling configuration. The most significant changes are enhancements to NuGet package information across multiple csproj files, ensuring better discoverability and compliance, as well as an update to the VS Code MCP configuration for a new development workflow.

NuGet package metadata improvements:

  • Added Title, Copyright, PackageTags, RepositoryType, PackageProjectUrl, and PackageRequireLicenseAcceptance fields to the Microsoft.Data.SqlClient.Extensions.Abstractions, Azure, and Logging project files, and updated Authors to "Microsoft" for consistency and compliance. [1] [2] [3]
  • Updated the Authors field in Microsoft.SqlServer.Server.csproj from "Microsoft Corporation" to "Microsoft" for consistency with other packages.

Development tooling configuration:

  • Added a new ado-devdiv entry to .vscode/mcp.json to support running the Azure DevOps MCP DevDiv workflow using npx, streamlining local development and testing.

Copilot AI review requested due to automatic review settings March 4, 2026 22:22
@mdaigle mdaigle marked this pull request as ready for review March 4, 2026 22:22
@mdaigle mdaigle requested a review from a team as a code owner March 4, 2026 22:22
@github-project-automation github-project-automation Bot moved this to To triage in SqlClient Board Mar 4, 2026
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates NuGet package metadata across several projects to improve compliance/discoverability consistency, and extends the repo’s VS Code MCP configuration to support an additional Azure DevOps workflow.

Changes:

  • Standardizes NuGet package metadata for the Extensions projects (Title/Authors/Tags/repo/project URL/license acceptance).
  • Aligns Microsoft.SqlServer.Server package Authors with the updated convention (Microsoft).
  • Adds a new .vscode/mcp.json MCP server entry (ado-devdiv) that runs @azure-devops/mcp devdiv via npx.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated no comments.

Show a summary per file
File Description
src/Microsoft.SqlServer.Server/Microsoft.SqlServer.Server.csproj Changes Authors to “Microsoft” for package metadata consistency.
src/Microsoft.Data.SqlClient.Extensions/Logging/src/Logging.csproj Adds missing NuGet metadata fields (Title/Tags/RepoType/ProjectUrl/LicenseAcceptance) and standardizes Authors.
src/Microsoft.Data.SqlClient.Extensions/Azure/src/Azure.csproj Adds missing NuGet metadata fields and standardizes Authors.
src/Microsoft.Data.SqlClient.Extensions/Abstractions/src/Abstractions.csproj Adds missing NuGet metadata fields and standardizes Authors.
.vscode/mcp.json Adds ado-devdiv MCP server configuration using npx.

@codecov
Copy link
Copy Markdown

codecov Bot commented Mar 4, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 67.10%. Comparing base (3303d80) to head (7b4df61).
⚠️ Report is 1 commits behind head on main.

❗ There is a different number of reports uploaded between BASE (3303d80) and HEAD (7b4df61). Click for more details.

HEAD has 2 uploads less than BASE
Flag BASE (3303d80) HEAD (7b4df61)
CI-SqlClient 2 0
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3999      +/-   ##
==========================================
- Coverage   74.38%   67.10%   -7.29%     
==========================================
  Files         287      282       -5     
  Lines       43982    67171   +23189     
==========================================
+ Hits        32717    45075   +12358     
- Misses      11265    22096   +10831     
Flag Coverage Δ
CI-SqlClient ?
PR-SqlClient-Project 67.10% <ø> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copy link
Copy Markdown
Contributor

@paulmedynski paulmedynski left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good. Have you been able to test pushing the resulting packages to nuget.org to see if they pass validation?

@paulmedynski paulmedynski merged commit 2fc206c into main Mar 5, 2026
300 checks passed
@paulmedynski paulmedynski deleted the dev/mdaigle/fix-metadata branch March 5, 2026 11:09
@github-project-automation github-project-automation Bot moved this from To triage to Done in SqlClient Board Mar 5, 2026
@mdaigle mdaigle added this to the 7.0.0-preview4 milestone Mar 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

4 participants