Skip to content

Fix package paths.#4436

Merged
mdaigle merged 1 commit into
mainfrom
dev/mdaigle/fix-package-release-path
Jul 9, 2026
Merged

Fix package paths.#4436
mdaigle merged 1 commit into
mainfrom
dev/mdaigle/fix-package-release-path

Conversation

@mdaigle

@mdaigle mdaigle commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

Only sqlclient sets a custom 'Package-Release' artifact path. Others place artifacts under 'Release'.

Validating with a nonofficial run: https://sqlclientdrivers.visualstudio.com/ADO.Net/_build/results?buildId=160021&view=results

Copilot AI review requested due to automatic review settings July 9, 2026 18:08
@github-project-automation github-project-automation Bot moved this to To triage in SqlClient Board Jul 9, 2026
@mdaigle mdaigle marked this pull request as ready for review July 9, 2026 18:08
@mdaigle mdaigle requested a review from a team as a code owner July 9, 2026 18:08

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 aligns OneBranch release-stage packagePath values with the actual package output layout, so only Microsoft.Data.SqlClient uses the Package-Release/ path while the other packages publish from Release/.

Changes:

  • Update release-stage publish steps for SqlServer.Server, Logging, Abstractions, Azure, and AKV Provider to use Release/<package>.<version>.nupkg.
  • Preserve SqlClient’s custom Package-Release/ path (consistent with its $(ReferenceType)-$(Configuration) output convention).

@mdaigle mdaigle merged commit 52ed94b into main Jul 9, 2026
12 of 13 checks passed
@mdaigle mdaigle deleted the dev/mdaigle/fix-package-release-path branch July 9, 2026 20:19
@github-project-automation github-project-automation Bot moved this from To triage to Done in SqlClient Board Jul 9, 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.

5 participants