Skip to content

Conversation

@coadaflorin
Copy link
Contributor

Made a small change to remove this comment from the changelog. It shouldn't be in there as it's an internal change.

Copilot AI review requested due to automatic review settings February 21, 2025 10:45
@coadaflorin coadaflorin requested a review from a team as a code owner February 21, 2025 10:45
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.

PR Overview

This PR removes an internal comment from the changelog and release notes that should not be publicly visible.

  • Removed an internal comment regarding the .NET 9 requirement from the changelog file.
  • Removed the same internal comment from the release notes file.

Reviewed Changes

File Description
csharp/ql/lib/CHANGELOG.md Removed internal comment about .NET 9 build requirement.
csharp/ql/lib/change-notes/released/3.1.1.md Removed internal comment about .NET 9 build requirement.

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

Tip: Copilot only keeps its highest confidence comments to reduce noise and keep you focused. Learn more

Copy link
Contributor

@michaelnebel michaelnebel left a comment

Choose a reason for hiding this comment

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

LGTM

@coadaflorin coadaflorin merged commit 967c1ad into main Feb 21, 2025
17 checks passed
@coadaflorin coadaflorin deleted the coadaflorin/csharp-changelog-clean branch February 21, 2025 11:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants