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

open-pr: fix the Git LFS link to use when opening a PR #103

Merged
merged 1 commit into from
Nov 28, 2024

Conversation

dscho
Copy link
Member

@dscho dscho commented Nov 28, 2024

It has been reported that the Git LFS link in the release notes was incorrect.

The reason is that we have some special handling of Git LFS so that we do not get the (rather ugly) tag URL but instead the release URL.

Example: git-for-windows/git#5266 refers to https://github.com/git-lfs/git-lfs/releases/tag/v3.6.0 because this is the URL from the feed from which the monitor-components gets it

The corresponding PR (opened via /open pr) has the link https://github.com/git-lfs/git-lfs/releases/tag/3.6.0 (note the missing v after the last slash).

This PR ensures that future /open prs in Git LFS issues will use the correct link.

It has been reported in
git-for-windows/git#5276 (comment)
that the Git LFS link in the release notes was incorrect.

The reason is that we have some special handling of Git LFS so that we
do not get the (rather ugly) _tag_ URL but instead the _release_ URL.

Example: git-for-windows/git#5266 refers to
https://github.com/git-lfs/git-lfs/releases/tag/v3.6.0 because this is
the URL from the feed from which the `monitor-components` gets it:
https://github.com/git-for-windows/git/blob/v2.47.1.windows.1/.github/workflows/monitor-components.yml#L29-L3

The corresponding PR at
git-for-windows/build-extra#581 (opened via
`/open pr` at
git-for-windows/git#5266 (comment))
has the link https://github.com/git-lfs/git-lfs/releases/tag/3.6.0 (note
the missing `v` after the last slash).

This commit ensures that future `/open pr`s in Git LFS issues will use
the correct link.

Signed-off-by: Johannes Schindelin <[email protected]>
@dscho dscho requested a review from rimrul November 28, 2024 08:10
@dscho dscho self-assigned this Nov 28, 2024
@rimrul rimrul merged commit 87bebe7 into git-for-windows:main Nov 28, 2024
@dscho dscho deleted the fix-git-lfs-link-in-/open-pr branch November 29, 2024 10:51
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