Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Retroactively add SDK release notes #548

Merged
merged 1 commit into from
Jan 6, 2025
Merged

Conversation

zijchen
Copy link
Member

@zijchen zijchen commented Jan 3, 2025

No description provided.

Choose a reason for hiding this comment

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

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

Files not reviewed (7)
  • release-notes/Microsoft.Build.Sql/0.1.3-preview.md: Evaluated as low risk
  • release-notes/Microsoft.Build.Sql/0.1.7-preview.md: Evaluated as low risk
  • release-notes/Microsoft.Build.Sql/0.1.9-preview.md: Evaluated as low risk
  • release-notes/Microsoft.Build.Sql/0.2.0-preview.md: Evaluated as low risk
  • release-notes/Microsoft.Build.Sql/0.2.3-preview.md: Evaluated as low risk
  • release-notes/Microsoft.Build.Sql/0.2.4-preview.md: Evaluated as low risk
  • release-notes/Microsoft.Build.Sql/README.md: Evaluated as low risk
Comments suppressed due to low confidence (1)

release-notes/Microsoft.Build.Sql/0.1.15-preview.md:14

  • The error message 'the Value cannot be null' should be enclosed in quotes to indicate that it is an error message.
Fixed an issue where references to system tables in a values clause would fail to build with an error message that the Value cannot be null.
@ErikEJ
Copy link
Contributor

ErikEJ commented Jan 3, 2025

This is nice, but updated docs wiki would add more value.

@dzsquared
Copy link
Contributor

This is nice, but updated docs wiki would add more value.

ack - continuing work on the MS Learn docs, including more details on the SDK.

@zijchen zijchen merged commit 5082e30 into main Jan 6, 2025
7 checks passed
@zijchen zijchen deleted the zijchen/more-release-notes branch January 6, 2025 17:59
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.

3 participants