Skip to content

git_ui: Ignore extraneous links in git output#49817

Merged
MrSubidubi merged 2 commits intozed-industries:mainfrom
kbidlack:ignore-non-request-links
Feb 21, 2026
Merged

git_ui: Ignore extraneous links in git output#49817
MrSubidubi merged 2 commits intozed-industries:mainfrom
kbidlack:ignore-non-request-links

Conversation

@kbidlack
Copy link
Contributor

@kbidlack kbidlack commented Feb 21, 2026

Closes #49746

Previously, the LinkFinder looked at all of the lines, which sometimes includes links unrelated to the pull request, like a post quantum cryptography warning that links to this article.

Release Notes:

  • When searching for pull request links in git output, only lines that start with remote: are searched.

@cla-bot cla-bot bot added the cla-signed The user has signed the Contributor License Agreement label Feb 21, 2026
Copy link
Member

@MrSubidubi MrSubidubi left a comment

Choose a reason for hiding this comment

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

Nice! Very good find and an even better fix for this. Thank you for this and also updating the test with it! Very nice contribution.

@MrSubidubi MrSubidubi merged commit da93e7d into zed-industries:main Feb 21, 2026
27 checks passed
@kbidlack kbidlack deleted the ignore-non-request-links branch February 23, 2026 17:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla-signed The user has signed the Contributor License Agreement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Clicking button to view GitLab merge request loads explainer on post-quantum cryptography

2 participants