Skip to content

Fix PR diff links and preserve workspace shorthand deps#44

Merged
theoephraim merged 2 commits into
mainfrom
fix/pr-diff-links-and-workspace-shorthand
Apr 22, 2026
Merged

Fix PR diff links and preserve workspace shorthand deps#44
theoephraim merged 2 commits into
mainfrom
fix/pr-diff-links-and-workspace-shorthand

Conversation

@theoephraim

Copy link
Copy Markdown
Member

Summary

  • Fix all PR diff links (version PR body + PR comments) to use full absolute URLs with /changes path and sha256 anchors, so they work when clicked from the conversation tab
  • Remove package.json link from version PR diff links (only link CHANGELOG.md)
  • For new version PRs, update the body after creation to backfill links with the correct PR number
  • Preserve workspace:^, workspace:~, and workspace:* shorthand ranges during versioning instead of resolving them to explicit versions

Test plan

  • Existing tests pass (196 pass, 0 fail)
  • New test for workspace shorthand preservation
  • Verify diff links work on a real version PR

- Use full absolute URLs with /changes path and sha256 anchors for all
  PR diff links (version PR body, PR comments)
- Remove package.json link from version PR (only link CHANGELOG.md)
- For new version PRs, update body after creation to include correct links
- Preserve workspace:^, workspace:~, workspace:* shorthand ranges during
  versioning instead of resolving them to explicit versions
@bumpy-bot

Copy link
Copy Markdown
Collaborator

bumpy-frog

The changes in this PR will be included in the next version bump.

patch Patch releases

  • @varlock/bumpy 1.2.0 → 1.2.1

Bump files in this PR

Click here if you want to add another bump file to this PR


This comment is maintained by bumpy.

@theoephraim theoephraim merged commit be450e9 into main Apr 22, 2026
2 checks passed
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