Skip to content

Conversation

@BYK
Copy link
Member

@BYK BYK commented Jan 13, 2026

Disable PR links in changelog previews to prevent unwanted GitHub cross-references.

The changelog preview feature (e.g., craft changelog --pr) was generating markdown links for PRs, which caused GitHub to create cross-reference notifications on those PRs. This change formats PR numbers as plain backticked text (e.g., `#123`) in previews, avoiding the unwanted notifications while keeping full links for actual release changelogs.

Fixes #711

The changelog preview was creating markdown links to PRs like
[#123](url), which caused GitHub to automatically post a comment
on each mentioned PR saying it was referenced. This was noisy
since the PRs don't usually have anything in common other than
being in the same release.

This change adds a `disablePRLinks` parameter that formats PR
references using backticks (`#123`) instead of markdown links,
preventing GitHub from auto-linking and creating cross-references.

The fix is applied specifically to the changelog preview feature
(used in the GitHub Actions workflow) by passing `disablePRLinks: true`
when calling `serializeChangelog()` from `generateChangelogWithHighlight()`.

Regular changelog generation (for actual releases) continues to use
full markdown links as before.

Fixes #78da
@cursor
Copy link
Contributor

cursor bot commented Jan 13, 2026

Cursor Agent can help with this pull request. Just @cursor in comments and I'll start working on changes in this branch.
Learn more about Cursor Agents

@github-actions
Copy link
Contributor

github-actions bot commented Jan 13, 2026

Semver Impact of This PR

🟢 Patch (bug fixes)

📋 Changelog Preview

This is how your changes will appear in the changelog.
Entries from this PR are highlighted with a left border (blockquote style).


Bug Fixes 🐛

  • (docker) Add GITHUB_API_TOKEN and x-access-token fallbacks for ghcr.io by BYK in #710
  • Don't mention PRs to avoid linking in changelog previews by BYK in #712

🤖 This preview updates automatically when you update the PR.

@BYK BYK changed the title Changelog preview mentions fix: Don't mention PRs to avoid linking in changelog previews Jan 13, 2026
@BYK BYK marked this pull request as ready for review January 13, 2026 14:08
@BYK BYK enabled auto-merge (squash) January 13, 2026 14:09
@BYK BYK requested a review from sentrivana January 13, 2026 14:09
Copy link

@sentrivana sentrivana left a comment

Choose a reason for hiding this comment

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

🙏🏻

@BYK BYK merged commit bca0ad7 into master Jan 13, 2026
18 checks passed
@BYK BYK deleted the cursor/changelog-preview-mentions-78da branch January 13, 2026 15:28
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.

Changelog preview pings all mentioned PRs

4 participants