Skip to content

Omit pre-release suffix for Fabric in VS Code changelog sync#3008

Merged
conniey merged 1 commit into
mainfrom
amhersch/fabric-vscode-changelog-no-prerelease
Jul 8, 2026
Merged

Omit pre-release suffix for Fabric in VS Code changelog sync#3008
conniey merged 1 commit into
mainfrom
amhersch/fabric-vscode-changelog-no-prerelease

Conversation

@AmosHersch

Copy link
Copy Markdown
Contributor

What does this PR do?

Updates the VS Code changelog sync script so the Fabric MCP Server does not get a (pre-release) suffix on its generated changelog entries.

  • eng/scripts/Sync-VsCodeChangelog.ps1 now selects the heading suffix based on the changelog path: Fabric gets no suffix, while other servers (Azure, Template, etc.) keep (pre-release) as before.

This matches how the Fabric VS Code extension changelog entries are formatted (e.g., ## 1.1.0 (2026-06-15)) and removes the need to manually strip the suffix during each Fabric release.

Engineering tooling change — no product code or tools were modified.

GitHub issue number?

N/A

Pre-merge Checklist

  • Required for All PRs
    • Read contribution guidelines
    • PR title clearly describes the change
    • Commit history is clean with descriptive messages
    • Added comprehensive tests for new/modified functionality — N/A (build/release script change)

Invoking Livetests

Copilot submitted PRs are not trustworthy by default. Users with write access to the repo need to validate the contents of this PR before leaving a comment with the text /azp run mcp - pullrequest - live. This will trigger the necessary livetest workflows to complete required validation.

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

Updates the VS Code changelog sync script so the Fabric MCP Server’s generated VS Code changelog headings do not include the default (pre-release) suffix, aligning Fabric’s formatting with existing Fabric VS Code extension entries while preserving current behavior for other servers.

Changes:

  • Add a $preReleaseSuffix value derived from -ChangelogPath (empty for Fabric; (pre-release) otherwise).
  • Use the computed suffix when emitting the ## <version> (<date>)... heading in the generated VS Code changelog entry.

@github-project-automation github-project-automation Bot moved this from Untriaged to In Progress in Azure MCP Server Jul 8, 2026
@conniey conniey merged commit 2e0e8e7 into main Jul 8, 2026
27 checks passed
@github-project-automation github-project-automation Bot moved this from In Progress to Done in Azure MCP Server Jul 8, 2026
@conniey conniey deleted the amhersch/fabric-vscode-changelog-no-prerelease branch July 8, 2026 14:25
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.

3 participants