Skip to content

Merge | Pack Mds Target, Wire PR/CI Pipelines to Build2.proj#4033

Merged
benrr101 merged 27 commits intomainfrom
dev/russellben/common/pack2
Mar 19, 2026
Merged

Merge | Pack Mds Target, Wire PR/CI Pipelines to Build2.proj#4033
benrr101 merged 27 commits intomainfrom
dev/russellben/common/pack2

Conversation

@benrr101
Copy link
Contributor

@benrr101 benrr101 commented Mar 11, 2026

Description

This PR adds a PackMds target to Build2.proj that conveniently packages the MDS projects into a NuGet package. This PR also reworks the CI/PR pipelines to use the build2.proj targets to build, test, and package the MDS project.

Commentary has been added in-line.

Testing

Ran PR validation while still in draft mode and verifies that it generates good output. There are a lot of things to go through to verify that everything is behaving correctly, but it seems good.

@benrr101 benrr101 requested a review from a team as a code owner March 11, 2026 18:25
Copilot AI review requested due to automatic review settings March 11, 2026 18:25
@github-project-automation github-project-automation bot moved this to To triage in SqlClient Board Mar 11, 2026
@benrr101 benrr101 marked this pull request as draft March 11, 2026 18:25

This comment was marked as outdated.

Copilot AI review requested due to automatic review settings March 11, 2026 21:18

This comment was marked as outdated.

Copilot AI review requested due to automatic review settings March 11, 2026 22:32

This comment was marked as outdated.

Copilot AI review requested due to automatic review settings March 12, 2026 04:30
@github-project-automation github-project-automation bot moved this from In review to In progress in SqlClient Board Mar 12, 2026
@paulmedynski paulmedynski added this to the 7.0.0 milestone Mar 13, 2026
@paulmedynski paulmedynski self-assigned this Mar 13, 2026
mdaigle
mdaigle previously approved these changes Mar 13, 2026
# Conflicts:
#	build2.proj
#	eng/pipelines/onebranch/steps/code-analyze-step.yml
@benrr101 benrr101 requested a review from paulmedynski March 13, 2026 21:14
@benrr101 benrr101 dismissed paulmedynski’s stale review March 13, 2026 22:34

He's on vacation and said it's ok

@cheenamalhotra cheenamalhotra modified the milestones: 7.0.0, 7.1.0-preview1 Mar 16, 2026
Copilot AI review requested due to automatic review settings March 16, 2026 17:27
Copy link
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

Copilot reviewed 21 out of 21 changed files in this pull request and generated 6 comments.

@benrr101 benrr101 changed the title Merge | Pack Mds Target, Wire Up Pipelines to Build2.proj Merge | Pack Mds Target, Wire PR/CI Pipelines to Build2.proj Mar 16, 2026
Copilot AI review requested due to automatic review settings March 18, 2026 18:10
Copy link
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

Copilot reviewed 19 out of 19 changed files in this pull request and generated 4 comments.

@benrr101 benrr101 merged commit 07a9280 into main Mar 19, 2026
299 checks passed
@benrr101 benrr101 deleted the dev/russellben/common/pack2 branch March 19, 2026 21:27
@github-project-automation github-project-automation bot moved this from In progress to Done in SqlClient Board Mar 19, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Common Project 🚮 Things that relate to the common project project

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

5 participants