Skip to content

Add Core NuGet package authoring requirements#3474

Merged
RubenCerna2079 merged 5 commits intomainfrom
dev/rubencerna/add-auth-nuget-package
Apr 29, 2026
Merged

Add Core NuGet package authoring requirements#3474
RubenCerna2079 merged 5 commits intomainfrom
dev/rubencerna/add-auth-nuget-package

Conversation

@RubenCerna2079
Copy link
Copy Markdown
Contributor

Why make this change?

What is this change?

How was this tested?

  • Integration Tests
  • Unit Tests

No tests are necessary for this change

Sample Request(s)

N/A

@RubenCerna2079 RubenCerna2079 added this to the April 2026 milestone Apr 27, 2026
@RubenCerna2079 RubenCerna2079 marked this pull request as ready for review April 27, 2026 21:25
Copilot AI review requested due to automatic review settings April 27, 2026 21:25
@RubenCerna2079 RubenCerna2079 linked an issue Apr 27, 2026 that may be closed by this pull request
@RubenCerna2079 RubenCerna2079 changed the title Add package auth Add NuGet package authoring requirements Apr 27, 2026
@RubenCerna2079 RubenCerna2079 changed the title Add NuGet package authoring requirements Add Core NuGet package authoring requirements Apr 27, 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 packaging metadata to support publishing Azure.DataApiBuilder.Core as a NuGet package and refreshes the CLI package metadata to mention MCP support, aligning with issue #3469.

Changes:

  • Add NuGet package metadata (id/title/authors/license/readme/icon/tags) and packable assets to Azure.DataApiBuilder.Core.csproj.
  • Update Cli.csproj package tags and description to include MCP endpoints.

Reviewed changes

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

File Description
src/Core/Azure.DataApiBuilder.Core.csproj Adds NuGet package authoring/publishing metadata and packs README/icon/license assets.
src/Cli/Cli.csproj Updates NuGet tags/description to advertise MCP endpoint support.

Comment thread src/Core/Azure.DataApiBuilder.Core.csproj Outdated
Comment thread src/Core/Azure.DataApiBuilder.Core.csproj
Comment thread src/Cli/Cli.csproj Outdated
Comment thread src/Core/Azure.DataApiBuilder.Core.csproj Outdated
Comment thread src/Core/Azure.DataApiBuilder.Core.csproj Outdated
Comment thread src/Core/Azure.DataApiBuilder.Core.csproj Outdated
Comment thread src/Core/Azure.DataApiBuilder.Core.csproj Outdated
Copy link
Copy Markdown
Collaborator

@Aniruddh25 Aniruddh25 left a comment

Choose a reason for hiding this comment

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

Needs different README, NOTICE files for this new package.

@github-project-automation github-project-automation Bot moved this from Todo to Review In Progress in Data API builder Apr 28, 2026
Comment thread nuget/nuget_core/README.md Outdated
Copy link
Copy Markdown
Collaborator

@Aniruddh25 Aniruddh25 left a comment

Choose a reason for hiding this comment

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

Once the changes to NOTICE.txt are done, you might need to include the right path in this PR as well.

@RubenCerna2079 RubenCerna2079 enabled auto-merge (squash) April 29, 2026 18:09
Comment thread src/Core/Azure.DataApiBuilder.Core.csproj Outdated
@RubenCerna2079 RubenCerna2079 merged commit 23ba6f7 into main Apr 29, 2026
12 checks passed
@RubenCerna2079 RubenCerna2079 deleted the dev/rubencerna/add-auth-nuget-package branch April 29, 2026 23:46
@github-project-automation github-project-automation Bot moved this from Review In Progress to Done in Data API builder Apr 29, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

Add Package Requirements to csproj

5 participants