Skip to content

Fix code for adding SQL project#383

Open
kattschan wants to merge 3 commits intomicrosoft:mainfrom
kattschan:patch-1
Open

Fix code for adding SQL project#383
kattschan wants to merge 3 commits intomicrosoft:mainfrom
kattschan:patch-1

Conversation

@kattschan
Copy link

Code in the current docs runs AddSqlProject on sql instead of builder. This results in an error when attempting to run code and is not in line with the documentation of the package in NuGet.

Copilot AI review requested due to automatic review settings February 7, 2026 02: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

Updates the SQL Database Projects integration documentation to use the correct API surface when adding a SQL project resource, matching the NuGet package README and preventing copy/paste runtime errors.

Changes:

  • Switches the AddSqlProject example from calling the method on the SQL database resource to calling it on the app builder.
  • Adds an explicit .WithReference(sql) to associate the SQL project deployment with the target database.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

…ts.mdx

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@IEvangelist
Copy link
Member

Hey @kattschan, there's no changes in this PR.

@kattschan
Copy link
Author

Sorry about that! I'm a git novice and must not have looked too closely at the copilot suggestion. Looks like it's showing the changes now.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants